Page MenuHomePhabricator
Feed All Stories

May 27 2016

chad updated subscribers of T11050: Allow resigning from a review you are not directly responsible for.

Basically any time you could use Projects + Owners + Herald to be more tactical and less shotgun-like in assigning reviews is likely much more beneficial to correct coverage and lessening workload.

May 27 2016, 7:09 PM · Prioritized, Restricted Project, Differential, Feature Request
a300600st added a comment to T11050: Allow resigning from a review you are not directly responsible for.

Neat thanks! That does look like it would help with our problem but I still think that expanding a group would be a useful feature.

May 27 2016, 7:08 PM · Prioritized, Restricted Project, Differential, Feature Request
chad added a comment to T11050: Allow resigning from a review you are not directly responsible for.

I want to say Pinterest had a similar workflow and built a custom herald rule to manage it, not sure if that sounds interesting to you: https://engineering.pinterest.com/blog/shipping-faster-new-pr-processing-system

May 27 2016, 7:03 PM · Prioritized, Restricted Project, Differential, Feature Request
a300600st added a comment to T11050: Allow resigning from a review you are not directly responsible for.

One reason it pops up is that we put all develops as reviewers and then when one person actually decides to claim it for review they remove everyone else. We put all developers as subscribers, though, so that everyone stays notified when things change. I often want to remove myself as a subscriber so that I don't get emails about revisions I don't care about.

May 27 2016, 7:01 PM · Prioritized, Restricted Project, Differential, Feature Request
chad added a comment to T11050: Allow resigning from a review you are not directly responsible for.

This feels bad in general that you would need such a feature. For example, if I'm not qualified to review something, I just don't and it has no effect to my daily flow. Can you go into more detail about why your getting so many review requests you shouldn't be getting and how your using Dashboards perhaps to keep track of stuff that you actually should be reviewing?

May 27 2016, 6:54 PM · Prioritized, Restricted Project, Differential, Feature Request
joseneas added a comment to Q407: Connect to Phabricator Conduit API with pure javascript (QML Javascript).

The API documentation is not clear for implement a conduit client using JavaScript.

May 27 2016, 6:42 PM
avivey added a comment to Q407: Connect to Phabricator Conduit API with pure javascript (QML Javascript).

The conduit app has examples on calling conduit using cURL - you should be able to translate that to javascript.

May 27 2016, 6:33 PM
joseneas updated Q407: Connect to Phabricator Conduit API with pure javascript (QML Javascript) from to Connect to Phabricator Conduit API with pure javascript (QML Javascript).
May 27 2016, 6:30 PM
a300600st created T11050: Allow resigning from a review you are not directly responsible for.
May 27 2016, 6:06 PM · Prioritized, Restricted Project, Differential, Feature Request
chad closed Q406: How do I limit the permissions to create a new project only to administrators? as resolved.
May 27 2016, 4:18 PM · Projects, Config
epriestley added a comment to T11049: Differential home page and "Differential Revisions" Panel not showing same results.

Yeah the way that flow works is real dumb / undiscoverable / confusing / hard to use right now. It works great if you know the secret magic, but the secret magic is pretty secret.

May 27 2016, 4:17 PM · Bug Report
featherless added a comment to T11049: Differential home page and "Differential Revisions" Panel not showing same results.

Ah you just connected the dots for me about how to create custom queries for Dashboards.

May 27 2016, 4:12 PM · Bug Report
epriestley added Q406: How do I limit the permissions to create a new project only to administrators? (Answer 384).
May 27 2016, 3:49 PM
epriestley added a comment to T11049: Differential home page and "Differential Revisions" Panel not showing same results.

There's some discussion of the behavior of limits in T9372#175512 / D15923.

May 27 2016, 3:47 PM · Bug Report
featherless closed T11049: Differential home page and "Differential Revisions" Panel not showing same results as Resolved.

Ah hah, I had a limit of 5 set. I'd assumed that meant "5 per bucket". I see now that it means "5 results across all buckets".

May 27 2016, 3:38 PM · Bug Report
remusvrm updated Q406: How do I limit the permissions to create a new project only to administrators? from to How do I limit the permissions to create a new project only to administrators?.
May 27 2016, 3:34 PM · Projects, Config
epriestley added a comment to T11049: Differential home page and "Differential Revisions" Panel not showing same results.

What query is the query panel actually executing? Does it look exactly like this?

May 27 2016, 3:29 PM · Bug Report
featherless created T11049: Differential home page and "Differential Revisions" Panel not showing same results.
May 27 2016, 3:08 PM · Bug Report
epriestley added a comment to T4292: Implement repository replication.

