Page MenuHomePhabricator
Feed Advanced Search

Jun 2 2016

epriestley changed the blog for post Blog Post: Stripe CTF3.
Jun 2 2016, 9:35 PM
epriestley updated the task description for T11076: Ship CORGI.
Jun 2 2016, 9:32 PM · Prioritized, Ops, Phacility
epriestley merged T8706: Serve blog.phacility.com from the admin host tier into T11076: Ship CORGI.
Jun 2 2016, 9:31 PM · Prioritized, Ops, Phacility
epriestley merged task T8706: Serve blog.phacility.com from the admin host tier into T11076: Ship CORGI.
Jun 2 2016, 9:31 PM · Ops, Phacility
epriestley added a comment to T8706: Serve blog.phacility.com from the admin host tier.

I'm just going to merge this into T11076 since this will happen with the rest of the deployment work there. I expect this to be straightforward.

Jun 2 2016, 9:31 PM · Ops, Phacility
epriestley updated the task description for T11076: Ship CORGI.
Jun 2 2016, 9:20 PM · Prioritized, Ops, Phacility
epriestley added a revision to T11076: Ship CORGI: Unknown Object (Differential Revision).
Jun 2 2016, 9:19 PM · Prioritized, Ops, Phacility
epriestley added a revision to T11076: Ship CORGI: Unknown Object (Differential Revision).
Jun 2 2016, 9:15 PM · Prioritized, Ops, Phacility
epriestley updated the task description for T11076: Ship CORGI.
Jun 2 2016, 9:08 PM · Prioritized, Ops, Phacility
epriestley closed T11087: Make permissions more granular in repositorys as Invalid.

From the downstream, it seems like the problem may be one specific user wanting to add mirrors to existing repositories:

Jun 2 2016, 8:46 PM · Wikimedia, Diffusion, Feature Request
epriestley closed T11087: Make permissions more granular in repositorys, a subtask of T9287: Working with many repositories, as Invalid.
Jun 2 2016, 8:46 PM · Restricted Project, Restricted Project, Diffusion
epriestley added a comment to T11087: Make permissions more granular in repositorys.

It's also vaguely possible that we'll add "suggest an edit" (ala T4104) in a generic way, then untrusted users could suggest edits and trusted users could approve them. I don't currently plan to build this as a general-purpose workflow, though.

Jun 2 2016, 5:53 PM · Wikimedia, Diffusion, Feature Request
epriestley added a comment to T11087: Make permissions more granular in repositorys.

I don't plan to pursue field-level permissions in any application. They were somewhat recently removed from Maniphest in T10003. That task discusses some of the complications that field-level permissions create in the general case.

Jun 2 2016, 5:42 PM · Wikimedia, Diffusion, Feature Request
epriestley edited P1987 Masterwork From Distant Lands.
Jun 2 2016, 5:23 PM
epriestley closed T11086: test as Spite.
Jun 2 2016, 5:11 PM · Support Request
epriestley added a comment to T11085: Prevent users from generating an ambiguous commit message by editing revisions with the web UI.

(See also T6297.)

Jun 2 2016, 5:10 PM · Prioritized, Restricted Project, Arcanist, Bug Report
epriestley added a comment to T11085: Prevent users from generating an ambiguous commit message by editing revisions with the web UI.

You're watching #twitter.

Jun 2 2016, 5:08 PM · Prioritized, Restricted Project, Arcanist, Bug Report
epriestley created Support Request.
Jun 2 2016, 5:04 PM
epriestley added a comment to T11083: ability to set visibility / editability of Herald rules.

Object rules have the same edit policy as the corresponding object.

Jun 2 2016, 4:52 PM · Feature Request
epriestley added a comment to T11084: LDAP Unable to Connect.

Thanks for the report, and for including version information.

Jun 2 2016, 4:50 PM · Bug Report
epriestley updated the task description for T9897: Unbeta Feedback on Phame.
Jun 2 2016, 2:46 PM · Phame
epriestley added a comment to T11083: ability to set visibility / editability of Herald rules.

Go to /applications/, then click this:

Jun 2 2016, 2:40 PM · Feature Request
epriestley added a revision to T9897: Unbeta Feedback on Phame: Unknown Object (Differential Revision).
Jun 2 2016, 2:39 PM · Phame
epriestley added a revision to T11076: Ship CORGI: Unknown Object (Differential Revision).
Jun 2 2016, 2:39 PM · Prioritized, Ops, Phacility
epriestley added a comment to T11083: ability to set visibility / editability of Herald rules.

Er, rather, the policy hint is fine since that's the view policy.

Jun 2 2016, 2:12 PM · Feature Request
epriestley added a comment to T11083: ability to set visibility / editability of Herald rules.

Global rules can only be edited by users with "Can Manage Global Rules" permission (Applications → Herald → Configure → Can Manage Global Rules). By default, this is administrators.

Jun 2 2016, 2:10 PM · Feature Request
epriestley edited this Badge.
Jun 2 2016, 2:04 PM
epriestley added a comment to T11082: Ordering of clone URIs incorrectly sorts less-preferred ID-based URIs above more-preferred callsign-based URIs.

Thanks for the report! This should be resolved at HEAD of master, and promote to stable in about 48 hours.

Jun 2 2016, 2:03 PM · Diffusion, Bug Report
epriestley closed T11082: Ordering of clone URIs incorrectly sorts less-preferred ID-based URIs above more-preferred callsign-based URIs as Resolved by committing Restricted Diffusion Commit.
Jun 2 2016, 1:57 PM · Diffusion, Bug Report
epriestley claimed T11082: Ordering of clone URIs incorrectly sorts less-preferred ID-based URIs above more-preferred callsign-based URIs.
Jun 2 2016, 1:55 PM · Diffusion, Bug Report
epriestley renamed T11082: Ordering of clone URIs incorrectly sorts less-preferred ID-based URIs above more-preferred callsign-based URIs from Harbourmaster api non-backward compatible to Ordering of clone URIs incorrectly sorts less-preferred ID-based URIs above more-preferred callsign-based URIs.
Jun 2 2016, 1:55 PM · Diffusion, Bug Report
epriestley added a comment to T11079: URI usernames and passwords containing forward slashes are not fully escaped.

Thanks for the report! This should be fixed in HEAD of master, and will promote to stable in about 48 hours. Let us know if you're still seeing issues after updating or if you run into anything else.

Jun 2 2016, 1:54 PM · libphutil, Bug Report
epriestley edited this Badge.
Jun 2 2016, 1:53 PM
epriestley closed T11079: URI usernames and passwords containing forward slashes are not fully escaped as Resolved by committing Restricted Diffusion Commit.
Jun 2 2016, 1:53 PM · libphutil, Bug Report
epriestley renamed T11079: URI usernames and passwords containing forward slashes are not fully escaped from Couldn't resolve host to URI usernames and passwords containing forward slashes are not fully escaped.
Jun 2 2016, 1:52 PM · libphutil, Bug Report
epriestley added a revision to T11079: URI usernames and passwords containing forward slashes are not fully escaped: Unknown Object (Differential Revision).
Jun 2 2016, 1:51 PM · libphutil, Bug Report
epriestley added a revision to T11082: Ordering of clone URIs incorrectly sorts less-preferred ID-based URIs above more-preferred callsign-based URIs: Unknown Object (Differential Revision).
Jun 2 2016, 1:44 PM · Diffusion, Bug Report
epriestley added a comment to T11082: Ordering of clone URIs incorrectly sorts less-preferred ID-based URIs above more-preferred callsign-based URIs.

Thanks, those specifics are helpful.

Jun 2 2016, 1:35 PM · Diffusion, Bug Report
epriestley closed Q412: Limit number of accesses during repository import as resolved.
Jun 2 2016, 1:26 PM
epriestley added Q412: Limit number of accesses during repository import (Answer 388).
Jun 2 2016, 1:26 PM
epriestley added a comment to T11082: Ordering of clone URIs incorrectly sorts less-preferred ID-based URIs above more-preferred callsign-based URIs.

Thanks for the report!

Jun 2 2016, 1:23 PM · Diffusion, Bug Report
epriestley closed T11080: llll as Spite.
Jun 2 2016, 11:42 AM · Feature Request
epriestley closed T10903: Customize "open in new window" link behavior for RemarkupEngine as Resolved.

