Page MenuHomePhabricator
Feed Advanced Search

Feb 25 2016

epriestley added a comment to T10452: Provide more flexible management over disabling/changing ldap user accounts.

Some possible tools that may form part of a solution here, roughly in order from greatest upstream willingness to greatest upstream resistance:

Feb 25 2016, 9:14 PM · Restricted Project, LDAP, Auth, Feature Request
epriestley updated subscribers of T3353: Make Phriction Document changes reviewable.
Feb 25 2016, 4:53 PM · Restricted Project, Prioritized, Phriction
epriestley merged task T10451: Phriction change diff or diffstat into T3353: Make Phriction Document changes reviewable.
Feb 25 2016, 4:52 PM · Feature Request
epriestley added a comment to T10449: Almanac v2 (Unprototype).

As a note to myself, I also need to adjust config for instances after pushing this, since empty values are now explicit. This only affects a handful of instances and they're easy to identify, so I may just do it manually rather than trying to write a migration.

Feb 25 2016, 4:51 PM · Almanac (v2), Phacility
epriestley added a comment to T10449: Almanac v2 (Unprototype).

I also have a small patch against rCORE, but it's pretty minor and I can't really test it fully until the rest of this stuff makes it into production.

Feb 25 2016, 4:45 PM · Almanac (v2), Phacility
epriestley added a revision to T10449: Almanac v2 (Unprototype): Unknown Object (Differential Revision).
Feb 25 2016, 4:43 PM · Almanac (v2), Phacility
epriestley added a revision to T10152: Allow uri.allowed-protocols to be configured in the Phacility cluster: Unknown Object (Differential Revision).
Feb 25 2016, 4:42 PM · Phacility
epriestley added a revision to T10449: Almanac v2 (Unprototype): Unknown Object (Differential Revision).
Feb 25 2016, 4:42 PM · Almanac (v2), Phacility
epriestley added a revision to T10449: Almanac v2 (Unprototype): Unknown Object (Differential Revision).
Feb 25 2016, 4:39 PM · Almanac (v2), Phacility
epriestley added a comment to T10450: Owner locked out of project.

Subscribers were removed from projects, so this policy no longer functions.

Feb 25 2016, 4:31 PM · Projects, Bug Report
epriestley renamed T10449: Almanac v2 (Unprototype) from Almanac v1 (Unprototype) to Almanac v2 (Unprototype).
Feb 25 2016, 2:06 PM · Almanac (v2), Phacility
epriestley added a revision to T10449: Almanac v2 (Unprototype): Unknown Object (Differential Revision).
Feb 25 2016, 2:05 PM · Almanac (v2), Phacility
epriestley created T10449: Almanac v2 (Unprototype).
Feb 25 2016, 1:40 PM · Almanac (v2), Phacility
epriestley added a comment to T9762: Removing devices from Almanac services.

Er, almanac.service.search, rather.

Feb 25 2016, 1:00 PM · Almanac (v2)
epriestley added a comment to T9762: Removing devices from Almanac services.

D15345 adds these capabilities:

Feb 25 2016, 1:00 PM · Almanac (v2)
epriestley added a revision to T9762: Removing devices from Almanac services: Unknown Object (Differential Revision).
Feb 25 2016, 12:56 PM · Almanac (v2)
epriestley added a revision to T10246: Deploy Drydock in the Phacility cluster: Unknown Object (Differential Revision).
Feb 25 2016, 12:56 PM · Harbormaster (v3), Drydock (v3), Almanac (v2), Phacility
epriestley closed T6741: Lock Almanac networks, and allow Almanac domain namespaces to be locked as Resolved by committing Restricted Diffusion Commit.
Feb 25 2016, 11:38 AM · Almanac (v2), Security, Phacility
epriestley closed T6741: Lock Almanac networks, and allow Almanac domain namespaces to be locked, a subtask of T5833: Build "Almanac", a service/host/device directory, as Resolved.
Feb 25 2016, 11:38 AM · Phacility, Almanac
epriestley closed T6741: Lock Almanac networks, and allow Almanac domain namespaces to be locked, a subtask of T10246: Deploy Drydock in the Phacility cluster, as Resolved.
Feb 25 2016, 11:38 AM · Harbormaster (v3), Drydock (v3), Almanac (v2), Phacility
epriestley added a comment to T10447: When the push phase of "Land Revision" fails, show the error in the UI.

I think var_dump() will normally be hard to observe, since it will go to stdout but that isn't connected to anything by default so it will just vanish (we don't write stdout to the daemon log). My guess is that the code is running, you just aren't seeing the var_dump() output.