Strategy I'm pursuing (for Git) is:

May 27 2016, 2:02 PM · Restricted Project, Diffusion
epriestley closed T11048: Testing as Spite.
May 27 2016, 1:48 PM · Feature Request
• murilowadt-test created T11048: Testing.
May 27 2016, 1:23 PM · Feature Request
epriestley added a comment to Q402: Why do my daemons keep <dying> ?.

You can email me directly at epriestley@phacility.com, or create a private thread in Conpherence on this install via the icon in the header.

May 27 2016, 1:08 PM · Daemons
alawson added a comment to Q402: Why do my daemons keep <dying> ?.

I worked on this problem for a couple hours last night, tailing log files
and checking various security features. This is our development server, the
lead Dev and I have complete control over it. I also had this problem on
another machine which is setup very similarly (CloudLinux, but locked done
a ton as its our hosting server), but on that machine I did not put the
Phabricator installation in a cPanel account. Had the same problem there,
so I don't think it's the security / CageFS getting in the way. Something
is timing the script out or killing it, and it's not the Kernel.

May 27 2016, 1:07 PM · Daemons
epriestley triaged T11047: Entering wrong or empty paths renders configtool unusable as Wishlist priority.

This needs to be the default behavior. For example, if you update lib-important-security/ and the new version has a bug which causes it to fatal, Phabricator must also fatal. It is not acceptable to just skip the extension and start without loading a library that may have important security/policy code an install could rely on, because this could make the install vulnerable to security or policy threats which you expect to be protected from.

May 27 2016, 12:29 PM · Config, Bug Report
hach-que added a comment to T11047: Entering wrong or empty paths renders configtool unusable.

Extensions are a really advanced feature, and you will get almost no support from upstream here if you have them installed. Or another way, anyone who is installing extensions should know Phabricator well enough to know how to fix this issue manually (hand editing local.json).

May 27 2016, 11:56 AM · Config, Bug Report
witrin added a comment to T10349: Projects v3 Errata.
In T10349#160861, @chad wrote:

Maybe a config option to make your own regex for display?

May 27 2016, 10:09 AM · Restricted Project, Projects (v3)
Sam2304 awarded T10374: Permit creating macros out of existing files a Like token.
May 27 2016, 9:21 AM · Restricted Project, Macros, Feature Request
• grindhold updated the task description for T11047: Entering wrong or empty paths renders configtool unusable.
May 27 2016, 9:01 AM · Config, Bug Report
• grindhold created T11047: Entering wrong or empty paths renders configtool unusable.
May 27 2016, 8:58 AM · Config, Bug Report
• mdchristoffer closed Q396: How are you suppose to work with Phabricator when you have revisions based on revisions? as resolved.
May 27 2016, 8:50 AM · Phabricator
• mdchristoffer added Q396: How are you suppose to work with Phabricator when you have revisions based on revisions? (Answer 383).
May 27 2016, 8:50 AM
aristedes added a comment to T11038: Herald rules linked to projects don't work as expected.

@hach-que Perhaps the terminology isn't really consistent then. When I add a project to an edit, it appears under a heading called "tags". Perhaps the Herald rule should say "tagged with"

May 27 2016, 5:29 AM · Owners, Herald, Bug Report
• snowmanlei added a comment to T10779: Cannot manage base project milestones if base project has subprojects.

项目权限管理问题:在建立项目后,创建人可以编辑项目描述,可以设置“修改描述”的权限,这次遇到问题就是这个配置是Editable By Project Members修改成 Editable By
和飞信_手机 问题解决。

May 27 2016, 3:35 AM · Restricted Project, Projects, Bug Report
epriestley closed T9643: XHPast Parse error when extracting i18n, a subtask of T5267: Localize Phabricator, as Resolved.
May 27 2016, 3:11 AM · Prioritized, Localization
epriestley closed T9643: XHPast Parse error when extracting i18n as Resolved by committing Restricted Diffusion Commit.
May 27 2016, 3:11 AM · XHPAST
epriestley added a revision to T5267: Localize Phabricator: Unknown Object (Differential Revision).
May 27 2016, 3:01 AM · Prioritized, Localization
epriestley added a revision to T9643: XHPast Parse error when extracting i18n: Unknown Object (Differential Revision).
May 27 2016, 3:01 AM · XHPAST
• snowmanlei added a comment to T10779: Cannot manage base project milestones if base project has subprojects.

Confirmed workaround: set base project edit permissions to something other than Project Members.