D16006 is the main fix; CORGI is resolved in secret at https://meta.phacility.com/D84.

Jun 2 2016, 2:08 AM · Phacility, Remarkup
epriestley closed T10902: Support third-party package definitions in Celerity, or remember how it works if I already built it as Resolved.

Fix for this is on meta, but the short version is that it works fine as-is.

Jun 2 2016, 2:00 AM · Phacility, Celerity
epriestley added a revision to T6299: Remarkup links to objects in Phame break if a CNAME is set: Unknown Object (Differential Revision).
Jun 2 2016, 1:53 AM · Haskell.org, Remarkup, Phame
epriestley added a revision to T6299: Remarkup links to objects in Phame break if a CNAME is set: Unknown Object (Differential Revision).
Jun 2 2016, 1:44 AM · Haskell.org, Remarkup, Phame
epriestley added a revision to T10903: Customize "open in new window" link behavior for RemarkupEngine: Unknown Object (Differential Revision).
Jun 2 2016, 1:44 AM · Phacility, Remarkup
epriestley moved T11076: Ship CORGI from Backlog to The Queue on the Prioritized board.
Jun 2 2016, 1:32 AM · Prioritized, Ops, Phacility
epriestley created T11076: Ship CORGI.
Jun 2 2016, 1:32 AM · Prioritized, Ops, Phacility

Jun 1 2016

epriestley added a revision to T4103: Implement "Role Profiles" to provide search, homepage and application defaults: Unknown Object (Differential Revision).
Jun 1 2016, 11:57 PM · Prioritized, Restricted Project, Restricted Project, Dashboards, Applications
epriestley added a comment to T10895: Support`arc browse --revision <commit>` and make `arc browse` with no arguments mean `... --revision HEAD`.

Yeah, --force is a little weird as a name given what it actually does.

Jun 1 2016, 11:06 PM · Arcanist, Prioritized, Restricted Project, Feature Request
epriestley added a comment to T10895: Support`arc browse --revision <commit>` and make `arc browse` with no arguments mean `... --revision HEAD`.

Maybe --force could even just do that, without needing a new argument, since it already forces argument interpretation as paths? So the behavior would be:

Jun 1 2016, 10:59 PM · Arcanist, Prioritized, Restricted Project, Feature Request
epriestley added a comment to T10895: Support`arc browse --revision <commit>` and make `arc browse` with no arguments mean `... --revision HEAD`.

Do you type that explicitly, or would something like find-some-files | xargs arc browse --paths ("force any arguments to be interpreted as paths") be good enough?

Jun 1 2016, 10:57 PM · Arcanist, Prioritized, Restricted Project, Feature Request
epriestley added a comment to T11074: Ability to get a line URL from revisions / commits.

It looks like the original user may have just wanted (or been satisfied with) a Diffusion file-line link anyway? Thread is sort of ambiguous:

Jun 1 2016, 9:53 PM · Differential, Feature Request
epriestley added a revision to T4103: Implement "Role Profiles" to provide search, homepage and application defaults: Unknown Object (Differential Revision).
Jun 1 2016, 9:06 PM · Prioritized, Restricted Project, Restricted Project, Dashboards, Applications
epriestley added a comment to T11074: Ability to get a line URL from revisions / commits.

We could change the URL as a side effect of starting an inline comment, although you'd have to cancel the inline comment afterward.

Jun 1 2016, 8:26 PM · Differential, Feature Request
epriestley added a comment to T11074: Ability to get a line URL from revisions / commits.

This is fairly involved in the general case:

Jun 1 2016, 8:19 PM · Differential, Feature Request
epriestley added a comment to T10078: Implement global readthrough cache pools.

D16001 likely provides an appropriate general-purpose mechanism for the user cache pool.

Jun 1 2016, 8:10 PM · Restricted Project, Infrastructure, Performance
epriestley added a revision to T4103: Implement "Role Profiles" to provide search, homepage and application defaults: Unknown Object (Differential Revision).
Jun 1 2016, 8:08 PM · Prioritized, Restricted Project, Restricted Project, Applications, Dashboards
epriestley added a comment to T4103: Implement "Role Profiles" to provide search, homepage and application defaults.

Mystery Query: Where is this coming from?