Feb 25 2016, 11:20 AM · Drydock (v3)
epriestley added a revision to T4245: Make repository URIs slightly prettier: Unknown Object (Differential Revision).
Feb 25 2016, 11:09 AM · Prioritized, Diffusion (v3), KDE, Restricted Project
wschroo awarded T4900: Workboards updating in real time a Like token.
Feb 25 2016, 10:43 AM · Workboards (v3), User Delight, Wikimedia

Feb 24 2016

epriestley added a comment to T4427: Add formal task points.

Let me know if you hit issues -- it should just be better in every way than maniphest.query was, but I might have missed some data or something like that.

Feb 24 2016, 9:02 PM · Wikimedia, Workboards, Maniphest
epriestley added a comment to T4427: Add formal task points.

Broadly, *.search and *.edit endpoint are replacing older *.query and *.variousnameslikeupdate endpoints, see:

Feb 24 2016, 8:58 PM · Wikimedia, Workboards, Maniphest
epriestley added a comment to T4427: Add formal task points.

It's present in maniphest.search.

Feb 24 2016, 8:57 PM · Wikimedia, Workboards, Maniphest
epriestley added a member for Blessed Committers: Firehed.
Feb 24 2016, 8:56 PM
epriestley added a comment to T10420: Code coverage reports from `arc unit` are sometimes inaccurate.

Sure, happy to bring those both upstream.

Feb 24 2016, 8:42 PM · Arcanist, Bug Report
jonathanrseawright awarded T4103: Implement "Role Profiles" to provide search, homepage and application defaults a Love token.
Feb 24 2016, 8:17 PM · Prioritized, Restricted Project, Restricted Project, Applications, Dashboards
epriestley updated subscribers of T7829: PhabricatorBotFeedNotificationHandler is completely broken and unusable.
Feb 24 2016, 5:25 PM · Conpherence, Phabot
epriestley merged task T10404: Ignore list for PhabricatorBotObjectNameHandler into T7829: PhabricatorBotFeedNotificationHandler is completely broken and unusable.
Feb 24 2016, 5:25 PM · Phabot
epriestley added a comment to T10447: When the push phase of "Land Revision" fails, show the error in the UI.

Oh, this is probably actually $operation->setWorkingCopyVCSError(...);

Feb 24 2016, 5:23 PM · Drydock (v3)
epriestley updated subscribers of T10447: When the push phase of "Land Revision" fails, show the error in the UI.

To fix this:

Feb 24 2016, 5:22 PM · Drydock (v3)
epriestley added a comment to T10412: Change Milestone sort options.

See T8227 for discussion of why adding an option is a measure of last resort.

Feb 24 2016, 5:13 PM · Feature Request
epriestley added a comment to T10446: Unable to simply mark a Diffusion comment as Done.

Ah, I missed "In Diffusion". Let me see if I can reproduce there.

Feb 24 2016, 4:58 PM · Audit, Bug Report
epriestley added a comment to T10412: Change Milestone sort options.

I'm not strongly attached to this behavior and don't mind swapping it and seeing how much pushback we get, it just seemed like the much better behavior at scale, and seeing it use on the Trello meta-board suggested to me that it was reasonable and possibly familiar to users coming from Trello:

Feb 24 2016, 4:51 PM · Feature Request
epriestley added a comment to T10444: Ever make a mistake? Catch it before its too late.??.

@wuup, I undisabled you since I think you weren't part of the "click shiny buttons" activity this morning.

Feb 24 2016, 4:43 PM
epriestley updated subscribers of T10444: Ever make a mistake? Catch it before its too late.??.
Feb 24 2016, 4:41 PM
epriestley added a comment to T10444: Ever make a mistake? Catch it before its too late.??.

I think @wuup might have been an innocent bystander who was caught in the crossfire. Poor guy!

Feb 24 2016, 4:41 PM
epriestley added a comment to T10436: Enhance the API (project.query) to find parent- and/or subprojects.

Can you walk us through what broad, overall goal you're trying to accomplish more generally? See Contributing Feature Requests and Describing Root Problems for help describing root problems. As those documents repeatedly emphasize, it's really important for us to understand root problems.

Feb 24 2016, 4:36 PM · Restricted Project, Projects, Feature Request
epriestley added a comment to T2226: Include a setup test for gzip enabled on the server.

T4854 is also related.

Feb 24 2016, 4:32 PM · Setup
epriestley created T10447: When the push phase of "Land Revision" fails, show the error in the UI.
Feb 24 2016, 4:26 PM · Drydock (v3)
epriestley added a comment to T10446: Unable to simply mark a Diffusion comment as Done.