May 27 2016, 2:56 AM · Restricted Project, Projects, Bug Report
epriestley added a subtask for T5267: Localize Phabricator: T9643: XHPast Parse error when extracting i18n.
May 27 2016, 2:48 AM · Prioritized, Localization
epriestley added a parent task for T9643: XHPast Parse error when extracting i18n: T5267: Localize Phabricator.
May 27 2016, 2:48 AM · XHPAST
epriestley merged T8700: Add missing Translations into T5267: Localize Phabricator.
May 27 2016, 2:46 AM · Prioritized, Localization
epriestley merged task T8700: Add missing Translations into T5267: Localize Phabricator.
May 27 2016, 2:46 AM · Localization
epriestley added a comment to T8700: Add missing Translations.

I'm going to merge this into T5267, which discusses a larger-scale effort to translate Phabricator in the relatively near term. We'll be interested in feedback like this once that's a little further along.

May 27 2016, 2:45 AM · Localization
chad added a comment to T11040: Linking Workboard lanes to Task activities.

I think we've shot down other requests in this vein (Herald + Workboards) due to overwhelming interest in T5474, which would likely be prohibitive or insanely complicated to support if Herald was in the mix.

May 27 2016, 2:42 AM · Workboards, Feature Request
chad created T11046: Create Countdown button on new installs links to wrong place.
May 27 2016, 2:38 AM · Countdown, Bug Report
epriestley merged T1139: Internationalize Phabricator into T5267: Localize Phabricator.
May 27 2016, 2:37 AM · Prioritized, Localization
epriestley merged task T1139: Internationalize Phabricator into T5267: Localize Phabricator.
May 27 2016, 2:37 AM · Localization
epriestley added a comment to T1139: Internationalize Phabricator.

I'm going to merge this into T5267, which is more actionable in the short term.

May 27 2016, 2:37 AM · Localization
epriestley moved T8153: Improve detection and recovery when resources are mangled outside of Drydock's control from Backlog to Preflight on the Prioritized board.
May 27 2016, 2:17 AM · Prioritized, Drydock
epriestley moved T11044: Support partitioning application databases across multiple database hosts from Backlog to Preflight on the Prioritized board.
May 27 2016, 2:17 AM · Clusters, Prioritized
hach-que added a comment to T11042: Cluster documentation leads you slightly awry about "read only mode".

I was kind of expecting that 'cluster.read-only' would be a database configuration value, not a local configuration value. Otherwise you need to update all nodes in the cluster instead of just setting the flag in the DB. My expectation when the cluster DB is read-only is that only the following operations work:

May 27 2016, 12:29 AM · Clusters, Bug Report
hach-que added a comment to T11038: Herald rules linked to projects don't work as expected.

@aristedes It means projects and subscribers of the commit. For example, you can add those projects here:

May 27 2016, 12:22 AM · Owners, Herald, Bug Report

May 26 2016

chad added a comment to T11038: Herald rules linked to projects don't work as expected.

Subscribers -> Are these watchers or members of the project? I don't recognise the word "subscribers" in Phabricator.

May 26 2016, 11:07 PM · Owners, Herald, Bug Report
aristedes added a comment to T11038: Herald rules linked to projects don't work as expected.

It confused at least one person, but I am relatively new to this product.

May 26 2016, 10:39 PM · Owners, Herald, Bug Report
mavit added a comment to Q404: Find a unique identifier for a Diffusion repository from a Git repository?.

The trouble is that that could sometimes give the wrong answer, because querying the Git URI is not the only way that arc itself finds the repository; for example, the repository callsign can be hardcoded into .arcconfig.

May 26 2016, 9:40 PM · Conduit, Diffusion
epriestley added a project to T11045: Inching an install toward repository clustering: Guides.
May 26 2016, 8:24 PM · Guides, Clusters
epriestley created T11045: Inching an install toward repository clustering.
May 26 2016, 8:24 PM · Guides, Clusters
avivey added a comment to Q404: Find a unique identifier for a Diffusion repository from a Git repository?.

I think the only way right now is to get the remote uri, call repository.query with it, and parse that.
It might make sense to just teach arc which to output it's things in json.

May 26 2016, 8:15 PM · Conduit, Diffusion
epriestley created T11044: Support partitioning application databases across multiple database hosts.
May 26 2016, 8:06 PM · Clusters, Prioritized
epriestley added a project to T8153: Improve detection and recovery when resources are mangled outside of Drydock's control: Prioritized.
May 26 2016, 7:45 PM · Prioritized, Drydock
epriestley added a comment to T11042: Cluster documentation leads you slightly awry about "read only mode".

