Page MenuHomePhabricator
Feed All Stories

Jun 18 2019

epriestley closed T13315: "bin/repository reparse" stops when encountering a "PhabricatorWorkerPermanentFailureException", but should continue as Resolved by committing Restricted Diffusion Commit.
Jun 18 2019, 6:33 PM · Diffusion
epriestley updated subscribers of T13156: Plans: Improve Phacility UI for managing instance managers and cards.

See PHI954. The "support pact members" cache is currently updated only when you actually click "Support", but this is often silly and unintuitive.

Jun 18 2019, 6:22 PM · Plans, Phacility
epriestley created T13318: Phacility developer setup documentation omits "local.*" hosts.
Jun 18 2019, 6:20 PM · Phacility
epriestley renamed T13317: Unverified/nonsense account emails can trigger inbound CCs (was: Inbound support pact email sometimes CCs disabled user `@maxhodges` in Phacility) from Inbound support pact email sometimes CCs disabled user `@maxhodges` in Phacility to Unverified/nonsense account emails can trigger inbound CCs (was: Inbound support pact email sometimes CCs disabled user `@maxhodges` in Phacility).
Jun 18 2019, 5:22 PM · Mail, Phacility
epriestley added a comment to T13317: Unverified/nonsense account emails can trigger inbound CCs (was: Inbound support pact email sometimes CCs disabled user `@maxhodges` in Phacility).

Aha! This user very cleverly added noreply@admin.phacility.com to their user account before they were disabled.

Jun 18 2019, 5:22 PM · Mail, Phacility
epriestley created T13317: Unverified/nonsense account emails can trigger inbound CCs (was: Inbound support pact email sometimes CCs disabled user `@maxhodges` in Phacility).
Jun 18 2019, 5:15 PM · Mail, Phacility
epriestley created T13316: On Workboards, "Move Tasks to Column..." has high friction in the common use case (selecting a column on the same board).
Jun 18 2019, 4:41 PM · Workboards
epriestley added a revision to T13315: "bin/repository reparse" stops when encountering a "PhabricatorWorkerPermanentFailureException", but should continue: Unknown Object (Differential Revision).
Jun 18 2019, 3:02 PM · Diffusion
epriestley created T13315: "bin/repository reparse" stops when encountering a "PhabricatorWorkerPermanentFailureException", but should continue.
Jun 18 2019, 2:57 PM · Diffusion

Jun 17 2019

epriestley added a comment to T13303: There is an extra or missing "array_reverse()" somewhere after recent transaction strength sorting changes.

I deployed that last round of things to secure. Not totally confident I got everything, but hopefully we're in better shape now.

Jun 17 2019, 9:25 PM · Transactions
epriestley closed T13311: Audit code does not fully respect committer identities as commit authors as Resolved by committing Restricted Diffusion Commit.
Jun 17 2019, 8:56 PM · Diffusion
epriestley closed T13310: Users with no access to Spaces can not log out as Resolved by committing Restricted Diffusion Commit.
Jun 17 2019, 8:44 PM · Auth, Spaces
epriestley closed T13313: "Download Raw Diff" may fail in a misleading way after failing to engage the chunk engine if the raw diff is larger than 4MB as Resolved by committing Restricted Diffusion Commit.
Jun 17 2019, 8:31 PM · Differential, Files
epriestley added a revision to T13303: There is an extra or missing "array_reverse()" somewhere after recent transaction strength sorting changes: Unknown Object (Differential Revision).
Jun 17 2019, 8:11 PM · Transactions
epriestley added a comment to T13296: PCRE segfaults after "fork()" on PHP 7.3+ on MacOS with "pcre.jit" enabled.

The change above seems to have worked around things, I'll leave this open until we can confirm that PHP 7.3.6 or some later version fixes the underlying issue.

Jun 17 2019, 6:02 PM · Infrastructure
epriestley added a revision to T13312: "Branch Status" in Diffusion has a bizarre pager at the top of the element: Unknown Object (Differential Revision).
Jun 17 2019, 5:52 PM · Diffusion
epriestley closed T13307: "Sign With MFA" requires "CAN_EDIT" permission, but should only require "CAN_INTERACT" permission as Resolved by committing Restricted Diffusion Commit.
Jun 17 2019, 5:41 PM · Auth
epriestley added a revision to T13303: There is an extra or missing "array_reverse()" somewhere after recent transaction strength sorting changes: Restricted Differential Revision.
Jun 17 2019, 5:40 PM · Transactions
epriestley added a revision to T13303: There is an extra or missing "array_reverse()" somewhere after recent transaction strength sorting changes: Unknown Object (Differential Revision).
Jun 17 2019, 5:36 PM · Transactions
epriestley added a comment to T13303: There is an extra or missing "array_reverse()" somewhere after recent transaction strength sorting changes.