I'm having trouble reproducing this -- I attempted to follow your steps, but didn't experience the issue. Here's what I did:

Feb 24 2016, 4:22 PM · Audit, Bug Report
epriestley added a comment to T10412: Change Milestone sort options.

The logic is that if you have 50 milestones, you are probably much more interested in milestones 50, 49 and 48 (this week / next week / last week) than milestone 1 (a year ago).

Feb 24 2016, 4:14 PM · Feature Request
epriestley added a comment to T10215: Provide tools to combat and recover from abuse.

One specific thing we're starting to see more of is test users signing up and bulk-associating or bulk-merging large numbers of tasks. There doesn't seem to be any specific intent behind this, but I think these UIs are shiny and make it easy to click a lot of buttons and have a large effect. Here's one such user on this install, who has registered three accounts just because they like clicking buttons so much:

Feb 24 2016, 4:13 PM · Abuse
epriestley added a comment to T10428: Behavior definitions not registered in some project dialogs.

See also T4854 -- mod_pagespeed is known to be incompatible with Phabricator, and should not be used.

Feb 24 2016, 3:53 PM · Bug Report (Needs Information)
• Persona awarded T9175: XHPAST doesn't build automatically a Love token.
Feb 24 2016, 11:09 AM · XHPAST, Arcanist
epriestley added a comment to T10433: Repo in Diffusion has no padding after Branches box if footer is not configured.

I guess "off" is default, so "disabled" isn't quite accurate.

Feb 24 2016, 1:30 AM · Diffusion, Bug Report
epriestley renamed T10433: Repo in Diffusion has no padding after Branches box if footer is not configured from Repo in Diffusion has no padding after Branches box if footer is disabled to Repo in Diffusion has no padding after Branches box if footer is not configured.
Feb 24 2016, 1:30 AM · Diffusion, Bug Report
epriestley renamed T10433: Repo in Diffusion has no padding after Branches box if footer is not configured from Repo in Diffusion has no padding after Branches box to Repo in Diffusion has no padding after Branches box if footer is disabled.
Feb 24 2016, 1:02 AM · Diffusion, Bug Report
epriestley added a revision to T6741: Lock Almanac networks, and allow Almanac domain namespaces to be locked: Unknown Object (Differential Revision).
Feb 24 2016, 12:59 AM · Almanac (v2), Security, Phacility
epriestley added a revision to T10246: Deploy Drydock in the Phacility cluster: Unknown Object (Differential Revision).
Feb 24 2016, 12:59 AM · Harbormaster (v3), Drydock (v3), Almanac (v2), Phacility

Feb 23 2016

epriestley edited projects for T10428: Behavior definitions not registered in some project dialogs, added: Bug Report (Needs Information); removed Bug Report.
Feb 23 2016, 11:42 PM · Bug Report (Needs Information)
epriestley closed T10368: Add ability to select ref mask used to fetch refs for git repos as Wontfix.

From elsewhere, it sounds like there isn't driving interest in pursuing this.

Feb 23 2016, 11:33 PM · Restricted Project, Feature Request
epriestley added a comment to T10368: Add ability to select ref mask used to fetch refs for git repos.

The expectation with the git fetch in question is that it will run very frequently on active repositories, so we should have all but the last few minutes (or even seconds) of data in most cases, and almost all the refs should be identical to the remote refs. The fetch schedule is here, but for enabled repositories we'll never wait more than 6 hours between fetches:

Feb 23 2016, 11:10 PM · Restricted Project, Feature Request
epriestley added a revision to T10349: Projects v3 Errata: Unknown Object (Differential Revision).
Feb 23 2016, 10:42 PM · Restricted Project, Projects (v3)
epriestley added a comment to T10368: Add ability to select ref mask used to fetch refs for git repos.

I also can't immediately reproduce the root issue. Here's what I did:

Feb 23 2016, 10:22 PM · Restricted Project, Feature Request
epriestley added a comment to T10432: Attempting to access attached data on AlmanacBinding.

I think I got 'em all, but let me know if you catch any more.

Feb 23 2016, 9:58 PM · Almanac, Bug Report
joshuaspence added a comment to T10432: Attempting to access attached data on AlmanacBinding.

Thanks :)

Feb 23 2016, 9:51 PM · Almanac, Bug Report
epriestley closed T10432: Attempting to access attached data on AlmanacBinding as Resolved by committing Restricted Diffusion Commit.
Feb 23 2016, 9:50 PM · Almanac, Bug Report
epriestley added a revision to T10432: Attempting to access attached data on AlmanacBinding: Restricted Differential Revision.
Feb 23 2016, 9:49 PM · Almanac, Bug Report
joshuaspence added a comment to T10432: Attempting to access attached data on AlmanacBinding.

