Page MenuHomePhabricator
Feed All Stories

Aug 5 2016

magnetik created T11434: Arc patch fails: EXCEPTION: (ConduitClientException) ERR-INVALID-AUTH: Unknown Error..
Aug 5 2016, 9:20 AM · Restricted Project, Arcanist, Bug Report
magnetik added a comment to Q398: Arc patch fails: EXCEPTION: (ConduitClientException) ERR-INVALID-AUTH: Unknown Error.

I've added loggers in libphutil/conduit, and found out that in the differential.querydiffs requests, the conduitToken property is null.

Aug 5 2016, 8:45 AM · Arcanist
reet- added a comment to T9640: Make Phabricator compatible with PHP7.

Indeed very annoying. Updated our instance to 16.04.1 yesterday, and my day was not so happy any more. Of course this is not Phabricators fault, but I doubt that reading the Ubuntu release notes would have helped as the problem is not that obvious.

Aug 5 2016, 7:49 AM · Prioritized, Restricted Project, Infrastructure
reet- created T11433: MySQL "-p<password>" flag produces a warning in recent versions of MySQL.
Aug 5 2016, 7:41 AM · Bug Report
chad added a subtask for T9379: Build Phriction v3: T8172: "Edit" button shouldn't be hidden in "Actions".
Aug 5 2016, 1:48 AM · Phriction
chad added a parent task for T8172: "Edit" button shouldn't be hidden in "Actions": T9379: Build Phriction v3.
Aug 5 2016, 1:48 AM · FreeBSD, Restricted Project, Phriction
chad added subtasks for T9379: Build Phriction v3: T8552: Increase user control over document hierarchy in Phriction, T8082: Exception when moving Phriction document into place of old one, T7783: phriction diffs don't default to remarkup, T4274: Creating a document from a link doesn't preserve case, T8708: Undefined variable: message when trying to move a phriction page to a lower level, T4815: Navigation buttons "Next version" and "Previous version" for phriction-documents, T8970: Ability to draft/archive Phriction documents, T9622: Allow Phriction pages to have Project tags, T7878: strange "moved from" signal in [Phriction] diff.
Aug 5 2016, 1:48 AM · Phriction
chad added a parent task for T4815: Navigation buttons "Next version" and "Previous version" for phriction-documents: T9379: Build Phriction v3.
Aug 5 2016, 1:48 AM · Badge Awarded, Phriction
chad added a parent task for T4274: Creating a document from a link doesn't preserve case: T9379: Build Phriction v3.
Aug 5 2016, 1:48 AM · Phriction
chad added a parent task for T8082: Exception when moving Phriction document into place of old one: T9379: Build Phriction v3.
Aug 5 2016, 1:48 AM · Restricted Project, Support Impact, Phriction
chad added a parent task for T7783: phriction diffs don't default to remarkup: T9379: Build Phriction v3.
Aug 5 2016, 1:48 AM · Remarkup, Phriction
chad added a parent task for T7878: strange "moved from" signal in [Phriction] diff: T9379: Build Phriction v3.
Aug 5 2016, 1:48 AM · Differential
fernofry awarded M1466: Phriction v3 a Love token.
Aug 5 2016, 12:19 AM
epriestley added a project to T4245: Make repository URIs slightly prettier: Prioritized.
Aug 5 2016, 12:01 AM · Prioritized, Diffusion (v3), KDE, Restricted Project

Aug 4 2016

jaufrecht added a comment to T10915: Allow story points to just be task count.

But I agree with Kevin that the point limit seems a lot less useful than the task limit. They're both useful for different reasons, but having been a scrum master and on agile teams for the past 10 years, I find the task limit more fundamental to the processes I've seen.