uhoh looks like I'm a dum dum

Jun 17 2019, 5:33 PM · Transactions
epriestley added a comment to T13303: There is an extra or missing "array_reverse()" somewhere after recent transaction strength sorting changes.

This is doing somewhat better now, but I've still seen:

Jun 17 2019, 5:23 PM · Transactions
epriestley added a revision to T13311: Audit code does not fully respect committer identities as commit authors: Unknown Object (Differential Revision).
Jun 17 2019, 5:11 PM · Diffusion
epriestley added a revision to T13311: Audit code does not fully respect committer identities as commit authors: Unknown Object (Differential Revision).
Jun 17 2019, 5:04 PM · Diffusion
epriestley added a revision to T13313: "Download Raw Diff" may fail in a misleading way after failing to engage the chunk engine if the raw diff is larger than 4MB: Unknown Object (Differential Revision).
Jun 17 2019, 4:51 PM · Differential, Files
epriestley created T13314: Poke around `bin/repository update` and chase off any ghosts.
Jun 17 2019, 4:06 PM · Diffusion
epriestley created T13313: "Download Raw Diff" may fail in a misleading way after failing to engage the chunk engine if the raw diff is larger than 4MB.
Jun 17 2019, 2:40 PM · Differential, Files
epriestley added a comment to T13312: "Branch Status" in Diffusion has a bizarre pager at the top of the element.

If this table is retained, there's a mystery meat "branch" icon meaning "default branch" in the leftmost column. This should get a label/tip.

Jun 17 2019, 2:31 PM · Diffusion
epriestley created T13312: "Branch Status" in Diffusion has a bizarre pager at the top of the element.
Jun 17 2019, 2:30 PM · Diffusion

Jun 14 2019

epriestley created T13311: Audit code does not fully respect committer identities as commit authors.
Jun 14 2019, 4:02 PM · Diffusion

Jun 13 2019

epriestley added a revision to T13310: Users with no access to Spaces can not log out: Unknown Object (Differential Revision).
Jun 13 2019, 11:59 PM · Auth, Spaces
epriestley created T13310: Users with no access to Spaces can not log out.
Jun 13 2019, 11:49 PM · Auth, Spaces
epriestley added a revision to T13309: Git protocol proxy can not parse protocol frames if repository on disk is a shallow clone: Unknown Object (Differential Revision).
Jun 13 2019, 11:31 PM · Diffusion
epriestley added a comment to T13309: Git protocol proxy can not parse protocol frames if repository on disk is a shallow clone.

The shallow <hash> frames can not advertise capabilities.

Jun 13 2019, 11:28 PM · Diffusion
epriestley added a comment to T13309: Git protocol proxy can not parse protocol frames if repository on disk is a shallow clone.

Here's what goes over the wire:

Jun 13 2019, 10:46 PM · Diffusion
epriestley added a comment to T13309: Git protocol proxy can not parse protocol frames if repository on disk is a shallow clone.

To reproduce this, you can:

Jun 13 2019, 10:42 PM · Diffusion
epriestley created T13309: Git protocol proxy can not parse protocol frames if repository on disk is a shallow clone.
Jun 13 2019, 10:42 PM · Diffusion
epriestley added a revision to T13308: Phacility account import flow can misfire when both invites and account links are present: Unknown Object (Differential Revision).
Jun 13 2019, 10:15 PM · Phacility
epriestley added a comment to T13308: Phacility account import flow can misfire when both invites and account links are present.

To reproduce the first issue, do this:

Jun 13 2019, 8:43 PM · Phacility
epriestley created T13308: Phacility account import flow can misfire when both invites and account links are present.
Jun 13 2019, 8:23 PM · Phacility
epriestley added a revision to T13146: Phacility account import tool fails to recognize usernames with differing case, e.g. "Alice" vs "alice": Unknown Object (Differential Revision).
Jun 13 2019, 8:10 PM · Phacility

Jun 11 2019

epriestley updated the task description for T12664: Update diff/patch parsing to extract more metadata and parse a wider range of formats.
Jun 11 2019, 6:18 PM · Arcanist, Differential

Jun 6 2019