This seems to be happening consistently for all new commits.

Feb 23 2016, 9:19 PM · Almanac, Bug Report
joshuaspence created T10432: Attempting to access attached data on AlmanacBinding.
Feb 23 2016, 9:18 PM · Almanac, Bug Report
epriestley added a comment to T10366: General support for multiple URIs for a repository.

Broadly, here are the general issues we've currently seen with Diffusion URIs:

Feb 23 2016, 8:09 PM · Diffusion (v3), Restricted Project, Feature Request
epriestley moved T8831: make `arc which` use mirrors from Backlog to URI Stuff on the Diffusion (v3) board.
Feb 23 2016, 7:50 PM · Restricted Project, Diffusion (v3), Arcanist
epriestley moved T8831: make `arc which` use mirrors from Backlog to v3 on the Diffusion board.
Feb 23 2016, 7:48 PM · Restricted Project, Diffusion (v3), Arcanist
epriestley added a comment to Blog Post: Updates to Workboards: Dragging, Cover Photos and Points.

Someone on Facebook noted that I omitted the Cyclone. Eek!

Feb 23 2016, 7:36 PM · Workboards, Maniphest
epriestley added a comment to T10368: Add ability to select ref mask used to fetch refs for git repos.

You spoke to the "Track Only" stuff a little bit in the description -- for complete clarity, we have two separate options that are only sometimes available:

Feb 23 2016, 7:36 PM · Restricted Project, Feature Request
epriestley added a comment to T10374: Permit creating macros out of existing files.

Oh, sorry, yeah -- for macros, you'd currently have to pick the file off your desktop with the file picker thing. I'm thinking more about a general "select some existing file" feature for other cases like cover images, where you can drag-and-drop.

Feb 23 2016, 7:31 PM · Restricted Project, Macros, Feature Request
epriestley added a comment to T10368: Add ability to select ref mask used to fetch refs for git repos.

Before pursuing this, I'd like to develop a better understanding of the root problem. Particularly:

Feb 23 2016, 7:27 PM · Restricted Project, Feature Request
epriestley closed T10413: Archived project tags are hidden everywhere, not just on Workboard cards as Resolved by committing Restricted Diffusion Commit.
Feb 23 2016, 6:47 PM · Projects (v3), Bug Report
epriestley added a revision to T10413: Archived project tags are hidden everywhere, not just on Workboard cards: Unknown Object (Differential Revision).
Feb 23 2016, 6:45 PM · Projects (v3), Bug Report
epriestley closed T10414: No natural sort on space dropdown as Resolved by committing Restricted Diffusion Commit.
Feb 23 2016, 6:41 PM · Bug Report
epriestley added a revision to T10414: No natural sort on space dropdown: Unknown Object (Differential Revision).
Feb 23 2016, 6:31 PM · Bug Report
epriestley renamed T10423: Huge diffs (>8MB) should stream directly into file storage in Diffusion when downloaded from Can not Download large diffs in diffusion "No configured storage engine can store this file" to Huge diffs (>8MB) should stream directly into file storage in Diffusion when downloaded.
Feb 23 2016, 6:26 PM · Phacility, Files, Diffusion, Bug Report
epriestley added a comment to T10428: Behavior definitions not registered in some project dialogs.

I'm having difficulty reproducing this. Here's what I did:

Feb 23 2016, 6:18 PM · Bug Report (Needs Information)
epriestley closed T10426: assignee is added to subscribers after task edit as Resolved by committing Restricted Diffusion Commit.
Feb 23 2016, 6:10 PM · Maniphest, Bug Report
epriestley added a comment to T10427: Errors when adding person to subproject.

What are the view, edit and join policies of the parent project set to?

Feb 23 2016, 6:09 PM · Projects
epriestley added a comment to T10374: Permit creating macros out of existing files.

Ah, interesting. I have F1 bound to "Expose: All Windows" so the flow for me is "Click > F1 > Drop Anywhere" to save, then "F1 > Drag > F1 > Drop" to load. I'm pretty sure I can get through that faster than I could copy-paste/type-select a file by Fxxx or from a browser UI.

Feb 23 2016, 6:06 PM · Restricted Project, Macros, Feature Request
epriestley added a comment to T10427: Errors when adding person to subproject.

Note that being an administrator doesn't matter: administrators have no special privileges, and can not bypass policies. See:

Feb 23 2016, 6:03 PM · Projects
epriestley added a revision to T10426: assignee is added to subscribers after task edit : Unknown Object (Differential Revision).
Feb 23 2016, 6:01 PM · Maniphest, Bug Report
epriestley added a comment to T10426: assignee is added to subscribers after task edit .

Thanks, your reproduction instructions are clear and complete. I was able to reproduce this by following your instructions and narrow it down to a root cause. This should be fixed shortly.

Feb 23 2016, 5:57 PM · Maniphest, Bug Report
epriestley added a comment to T10427: Errors when adding person to subproject.

I'm having trouble reproducing this. I tried to follow the instructions you provided to recreate the issue, but didn't seem to experience it. Here's what I did.

Feb 23 2016, 5:53 PM · Projects
epriestley added a comment to T10427: Errors when adding person to subproject.

Hi! Can you please let us know what version of Phabricator you're running?

Feb 23 2016, 5:48 PM · Projects
epriestley closed T10411: Modernize Almanac to support EditEngine, a subtask of T10246: Deploy Drydock in the Phacility cluster, as Resolved.
Feb 23 2016, 4:21 PM · Harbormaster (v3), Drydock (v3), Almanac (v2), Phacility
epriestley closed T10411: Modernize Almanac to support EditEngine as Resolved by committing Restricted Diffusion Commit.
Feb 23 2016, 4:21 PM · Almanac (v2)
epriestley created P1947 WarningCustomField.php.
Feb 23 2016, 4:15 PM
epriestley added a comment to T10424: Background colors (vs gradients) do not work on Workboards if "Dark" theme is set.

In conclusion, we should never add options to the product.

Feb 23 2016, 3:59 PM · Design, Workboards
epriestley renamed T10424: Background colors (vs gradients) do not work on Workboards if "Dark" theme is set from Background colors (vs gradients) do not work on Workboards to Background colors (vs gradients) do not work on Workboards if "Dark" theme is set.
Feb 23 2016, 3:59 PM · Design, Workboards
epriestley added a comment to T10424: Background colors (vs gradients) do not work on Workboards if "Dark" theme is set.

(The difference between the dark and default background colors is so subtle on my monitor that I never noticed it.)

Feb 23 2016, 3:11 PM · Design, Workboards
epriestley added a comment to T10424: Background colors (vs gradients) do not work on Workboards if "Dark" theme is set.

Oh, I do have that set locally -- I thought phui-theme-dark was just the default (since we got rid of light?) but this fixes itself if I switch to, e.g., the red header.

Feb 23 2016, 3:09 PM · Design, Workboards
epriestley added a comment to T10349: Projects v3 Errata.

When creating the first subproject there is a note that all members of the project will be moved to the subproject. In fact, members were copied to the subproject

Feb 23 2016, 1:25 PM · Restricted Project, Projects (v3)
epriestley added a comment to T10374: Permit creating macros out of existing files.

T10349 has a similar use case, where a user would like to select an existing file for use as a cover image.

Feb 23 2016, 1:17 PM · Restricted Project, Macros, Feature Request
epriestley renamed T10424: Background colors (vs gradients) do not work on Workboards if "Dark" theme is set from Colors (vs gradients) do not work on Workboards to Background colors (vs gradients) do not work on Workboards.
Feb 23 2016, 1:03 PM · Design, Workboards
epriestley created T10424: Background colors (vs gradients) do not work on Workboards if "Dark" theme is set.
Feb 23 2016, 1:03 PM · Design, Workboards
epriestley removed a reviewer for D15309: TEstPhab: Blessed Reviewers.
Feb 23 2016, 12:38 PM
epriestley added a project to T10423: Huge diffs (>8MB) should stream directly into file storage in Diffusion when downloaded: Diffusion.
Feb 23 2016, 12:37 PM · Phacility, Files, Diffusion, Bug Report
epriestley triaged T10423: Huge diffs (>8MB) should stream directly into file storage in Diffusion when downloaded as Wishlist priority.

Thanks, that's helpful.

Feb 23 2016, 12:36 PM · Phacility, Files, Diffusion, Bug Report
epriestley closed T10419: Sample herald rule results as Resolved.

This gets a brief mention in the documentation, but generally:

Feb 23 2016, 12:25 PM · Herald, Feature Request
epriestley added a comment to T10423: Huge diffs (>8MB) should stream directly into file storage in Diffusion when downloaded.

Hi! Can you tell me what version of Phabricator you're running?

Feb 23 2016, 12:16 PM · Phacility, Files, Diffusion, Bug Report