Aug 4 2016, 10:27 PM · Restricted Project, Feature Request
epriestley closed T11431: ./bin/* files don't seem to be valid executables as Resolved.

Cool -- I don't think we can really detect this because we don't have an opportunity to run any code to check for it. We could possibly have the web UI run a setup check, but I think we get about 1 of these a year and it's reasonably likely that users try to run bin/* before hitting the web UI anyway. If you have any ideas about how we could have caught/prevented this, let us know.

Aug 4 2016, 10:27 PM · Bug Report (Needs Information)
ryanniehaus added a comment to T11431: ./bin/* files don't seem to be valid executables.

Ah, that was the case, I used sftp to transfer the data because the machine I'm trying to use it on is behind a restrictive firewall. Sorry for the confusion..

Aug 4 2016, 10:07 PM · Bug Report (Needs Information)
chad edited projects for T11431: ./bin/* files don't seem to be valid executables, added: Bug Report (Needs Information); removed Bug Report.
Aug 4 2016, 9:52 PM · Bug Report (Needs Information)
• st_we awarded T5514: Allow dashboard panels to have an auto-refresh time a Love token.
Aug 4 2016, 8:56 PM · Restricted Project, Dashboards
donaldguy added a comment to T11432: Ability to exclude self ("Current Viewer") from Feed panels.

In general some kind of batching (e.g. "Donald commented on T### , approved D###, did it N other actions, click to expand") would also address this, but a view level filter seemed like an easier ask

Aug 4 2016, 7:55 PM · Feed, Feature Request
chad removed a project from T11432: Ability to exclude self ("Current Viewer") from Feed panels: Dashboards.
Aug 4 2016, 7:54 PM · Feed, Feature Request
donaldguy added projects to T11432: Ability to exclude self ("Current Viewer") from Feed panels: Feed, Dashboards.
Aug 4 2016, 7:50 PM · Feed, Feature Request
donaldguy created T11432: Ability to exclude self ("Current Viewer") from Feed panels.
Aug 4 2016, 7:50 PM · Feed, Feature Request
chad added a comment to T8552: Increase user control over document hierarchy in Phriction.

Some sketches in M1466, but essentially we'll give you some default options, and a "navigation builder" which you can select on a per-page basis. Based on the mocks, the "hierarchy" moves to being a side navigation, and defaults now to one level. From that you can "Manage Page" and change the default for that page. Rough thinking is:

Aug 4 2016, 7:48 PM · Phriction, Restricted Project
swisspol awarded M1466: Phriction v3 a Like token.
Aug 4 2016, 7:46 PM
chad added 1 mock(s) for T8552: Increase user control over document hierarchy in Phriction: M1466: Phriction v3.
Aug 4 2016, 7:44 PM · Phriction, Restricted Project
chad added 1 task(s) for M1466: Phriction v3: T8552: Increase user control over document hierarchy in Phriction.
Aug 4 2016, 7:44 PM
chad updated the task description for T9379: Build Phriction v3.
Aug 4 2016, 7:43 PM · Phriction
chad added 1 mock(s) for T9379: Build Phriction v3: M1466: Phriction v3.
Aug 4 2016, 7:39 PM · Phriction
chad added 1 task(s) for M1466: Phriction v3: T9379: Build Phriction v3.
Aug 4 2016, 7:39 PM
chad updated images of M1466: Phriction v3.
Aug 4 2016, 7:35 PM
epriestley added a comment to T11431: ./bin/* files don't seem to be valid executables.

They should be symlinks. As git represents both symlinks and the executable flag, it should be unnecessary to chmod them.

Aug 4 2016, 7:22 PM · Bug Report (Needs Information)
ryanniehaus created T11431: ./bin/* files don't seem to be valid executables.
Aug 4 2016, 7:14 PM · Bug Report (Needs Information)
eadler added a project to T9529: Upload coverage to a diff: Restricted Project.
Aug 4 2016, 6:47 PM · Restricted Project, Restricted Project, Conduit, Differential, Harbormaster
epriestley merged T11430: Add PyLint3 to Arcanist Linters into T10038: Plan the mid-term pathway for unit-test/linter bindings.
Aug 4 2016, 6:41 PM · Lint, Arcanist
epriestley merged task T11430: Add PyLint3 to Arcanist Linters into T10038: Plan the mid-term pathway for unit-test/linter bindings.
Aug 4 2016, 6:41 PM · Lint, Arcanist, Feature Request
avivey added a comment to T11430: Add PyLint3 to Arcanist Linters.

If it functions identically to pylint, then you can use the same linter binding and set bin to the new executable...

Aug 4 2016, 6:31 PM · Lint, Arcanist, Feature Request
CodeMouse92 awarded T8145: Support global/broadcast notifications a Mountain of Wealth token.
Aug 4 2016, 6:17 PM · Notifications
CodeMouse92 created T11430: Add PyLint3 to Arcanist Linters.
Aug 4 2016, 6:17 PM · Lint, Arcanist, Feature Request
epriestley added a comment to T11371: Restore PHP to Glory.

ICFP2016 starts in about 6 hours:

Aug 4 2016, 6:16 PM · Public & Media Relations
epriestley created an object: C146: ICFP Contest 2016.
Aug 4 2016, 6:08 PM
epriestley added a comment to T8145: Support global/broadcast notifications.

(Possibly it might be easier to select the correct audience in Phabricator since you can use projects or other rules, but all of the use cases here and elsewhere just want to hit everyone, I think.)

Aug 4 2016, 5:32 PM · Notifications
epriestley added a comment to T8145: Support global/broadcast notifications.

Yeah, basically like the Facebook Rooster system except making it act as "pinned notifications" instead of "pinned dashboard panels". We could do either, reasonably, although it's slightly messier to do a real-time notification of "a new panel appeared on all dashboards, go look". But I'm basically not sure why this is better than email in any case where the users are employees.

Aug 4 2016, 5:30 PM · Notifications
epriestley added a comment to T8145: Support global/broadcast notifications.

I think a global banner (or some equally omnipresent, persistent element) is reasonable for "this system is going down in a couple of minutes". But if that's the only really good use case we have for it, though, I wonder if we shouldn't just build a "shutdown in X minutes" feature that automatically does this, counts down, stops new batch jobs from starting, maybe pauses builds, puts the system into read-only when the countdown finishes, etc. -- a dedicated "graceful shutdown" feature instead of just a broadcast feature. I don't think any of these things are really problems today, but I could imagine more of a "shutdown process" being useful in the future.

Aug 4 2016, 5:27 PM · Notifications
chad added a comment to T8145: Support global/broadcast notifications.

The "Rooster" system we had at Facebook is essentially a panel (closable) that would persist on every homepage. I would be OK forcing a panel onto all dashboards if the users can also close it. I think we need closable panels for NUX anyways. But the issue is this is still a "pull" notification, meaning the user has to visit a page to get the information. A "push" system (notification, email) is better for things like downtime. "pull" systems are good for just general FYI stuff (all-hands meeting upcoming).

Aug 4 2016, 5:23 PM · Notifications
bjshively added a comment to T8145: Support global/broadcast notifications.

To your point, we did have some users complain that even just on the primary dashboard a ticking timer was a bit distracting. But, you know, when your boss's boss's boss asks if something is feasible, you at least look into it. :) And I think countdown scenario aside, a global information display that doesn't assume the user has some specific dashboard config is potentially valuable.

Aug 4 2016, 5:10 PM · Notifications
bjshively added a comment to T8145: Support global/broadcast notifications.

@epriestley, somewhat an edict from on high, but I do think there is potentially some value in a persistent banner (a single row tall, similar to the main phab bar but perhaps even a bit skinnier.)

Aug 4 2016, 5:08 PM · Notifications
johnny-bit awarded T11428: Upcoming: Personal Herald Rules No Longer Semi-Private a Like token.
Aug 4 2016, 4:54 PM · Policy, Installing & Upgrading, Herald
epriestley added a comment to T11427: Banner/MOTD/way to display a persistent message on all pages?.

Yeah, agreed. I added a little discussion to T8145.

Aug 4 2016, 4:52 PM · Feature Request
epriestley updated subscribers of T8145: Support global/broadcast notifications.

@bjshively, for this use case:

Aug 4 2016, 4:49 PM · Notifications
chad added a comment to T11427: Banner/MOTD/way to display a persistent message on all pages?.

Strong preference overall to using Dashboards / Notifications. Persistent banner on all pages generally reduces employee productivity just to announce an all-hands or ice cream social, doesn't seem like the right solution.

Aug 4 2016, 4:46 PM · Feature Request
epriestley added a comment to T8145: Support global/broadcast notifications.
  • T11427 is similar.
  • T10855 discusses a similar and possibly overlapping use case (mirroring system status into Phabricator), but I believe that use case isn't after global/on-all-pages notifications.
Aug 4 2016, 4:44 PM · Notifications
epriestley renamed T8145: Support global/broadcast notifications from Send notification from command line? to Support global/broadcast notifications.
Aug 4 2016, 4:37 PM · Notifications
epriestley added a comment to T11427: Banner/MOTD/way to display a persistent message on all pages?.

T8145 is also related, and maybe a little closer to intent here. I can retarget that.

Aug 4 2016, 4:35 PM · Feature Request
epriestley created T11429: Upcoming: Changes to Arcanist.
Aug 4 2016, 4:07 PM · Public & Media Relations, Installing & Upgrading, Arcanist
clemvangelis added a comment to T10893: repo update fails for repos with 1000s of closing branches.

I actually encouter the same problem. Setting autoclose on master only fixed it.

Aug 4 2016, 3:29 PM · Diffusion, Bug Report
epriestley added a comment to T9410: Policy behavior for Herald rules is murky and inconsistent.

T11428 puts the corporate communication wheels in motion to make this change.

Aug 4 2016, 3:23 PM · Restricted Project, Herald, Restricted Project
epriestley added a subtask for T11428: Upcoming: Personal Herald Rules No Longer Semi-Private: T9410: Policy behavior for Herald rules is murky and inconsistent.
Aug 4 2016, 3:20 PM · Policy, Installing & Upgrading, Herald
epriestley added a parent task for T9410: Policy behavior for Herald rules is murky and inconsistent: T11428: Upcoming: Personal Herald Rules No Longer Semi-Private.
Aug 4 2016, 3:20 PM · Restricted Project, Herald, Restricted Project
michaeljs1990 awarded T11427: Banner/MOTD/way to display a persistent message on all pages? a Hungry Hippo token.
Aug 4 2016, 3:20 PM · Feature Request
chad merged T11427: Banner/MOTD/way to display a persistent message on all pages? into T10855: Allow external systems to edit the content of dashboard panels so they can publish information.
Aug 4 2016, 3:20 PM · Dashboards, Restricted Project, Conduit, Feature Request
chad merged task T11427: Banner/MOTD/way to display a persistent message on all pages? into T10855: Allow external systems to edit the content of dashboard panels so they can publish information.
Aug 4 2016, 3:20 PM · Feature Request
michaeljs1990 updated the task description for T11427: Banner/MOTD/way to display a persistent message on all pages?.
Aug 4 2016, 3:18 PM · Feature Request
epriestley created T11428: Upcoming: Personal Herald Rules No Longer Semi-Private.
Aug 4 2016, 3:16 PM · Policy, Installing & Upgrading, Herald
bjshively created T11427: Banner/MOTD/way to display a persistent message on all pages?.
Aug 4 2016, 3:12 PM · Feature Request
epriestley moved T9410: Policy behavior for Herald rules is murky and inconsistent from Backlog to Next on the Herald board.
Aug 4 2016, 2:59 PM · Restricted Project, Herald, Restricted Project
epriestley added a comment to T11391: Unable to add uri to Diffusion.

Update past rPa5b26104 (Jul 9).

Aug 4 2016, 2:51 PM · Bug Report (Needs Information), Diffusion
• marfago added a comment to T11391: Unable to add uri to Diffusion.

Sorry I missed the previous notification.

Aug 4 2016, 2:48 PM · Bug Report (Needs Information), Diffusion
epriestley closed T11391: Unable to add uri to Diffusion as Invalid.

Closing this since we haven't seen a response for about a week, this report didn't follow the instructions by omitting version information, and it's very likely that this is resolved by updating.

Aug 4 2016, 2:40 PM · Bug Report (Needs Information), Diffusion
chad edited projects for T11426: Issues while review commiting the code, added: Bug Report (Needs Information), Differential; removed Bug Report.
Aug 4 2016, 2:37 PM · Differential, Bug Report (Needs Information)
chad added Q458: How to install existing database? (Answer 427).
Aug 4 2016, 2:15 PM
chad added a comment to T11426: Issues while review commiting the code.

Reproduction steps are required for all bug reports. How can we reproduce the error you are seeing?

Aug 4 2016, 2:12 PM · Differential, Bug Report (Needs Information)
• st_we added a comment to P1792 PhabricatorGermanTranslation.php.

I've changed and added some translation in the Paste. In Addition you can change two config variables to have german Maniphest status and priorities :

Aug 4 2016, 11:46 AM · Localization
• st_we edited P1792 PhabricatorGermanTranslation.php.
Aug 4 2016, 11:45 AM · Localization
gokulpd created T11426: Issues while review commiting the code.
Aug 4 2016, 11:12 AM · Differential, Bug Report (Needs Information)
• st_we added a comment to T5267: Localize Phabricator.
Aug 4 2016, 9:18 AM · Prioritized, Localization
• alorencwork updated Q458: How to install existing database? from to How to install existing database?.
Aug 4 2016, 8:45 AM
• st_we awarded P1792 PhabricatorGermanTranslation.php a Like token.
Aug 4 2016, 8:35 AM · Localization
• st_we edited P1792 PhabricatorGermanTranslation.php.
Aug 4 2016, 8:35 AM · Localization
featherless awarded M1466: Phriction v3 a Like token.
Aug 4 2016, 5:57 AM
chad updated images of M1466: Phriction v3.
Aug 4 2016, 3:42 AM
venky created T11425: Add "Locate File" functionality to branch view in Diffusion.
Aug 4 2016, 1:53 AM · Feature Request
epriestley closed T11424: Differential inline comments (their positions) are misleading after new diff as Resolved.

Yeah, T7447 is probably the best source of discussion about this.

Aug 4 2016, 1:10 AM · Bug Report
avivey added a comment to T11424: Differential inline comments (their positions) are misleading after new diff.

There's lots of discussion about this in T7447 and related T1460.

Aug 4 2016, 1:01 AM · Bug Report
avivey updated the task description for T10856: Small Revision page enhanchments.
Aug 4 2016, 12:55 AM · Restricted Project, Feature Request, Differential
venky created T11424: Differential inline comments (their positions) are misleading after new diff.
Aug 4 2016, 12:55 AM · Bug Report

Aug 3 2016

epriestley closed T11420: Stable sort project tags in query results as Resolved by committing Restricted Diffusion Commit.
Aug 3 2016, 11:46 PM · Projects, Feature Request
epriestley added a revision to T7939: Support Herald rules for Calendar events: Unknown Object (Differential Revision).
Aug 3 2016, 11:46 PM · Herald, Calendar
epriestley added a comment to T11420: Stable sort project tags in query results.

D16367 make the order consistent (instead of reversed) in the task list view and on workboard cards.

Aug 3 2016, 11:20 PM · Projects, Feature Request
epriestley added a revision to T11420: Stable sort project tags in query results: Unknown Object (Differential Revision).
Aug 3 2016, 11:16 PM · Projects, Feature Request
epriestley closed T9719: Herald test console doesn't appear to work for commit hooks (pre-receive), and also has a big `instanceof` mess as Resolved by committing Restricted Diffusion Commit.
Aug 3 2016, 11:12 PM · Documentation, Herald
epriestley closed T9719: Herald test console doesn't appear to work for commit hooks (pre-receive), and also has a big `instanceof` mess, a subtask of T7939: Support Herald rules for Calendar events, as Resolved.
Aug 3 2016, 11:12 PM · Herald, Calendar
venky added a comment to T11423: arcanist segmentation faults.
$ php --version
PHP 7.1.0beta1 (cli) (built: Jul 25 2016 13:58:06) ( NTS )
Copyright (c) 1997-2016 The PHP Group
Zend Engine v3.1.0-dev, Copyright (c) 1998-2016 Zend Technologies

That might explain it (not sure how I got on the beta, wasn't aiming to). I've switched to the 7.0 version, will see if that fixes it.

Aug 3 2016, 10:45 PM · Bug Report (Needs Information)
chad added a comment to T11423: arcanist segmentation faults.

What PHP are you running? I seem to recall an issue with one of the brew packages.

Aug 3 2016, 10:38 PM · Bug Report (Needs Information)
venky created T11423: arcanist segmentation faults.
Aug 3 2016, 10:11 PM · Bug Report (Needs Information)
chad added a comment to T8970: Ability to draft/archive Phriction documents.

Combining Draft / Archive into a single task since it's easier to build both at once.

Aug 3 2016, 8:32 PM · Phriction
chad renamed T8970: Ability to draft/archive Phriction documents from Ability to draft Phriction documents to Ability to draft/archive Phriction documents.
Aug 3 2016, 8:31 PM · Phriction
chad merged task T7638: Archiving wiki pages in Phriction into T8970: Ability to draft/archive Phriction documents.
Aug 3 2016, 8:31 PM · Phriction
chad merged T7638: Archiving wiki pages in Phriction into T8970: Ability to draft/archive Phriction documents.
Aug 3 2016, 8:31 PM · Phriction
markwei merged T11192: Differential should have a "Not done comments" section. into T8250: Would like to see all comments and their status in a list.
Aug 3 2016, 6:59 PM · Inline Comments, Prioritized, Restricted Project, Differential