epriestley added a project to T13307: "Sign With MFA" requires "CAN_EDIT" permission, but should only require "CAN_INTERACT" permission: Auth.
Jun 6 2019, 2:23 AM · Auth
epriestley added a revision to T13307: "Sign With MFA" requires "CAN_EDIT" permission, but should only require "CAN_INTERACT" permission: Unknown Object (Differential Revision).
Jun 6 2019, 2:23 AM · Auth
epriestley created T13307: "Sign With MFA" requires "CAN_EDIT" permission, but should only require "CAN_INTERACT" permission.
Jun 6 2019, 2:18 AM · Auth
epriestley added a subtask for T10469: Implement "Tabs", an amazing feature where multiple different pages are loaded in the same browser window!: T13302: When a user clicks a link in a dialog: we close the dialog; but might prefer to transition it away with the navigation event under Quicksand so "Back" can reopen it.
Jun 6 2019, 1:58 AM · Quicksand, Feature Request, Workboards
epriestley added a parent task for T13302: When a user clicks a link in a dialog: we close the dialog; but might prefer to transition it away with the navigation event under Quicksand so "Back" can reopen it: T10469: Implement "Tabs", an amazing feature where multiple different pages are loaded in the same browser window!.
Jun 6 2019, 1:58 AM · Quicksand
epriestley renamed T13302: When a user clicks a link in a dialog: we close the dialog; but might prefer to transition it away with the navigation event under Quicksand so "Back" can reopen it from With Quicksand enabled, "Choose Subtype" dialog on "Create Subtask" workflow is not dismissed by making a selection to When a user clicks a link in a dialog: we close the dialog; but might prefer to transition it away with the navigation event under Quicksand so "Back" can reopen it.
Jun 6 2019, 1:57 AM · Quicksand
epriestley added a comment to T13302: When a user clicks a link in a dialog: we close the dialog; but might prefer to transition it away with the navigation event under Quicksand so "Back" can reopen it.

D20573 will fix this, but perhaps not in the most graceful way possible.

Jun 6 2019, 1:56 AM · Quicksand
epriestley added a revision to T13302: When a user clicks a link in a dialog: we close the dialog; but might prefer to transition it away with the navigation event under Quicksand so "Back" can reopen it: Unknown Object (Differential Revision).
Jun 6 2019, 1:50 AM · Quicksand

Jun 5 2019

epriestley added a revision to T13304: Phabricator "Backups" instructions are somewhat out of date: Unknown Object (Differential Revision).
Jun 5 2019, 10:00 PM · Database
epriestley added a revision to T13306: Provide a mechanism for importing local disk file backups into the Phacility cluster: Unknown Object (Differential Revision).
Jun 5 2019, 9:36 PM · Phacility, Files
epriestley added a revision to T13305: Migration "20180208.maniphest.02.populate.php" may fail when queueing indexing tasks: Unknown Object (Differential Revision).
Jun 5 2019, 6:43 PM · Installing & Upgrading, Database
epriestley claimed T13306: Provide a mechanism for importing local disk file backups into the Phacility cluster.
Jun 5 2019, 5:59 PM · Phacility, Files
epriestley created T13306: Provide a mechanism for importing local disk file backups into the Phacility cluster.
Jun 5 2019, 5:59 PM · Phacility, Files
epriestley added a comment to T13305: Migration "20180208.maniphest.02.populate.php" may fail when queueing indexing tasks.

I applied this change to 20180208.maniphest.02.populate.php:

Jun 5 2019, 5:24 PM · Installing & Upgrading, Database
epriestley created T13305: Migration "20180208.maniphest.02.populate.php" may fail when queueing indexing tasks.
Jun 5 2019, 5:18 PM · Installing & Upgrading, Database
yelirekim added a comment to T13289: Plans: 2019 Week 21-23 Bonus Content.

Meow Meow

Jun 5 2019, 4:19 PM · Plans
epriestley created T13304: Phabricator "Backups" instructions are somewhat out of date.
Jun 5 2019, 3:26 PM · Database

Jun 4 2019

epriestley merged T10307: Option to move child pages when parent in renamed into T6004: When moving a wiki page with subpages prompt to also move subpages.
Jun 4 2019, 8:01 PM · Phriction
epriestley merged task T10307: Option to move child pages when parent in renamed into T6004: When moving a wiki page with subpages prompt to also move subpages.
Jun 4 2019, 8:01 PM · Feature Request, Phriction
epriestley added a comment to T6004: When moving a wiki page with subpages prompt to also move subpages.

See also PHI1275. Now that Phriction supports relative references in [[ ... ]] syntax, moves should be able to modify those references.