Jun 1 2016, 6:23 PM · Prioritized, Restricted Project, Restricted Project, Applications, Dashboards
epriestley added a comment to T4103: Implement "Role Profiles" to provide search, homepage and application defaults.

Summarizing caching for my own benefit. See also T10078, T10077.

Jun 1 2016, 6:14 PM · Prioritized, Restricted Project, Restricted Project, Applications, Dashboards
epriestley added a revision to T4103: Implement "Role Profiles" to provide search, homepage and application defaults: Unknown Object (Differential Revision).
Jun 1 2016, 5:26 PM · Prioritized, Restricted Project, Restricted Project, Applications, Dashboards
epriestley closed T11071: Wrong tooltip text for "Audit" application on homepage as Resolved by committing Restricted Diffusion Commit.
Jun 1 2016, 2:12 PM · Audit, Bug Report
epriestley removed a parent task for T11071: Wrong tooltip text for "Audit" application on homepage: Unknown Object (Maniphest Task).
Jun 1 2016, 12:25 PM · Audit, Bug Report
epriestley added a revision to T11071: Wrong tooltip text for "Audit" application on homepage: Unknown Object (Differential Revision).
Jun 1 2016, 11:56 AM · Audit, Bug Report
epriestley claimed T11071: Wrong tooltip text for "Audit" application on homepage.
Jun 1 2016, 11:56 AM · Audit, Bug Report
epriestley closed T11070: Cannot import users from LDAP as Resolved.

It sounds like this is resolved. We almost certainly can not raise a more specific exception.

Jun 1 2016, 11:50 AM · Auth, Bug Report
epriestley triaged T11069: Add support for Slack integration as Wishlist priority.

We may pursue this eventually but there are hundreds of hours of work between where we are today and scalable, no-code-required bot support.

Jun 1 2016, 11:49 AM · Doorkeeper, Feature Request
mrjoops awarded T5474: Support workboard column triggers which activate when a task is dropped into a column a Like token.
Jun 1 2016, 10:51 AM · Workboards (v3), Restricted Project
• AnysM awarded T5474: Support workboard column triggers which activate when a task is dropped into a column a Like token.
Jun 1 2016, 10:51 AM · Workboards (v3), Restricted Project
mrjoops awarded T1130: Add Mercurial tag/bookmark support to Diffusion a Like token.
Jun 1 2016, 10:51 AM · Mercurial, Diffusion

May 31 2016

epriestley added a revision to T4103: Implement "Role Profiles" to provide search, homepage and application defaults: Unknown Object (Differential Revision).
May 31 2016, 11:53 PM · Prioritized, Restricted Project, Restricted Project, Applications, Dashboards
epriestley added a revision to T4103: Implement "Role Profiles" to provide search, homepage and application defaults: Unknown Object (Differential Revision).
May 31 2016, 8:59 PM · Prioritized, Restricted Project, Restricted Project, Applications, Dashboards
epriestley added a revision to T4103: Implement "Role Profiles" to provide search, homepage and application defaults: Unknown Object (Differential Revision).
May 31 2016, 3:35 PM · Prioritized, Restricted Project, Restricted Project, Applications, Dashboards
epriestley closed T11061: TEST_3 as Invalid.
May 31 2016, 3:31 PM
carlos.prado.vsys awarded T4214: Allow installs to change or supplement the "Phabricator" icon a Like token.
May 31 2016, 10:08 AM · Design
galendai awarded T10034: Plan the future of Maniphest task relationships a Love token.
May 31 2016, 2:32 AM · Maniphest

May 30 2016

epriestley moved T10751: Make Phabricator Highly Available from The Queue to Paused on the Prioritized board.

Observed repositories should now version in a reasonable way.

May 30 2016, 6:23 PM · Restricted Project, Infrastructure, Prioritized
epriestley created T11057: Allow installs to manage which cluster service new repositories allocate on.
May 30 2016, 6:14 PM · Diffusion, Clusters
epriestley added a comment to T11055: Parent Project membership editing sometimes unclear.

Can you show us a screenshot of the top section of the "Members" page for the project? It should look something like this:

May 30 2016, 5:59 PM · Projects, Feature Request
epriestley created T11056: Send some database traffic to replicas even while the master is still alive.
May 30 2016, 5:53 PM · Clusters
epriestley added a comment to T11055: Parent Project membership editing sometimes unclear.