I think I didn't actually do these things, I did a different and more horrible set of things and then wrote these great instructions to avoid all the msitakes I made. I'll attempt to let dump pierce through read-only, or find some other solution if that isn't straightforward.

May 26 2016, 7:45 PM · Clusters, Bug Report
mavit updated Q404: Find a unique identifier for a Diffusion repository from a Git repository? from to Find a unique identifier for a Diffusion repository from a Git repository?.
May 26 2016, 7:42 PM · Conduit, Diffusion
epriestley claimed T11042: Cluster documentation leads you slightly awry about "read only mode".
May 26 2016, 7:20 PM · Clusters, Bug Report
epriestley added a comment to T11043: Database status page throws an exception if `mysql.host` not explicitly set in a clustered environment.

Should be fixed in HEAD of master. Thanks for the report!

May 26 2016, 7:17 PM · Database, Clusters, Bug Report
epriestley closed T11043: Database status page throws an exception if `mysql.host` not explicitly set in a clustered environment as Resolved by committing Restricted Diffusion Commit.
May 26 2016, 7:14 PM · Database, Clusters, Bug Report
epriestley added a revision to T11043: Database status page throws an exception if `mysql.host` not explicitly set in a clustered environment: Unknown Object (Differential Revision).
May 26 2016, 7:13 PM · Database, Clusters, Bug Report
epriestley added projects to T11043: Database status page throws an exception if `mysql.host` not explicitly set in a clustered environment: Clusters, Database.
May 26 2016, 7:03 PM · Database, Clusters, Bug Report
epriestley claimed T11043: Database status page throws an exception if `mysql.host` not explicitly set in a clustered environment.
May 26 2016, 7:03 PM · Database, Clusters, Bug Report
epriestley added a comment to T11043: Database status page throws an exception if `mysql.host` not explicitly set in a clustered environment.

Ah! I got a repro.

May 26 2016, 7:03 PM · Database, Clusters, Bug Report
Taffer added a comment to T11040: Linking Workboard lanes to Task activities.

Sorry, the Root Problem would be something like:

May 26 2016, 6:48 PM · Workboards, Feature Request
yelirekim added a comment to T11043: Database status page throws an exception if `mysql.host` not explicitly set in a clustered environment.
  • Entirely unset mysql.host
  • Configure a master and a replica in cluster.databases
  • Go to /config/database/
May 26 2016, 6:39 PM · Database, Clusters, Bug Report
epriestley added a comment to T11043: Database status page throws an exception if `mysql.host` not explicitly set in a clustered environment.

I can't immediately reproduce this. Here's things running fine with mysql.host set to a bogus value:

May 26 2016, 6:30 PM · Database, Clusters, Bug Report
yelirekim created T11043: Database status page throws an exception if `mysql.host` not explicitly set in a clustered environment.
May 26 2016, 6:05 PM · Database, Clusters, Bug Report
yelirekim created T11042: Cluster documentation leads you slightly awry about "read only mode".
May 26 2016, 6:02 PM · Clusters, Bug Report
epriestley reopened T8510: Typeahead project proposals in Maniphest advanced search do not always include exact matches as "Open".