Jun 4 2019, 7:46 PM · Phriction
epriestley added 1 mock(s) for T13289: Plans: 2019 Week 21-23 Bonus Content: M1479: Meow Meow.
Jun 4 2019, 7:38 PM · Plans
epriestley added 1 task(s) for M1479: Meow Meow: T13289: Plans: 2019 Week 21-23 Bonus Content.
Jun 4 2019, 7:38 PM
epriestley added a comment to T13289: Plans: 2019 Week 21-23 Bonus Content.

  • Jun 4 2019, 7:37 PM · Plans
    epriestley updated images of M1479: Meow Meow.
    Jun 4 2019, 7:36 PM
    epriestley added a revision to T13160: Support <colgroup> in the verbose remarkup <table> syntax for specifying column widths: Unknown Object (Differential Revision).
    Jun 4 2019, 4:32 PM · Remarkup
    epriestley closed T13300: When a "Draft" revision is closed into "Published" (not "Abandoned"), it should immediately promote as Resolved by committing Restricted Diffusion Commit.
    Jun 4 2019, 1:56 PM · Diffusion, Differential

    May 31 2019

    epriestley added a revision to T13298: Improve Herald profiling information available in transcripts: Unknown Object (Differential Revision).
    May 31 2019, 2:59 PM · Herald
    epriestley added a revision to T13298: Improve Herald profiling information available in transcripts: Unknown Object (Differential Revision).
    May 31 2019, 2:33 PM · Herald
    epriestley added a revision to T13300: When a "Draft" revision is closed into "Published" (not "Abandoned"), it should immediately promote: Unknown Object (Differential Revision).
    May 31 2019, 1:18 PM · Diffusion, Differential

    May 30 2019

    epriestley updated the task description for T13289: Plans: 2019 Week 21-23 Bonus Content.
    May 30 2019, 11:48 PM · Plans
    epriestley added a comment to T13297: "vi" and "vim" may exit with errors even though edits succeed.

    I'm just going to close this in the hope it fades quietly into the night now that it has been "fixed".

    May 30 2019, 11:47 PM · Arcanist
    epriestley added a comment to T13303: There is an extra or missing "array_reverse()" somewhere after recent transaction strength sorting changes.

    D20563 probably fixes mail. I don't think (?) it will fix notifications but haven't hunted that down yet.

    May 30 2019, 11:15 PM · Transactions
    epriestley added a revision to T13303: There is an extra or missing "array_reverse()" somewhere after recent transaction strength sorting changes: Unknown Object (Differential Revision).
    May 30 2019, 10:37 PM · Transactions
    epriestley updated the task description for T13302: When a user clicks a link in a dialog: we close the dialog; but might prefer to transition it away with the navigation event under Quicksand so "Back" can reopen it.
    May 30 2019, 7:10 PM · Quicksand
    epriestley renamed T13303: There is an extra or missing "array_reverse()" somewhere after recent transaction strength sorting changes from There is an extra or missing "array_reverse()" somewhere after recent transaction strength sorting changs to There is an extra or missing "array_reverse()" somewhere after recent transaction strength sorting changes.
    May 30 2019, 7:09 PM · Transactions
    epriestley created T13303: There is an extra or missing "array_reverse()" somewhere after recent transaction strength sorting changes.
    May 30 2019, 7:09 PM · Transactions
    epriestley added a comment to T13302: When a user clicks a link in a dialog: we close the dialog; but might prefer to transition it away with the navigation event under Quicksand so "Back" can reopen it.

    This likely affects other similar dialogs. The fix is to duct-tape something in Quicksand to something in the "click an area to click a link" behavior, most likely.

    May 30 2019, 7:07 PM · Quicksand
    epriestley created T13302: When a user clicks a link in a dialog: we close the dialog; but might prefer to transition it away with the navigation event under Quicksand so "Back" can reopen it.
    May 30 2019, 7:05 PM · Quicksand
    asherkin added a comment to T13296: PCRE segfaults after "fork()" on PHP 7.3+ on MacOS with "pcre.jit" enabled.

    The changelog for PHP 7.3.6 mentions "Fixed possible crashes, because of inconsistent PCRE cache and opcache SHM reset." which might be relevant, but no bug link so who knows.

    May 30 2019, 6:52 PM · Infrastructure
    epriestley created T13301: Browser "Find..." feature isn't very useful in Safari on Workboards.
    May 30 2019, 3:29 PM · Workboards

    May 28 2019

    epriestley edited the content of Changelog.
    May 28 2019, 5:29 PM
    epriestley created an object: 2019 Week 21 (Very Late May).
    May 28 2019, 5:27 PM
    epriestley triaged T13300: When a "Draft" revision is closed into "Published" (not "Abandoned"), it should immediately promote as Low priority.
    May 28 2019, 5:22 PM · Diffusion, Differential
    epriestley created T13300: When a "Draft" revision is closed into "Published" (not "Abandoned"), it should immediately promote.
    May 28 2019, 5:22 PM · Diffusion, Differential
    epriestley added a revision to T13289: Plans: 2019 Week 21-23 Bonus Content: Unknown Object (Differential Revision).
    May 28 2019, 5:14 PM · Plans

    May 27 2019

    epriestley moved T13166: Herald "block" and "send an email" actions are mutually exclusive, which isn't intuitive from Far Future to Next on the Herald board.
    May 27 2019, 3:36 PM · Diffusion, Herald
    epriestley added a comment to T13166: Herald "block" and "send an email" actions are mutually exclusive, which isn't intuitive.

    A sort of broad issue here is that Herald sometimes knows (or could know, or could guess, or maybe could speculate) that a rule won't do what you expect, but it doesn't tell you.

    May 27 2019, 3:36 PM · Diffusion, Herald
    epriestley triaged T13299: Herald only triggers for commits once, during the initial publish phase as Normal priority.
    May 27 2019, 3:34 PM · Diffusion, Herald
    epriestley closed T13283: When Herald acts, include the transaction group as part of the "state" it acts upon as Resolved.

    I think this is about as good as we're going to get, and we've only taken a very small step toward the precipice of a self-aware Herald that hates humankind.

    May 27 2019, 2:56 PM · Herald
    epriestley triaged T13298: Improve Herald profiling information available in transcripts as Normal priority.
    May 27 2019, 2:54 PM · Herald
    epriestley closed T12257: Add an "author email" field to the "Differential Diff" Herald rule as Wontfix.

    KDE appears to be moving to GitLab (see: https://gitlab.com/gitlab-org/gitlab-ce/issues/53206) and we haven't seen this request from other installs, so I'm just going to close this out.

    May 27 2019, 2:38 PM · KDE, Herald, Differential
    epriestley closed T8804: Replacing a Herald rule with a Doorkeeper extension as Wontfix.

    I think the modern answer here is "use Webhooks". They may not do everything you want if you're writing a chat bot (notably, they intentionally do not currently provide a human-readable text representation of transactions) but there generally suitable for publishing changes to Phabricator objects into a remote system and will produce a program with generally reasonable behaviors and no weird demons lurking under the surface.

    May 27 2019, 2:30 PM · Restricted Project, Herald, Doorkeeper

    May 24 2019

    epriestley added a comment to T13297: "vi" and "vim" may exit with errors even though edits succeed.

    Here's a more concise test:

    May 24 2019, 9:10 PM · Arcanist
    avivey added a comment to T13297: "vi" and "vim" may exit with errors even though edits succeed.

    Running the thing on my ubuntu exists with 0...

    $ vi --version
    VIM - Vi IMproved 8.0 (2016 Sep 12, compiled Apr 10 2018 21:31:58)
    Included patches: 1-1453
    .....
    .....
    Compilation: gcc -c -I. -Iproto -DHAVE_CONFIG_H   -Wdate-time  -g -O2 -fdebug-prefix-map=/build/vim-NQEcoP/vim-8.0.1453=. -fstack-protector-strong -Wformat -Werror=format-security -DTINY_VIMRC -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1
    Linking: gcc   -Wl,-Bsymbolic-functions -Wl,-z,relro -Wl,-z,now -Wl,--as-needed -o vim    -lSM -lICE -lXpm -lXt -lX11 -lXdmcp -lSM -lICE  -lm -ltinfo  -lselinux -lacl -lattr -ldl
    May 24 2019, 9:04 PM · Arcanist
    epriestley triaged T13297: "vi" and "vim" may exit with errors even though edits succeed as Low priority.
    May 24 2019, 8:15 PM · Arcanist
    trevorsummerssmith awarded T6004: When moving a wiki page with subpages prompt to also move subpages a Like token.
    May 24 2019, 1:54 PM · Phriction
    thuffir added a comment to T4752: Daemons may not have permission to remove files from local disk storage engine.

    setfacl -Rdm u:phab-phd:rw /var/local/phabricator/filestore
    setfacl -Rm u:phab-phd:rw /var/local/phabricator/filestore

    May 24 2019, 9:21 AM · Files

    May 23 2019

    epriestley lowered the priority of T13296: PCRE segfaults after "fork()" on PHP 7.3+ on MacOS with "pcre.jit" enabled from Normal to Low.
    May 23 2019, 11:54 PM · Infrastructure
    epriestley added a comment to T13296: PCRE segfaults after "fork()" on PHP 7.3+ on MacOS with "pcre.jit" enabled.

    I built my PHP from source in T13232 so this is presumably not a Homebrew issue.

    May 23 2019, 11:49 PM · Infrastructure