Bug reports must include reproduction steps. See Providing Reproduction Steps.

May 30 2016, 5:35 PM · Projects, Feature Request
epriestley closed T4292: Implement repository replication as Resolved.

That last part which I just landed hasn't been vetted in production for very long yet, but I think this all works now.

May 30 2016, 5:14 PM · Restricted Project, Diffusion
epriestley closed T4292: Implement repository replication, a subtask of T10751: Make Phabricator Highly Available, as Resolved.
May 30 2016, 5:14 PM · Restricted Project, Infrastructure, Prioritized
epriestley closed T4292: Implement repository replication, a subtask of T4209: Multiserver / High-Availability Configuration, as Resolved.
May 30 2016, 5:14 PM · Restricted Project, Phacility
epriestley closed T11054: Documentation for Inbound mail is outdated (Lamson Part) as Resolved by committing Restricted Diffusion Commit.
May 30 2016, 5:11 PM · Documentation, Bug Report
epriestley added a comment to T11054: Documentation for Inbound mail is outdated (Lamson Part).

D15990 removes the Lamson instructions from the documentation.

May 30 2016, 5:06 PM · Documentation, Bug Report
epriestley added a revision to T11054: Documentation for Inbound mail is outdated (Lamson Part): Unknown Object (Differential Revision).
May 30 2016, 5:05 PM · Documentation, Bug Report
epriestley added a comment to T11051: "Explain Why" isn't properly explaining why revisions were closed.

After D15989, this dialog should properly include information about tree hash and commit hash matches. It could still be more clear, but appears to at least be working as intended now.

May 30 2016, 4:59 PM · Restricted Project, Differential, Bug Report
epriestley added a comment to T11051: "Explain Why" isn't properly explaining why revisions were closed.

Basically, the metadata is sufficient to generate the "Local Commits" table and little else.

May 30 2016, 4:48 PM · Restricted Project, Differential, Bug Report
epriestley added a comment to T11051: "Explain Why" isn't properly explaining why revisions were closed.

The commits are not in Diffusion. Here's a real example of the metadata we store for a commit:

May 30 2016, 4:48 PM · Restricted Project, Differential, Bug Report
epriestley closed T11042: Cluster documentation leads you slightly awry about "read only mode" as Resolved by committing Restricted Diffusion Commit.
May 30 2016, 4:30 PM · Clusters, Bug Report
epriestley added a revision to T11051: "Explain Why" isn't properly explaining why revisions were closed: Unknown Object (Differential Revision).
May 30 2016, 4:29 PM · Restricted Project, Differential, Bug Report
epriestley added a comment to T11051: "Explain Why" isn't properly explaining why revisions were closed.

When you run arc diff, you provide (explicitly or implicitly) a commit range. Phabricator stores:

May 30 2016, 4:25 PM · Restricted Project, Differential, Bug Report
epriestley added a revision to T10769: Read-Only Mode Errata: Unknown Object (Differential Revision).
May 30 2016, 3:46 PM · Clusters
epriestley added a comment to T11042: Cluster documentation leads you slightly awry about "read only mode".

Prior to D15987, the easiest fix for this was to add --force:

May 30 2016, 3:37 PM · Clusters, Bug Report
epriestley added a revision to T11042: Cluster documentation leads you slightly awry about "read only mode": Unknown Object (Differential Revision).
May 30 2016, 3:34 PM · Clusters, Bug Report
epriestley added a revision to T4292: Implement repository replication: Unknown Object (Differential Revision).
May 30 2016, 2:49 PM · Restricted Project, Diffusion
epriestley moved T8909: Provide ability to hide all inline or done comments from Backlog to Nits on the FreeBSD board.
May 30 2016, 2:41 PM · Inline Comments, Prioritized, Restricted Project, Restricted Project, Differential, FreeBSD
epriestley added a comment to T11051: "Explain Why" isn't properly explaining why revisions were closed.

Here and in one other report, revisions which are closed by matching a commit or tree hash with a discovered commit are not explaining this in "Explain Why". "Explain Why" is expected to be able to provide details about this:

May 30 2016, 1:58 PM · Restricted Project, Differential, Bug Report