(I marked D15981 as fixing this but I'm going to leave it open for confirmation since I'm not completely sure there weren't more cases that I missed.)

May 26 2016, 5:47 PM · Typeahead, Prioritized, Wikimedia, Infrastructure, Search
epriestley closed T8510: Typeahead project proposals in Maniphest advanced search do not always include exact matches as Resolved by committing Restricted Diffusion Commit.
May 26 2016, 5:47 PM · Typeahead, Prioritized, Wikimedia, Infrastructure, Search
epriestley closed T11041: Not possible to move a URI without destroying the old one as Invalid.

After you created the second URI, you can switch the first one to active IO back, so there the error messages does not work:

May 26 2016, 5:31 PM · Diffusion, Bug Report
epriestley moved T8510: Typeahead project proposals in Maniphest advanced search do not always include exact matches from Backlog to Paused on the Prioritized board.

T6102 and the downstream also contain discussion of a second case which does center around exact matches -- searching for "Wikidata" -- but I can't reproduce it at HEAD:

May 26 2016, 5:29 PM · Typeahead, Prioritized, Wikimedia, Infrastructure, Search
Luke081515.2 added a comment to T11041: Not possible to move a URI without destroying the old one.

Strange. I tried it again, now I can't reproduce it too, but the history said, that I didn't make a fault.

May 26 2016, 5:26 PM · Diffusion, Bug Report
epriestley added a comment to T11041: Not possible to move a URI without destroying the old one.

I can't reproduce this. Here's the first repository (I/O off, disabled);

May 26 2016, 5:21 PM · Diffusion, Bug Report
Luke081515.2 created T11041: Not possible to move a URI without destroying the old one.
May 26 2016, 5:19 PM · Diffusion, Bug Report
epriestley added a comment to T8510: Typeahead project proposals in Maniphest advanced search do not always include exact matches.

D15981 doesn't, strictly speaking, prioritize exact matches. However, it should fix the underlying issue.

May 26 2016, 5:11 PM · Typeahead, Prioritized, Wikimedia, Infrastructure, Search
epriestley added a revision to T8510: Typeahead project proposals in Maniphest advanced search do not always include exact matches: Unknown Object (Differential Revision).
May 26 2016, 5:06 PM · Typeahead, Prioritized, Wikimedia, Infrastructure, Search
rfreebern added a comment to T11040: Linking Workboard lanes to Task activities.

This seems related to T5474: Support workboard column triggers which activate when a task is dropped into a column.

May 26 2016, 4:23 PM · Workboards, Feature Request
chad added a comment to T11040: Linking Workboard lanes to Task activities.

I think anything automatic here would likely only be truly correct maybe 50% of the time, and I think that's really bad from an accuracy perspective if that's what's important to your company. I'd rather look at what information is actually missing from the workboard cards (ie, diff attached) and explore options there.

May 26 2016, 2:58 PM · Workboards, Feature Request
epriestley added a comment to T11040: Linking Workboard lanes to Task activities.

I don't understand the root problem this is seeking to solve. Please read Describing Root Problems carefully and try to follow the instructions there to describe a problem we can solve.

May 26 2016, 2:54 PM · Workboards, Feature Request
epriestley added a revision to T5267: Localize Phabricator: Unknown Object (Differential Revision).
May 26 2016, 2:49 PM · Prioritized, Localization
Taffer created T11040: Linking Workboard lanes to Task activities.
May 26 2016, 2:44 PM · Workboards, Feature Request
epriestley added a revision to T5267: Localize Phabricator: Unknown Object (Differential Revision).
May 26 2016, 2:37 PM · Prioritized, Localization
epriestley added a revision to T4103: Implement "Role Profiles" to provide search, homepage and application defaults: Unknown Object (Differential Revision).
May 26 2016, 1:54 PM · Prioritized, Restricted Project, Restricted Project, Dashboards, Applications
epriestley added a revision to T5267: Localize Phabricator: Unknown Object (Differential Revision).
May 26 2016, 1:54 PM · Prioritized, Localization
Sam2304 awarded T4207: Implement (tree) subtasks, as distinct from (graph) task dependencies a Like token.
May 26 2016, 1:47 PM · Maniphest
d.maznekov awarded T10922: Upgrading: New Repository UI, APIs, and URI management a Like token.
May 26 2016, 1:25 PM · Diffusion (v3), Installing & Upgrading
d.maznekov added a comment to T10922: Upgrading: New Repository UI, APIs, and URI management.

Thank you very much !

May 26 2016, 1:25 PM · Diffusion (v3), Installing & Upgrading
epriestley added a comment to T10922: Upgrading: New Repository UI, APIs, and URI management.

To edit project tags for a repository, navigate to the repository main page, then click Manage Repository → Edit Basic Information and add project tags to the "Tags" field.

May 26 2016, 1:14 PM · Diffusion (v3), Installing & Upgrading
epriestley added a comment to T11038: Herald rules linked to projects don't work as expected.

I don't think the behavior of the fields is unclear and we haven't seen other users run into this particular difficulty.

May 26 2016, 1:06 PM · Owners, Herald, Bug Report
epriestley added Q396: How are you suppose to work with Phabricator when you have revisions based on revisions? (Answer 382).
May 26 2016, 12:58 PM
• felix.rently awarded T3569: Make Phrequent somewhat usable a Like token.
May 26 2016, 12:54 PM · Phrequent
epriestley closed T11039: Trello to Phabricator Migration issue with trello_create.py and trello_makePHIDs.py as Invalid.

Unfortunately, we can't help you with this problem. We didn't write this script, do not maintain it, and have never seen it before. You should try getting in touch with the script maintainers instead. We don't have the resources to provide support for third-party code.

May 26 2016, 12:39 PM · Bug Report
epriestley added a comment to T4334: Support PHP5.4+ syntax in XHPAST.

That seems fine to me.

May 26 2016, 12:35 PM · XHPAST