Page MenuHomePhabricator
Feed Advanced Search

Jan 29 2018

epriestley closed T5391: Support custom fields in 'Export to Excel' as Resolved.

Resolved by D18959 + D18960. See also T13049.

Jan 29 2018, 11:08 PM · Data Exporters
epriestley added a comment to T13051: Transaction edge storage is inefficient / an instance has 130GB of commit cross-refs.

Even on our fairly normal data, the effect was a little bit more dramatic than I'd expected:

Jan 29 2018, 11:04 PM · Transactions, Ops, Phacility
epriestley closed T5166: Expand the BulkJob tool as Resolved.

D18962 uses this to implement "Export Data" for large result sets.

Jan 29 2018, 10:59 PM · Restricted Project, Phacility, Daemons
epriestley closed T5166: Expand the BulkJob tool, a subtask of T1816: Support downloading a compressed working copy (tar/zip) of a repository at a specific commit or tag, as Resolved.
Jan 29 2018, 10:59 PM · Drydock, Diffusion
epriestley closed T5166: Expand the BulkJob tool, a subtask of T7307: Daemons UI is confusing when you aren't an administrator, as Resolved.
Jan 29 2018, 10:59 PM · Daemons, Phacility
epriestley closed T5166: Expand the BulkJob tool, a subtask of T6004: When moving a wiki page with subpages prompt to also move subpages, as Resolved.
Jan 29 2018, 10:59 PM · Phriction
epriestley added a comment to T13051: Transaction edge storage is inefficient / an instance has 130GB of commit cross-refs.

I'm going to optimize + probe secure001 now and see if any of the tables above shrunk. I'm expecting a very modest effect combined with zero user visible changes in the UI despite throwing away a bunch of data.

Jan 29 2018, 8:42 PM · Transactions, Ops, Phacility
epriestley added a comment to T13051: Transaction edge storage is inefficient / an instance has 130GB of commit cross-refs.

Took like 3-ish minutes and did this:

Jan 29 2018, 8:40 PM · Transactions, Ops, Phacility
epriestley added a comment to T13051: Transaction edge storage is inefficient / an instance has 130GB of commit cross-refs.

I double checked that our backups are working.

Jan 29 2018, 8:35 PM · Transactions, Ops, Phacility
epriestley removed a project from T13051: Transaction edge storage is inefficient / an instance has 130GB of commit cross-refs: 🐳.
Jan 29 2018, 7:41 PM · Transactions, Ops, Phacility
epriestley added a project to T13051: Transaction edge storage is inefficient / an instance has 130GB of commit cross-refs: 🐳.

Editing some edges on the new code as a sanity check before I compact things.

Jan 29 2018, 7:41 PM · Transactions, Ops, Phacility
epriestley added a comment to T13051: Transaction edge storage is inefficient / an instance has 130GB of commit cross-refs.

(Pushing this to secure, stuff might be funky for a minute while I gently massage the database.)

Jan 29 2018, 7:36 PM · Transactions, Ops, Phacility
epriestley added a revision to T13049: Plans: Data Exporters (Excel, JSON, CSV, etc.): Unknown Object (Differential Revision).
Jan 29 2018, 7:31 PM · Plans, Data Exporters
epriestley added a revision to T13049: Plans: Data Exporters (Excel, JSON, CSV, etc.): Restricted Differential Revision.
Jan 29 2018, 4:59 PM · Plans, Data Exporters
epriestley added a revision to T13049: Plans: Data Exporters (Excel, JSON, CSV, etc.): Unknown Object (Differential Revision).
Jan 29 2018, 4:52 PM · Plans, Data Exporters
epriestley added a revision to T13049: Plans: Data Exporters (Excel, JSON, CSV, etc.): Unknown Object (Differential Revision).
Jan 29 2018, 4:25 PM · Plans, Data Exporters
epriestley added a revision to T13049: Plans: Data Exporters (Excel, JSON, CSV, etc.): Unknown Object (Differential Revision).
Jan 29 2018, 3:57 PM · Plans, Data Exporters
epriestley added a comment to T13049: Plans: Data Exporters (Excel, JSON, CSV, etc.).

It looks like PHPExcel was formally deprecated on December 24, 2017 but I'm not planning to switch to its successor, PhpSpreadsheet, until we run into some specific motivator.

Jan 29 2018, 3:46 PM · Plans, Data Exporters
epriestley added a revision to T13049: Plans: Data Exporters (Excel, JSON, CSV, etc.): Unknown Object (Differential Revision).
Jan 29 2018, 3:35 PM · Plans, Data Exporters
epriestley added a revision to T13049: Plans: Data Exporters (Excel, JSON, CSV, etc.): Unknown Object (Differential Revision).
Jan 29 2018, 3:26 PM · Plans, Data Exporters
epriestley added a revision to T13049: Plans: Data Exporters (Excel, JSON, CSV, etc.): Unknown Object (Differential Revision).
Jan 29 2018, 3:22 PM · Plans, Data Exporters
epriestley added a revision to T13049: Plans: Data Exporters (Excel, JSON, CSV, etc.): Unknown Object (Differential Revision).
Jan 29 2018, 5:28 AM · Plans, Data Exporters
epriestley added a revision to T13049: Plans: Data Exporters (Excel, JSON, CSV, etc.): Unknown Object (Differential Revision).
Jan 29 2018, 3:04 AM · Plans, Data Exporters
epriestley added a revision to T13049: Plans: Data Exporters (Excel, JSON, CSV, etc.): Unknown Object (Differential Revision).
Jan 29 2018, 2:35 AM · Plans, Data Exporters
epriestley added a revision to T13049: Plans: Data Exporters (Excel, JSON, CSV, etc.): Unknown Object (Differential Revision).
Jan 29 2018, 2:28 AM · Plans, Data Exporters
epriestley added a revision to T13048: Implement a "take action only when this rule didn't match last time" repetition policy for Herald: Unknown Object (Differential Revision).
Jan 29 2018, 1:54 AM · Herald
epriestley added a comment to T13049: Plans: Data Exporters (Excel, JSON, CSV, etc.).

See PHI324.

Jan 29 2018, 1:31 AM · Plans, Data Exporters

Jan 28 2018

epriestley added a comment to T13049: Plans: Data Exporters (Excel, JSON, CSV, etc.).

See PHI323.

Jan 28 2018, 3:53 PM · Plans, Data Exporters
epriestley closed T12484: Write some tests for Phortune as Resolved.

This hasn't had any stability issues for a while.

Jan 28 2018, 12:20 AM · Phortune
epriestley closed T12495: Provide a conduit method to retrieve inline comments associated with a diff as Resolved.

transaction.search now returns comments and inlines, including path, line, length, and reply information.

Jan 28 2018, 12:19 AM · Feature Request
epriestley closed T12974: Upgrading: "Ferret" Fulltext Engine as Resolved.

I think everything we know about has been resolved.

Jan 28 2018, 12:03 AM · Installing & Upgrading, Search

Jan 27 2018

epriestley moved T4190: Rebuild remarkup image proxying from Backlog to Intermediate on the Contributor Onboarding board.
Jan 27 2018, 11:49 PM · Contributor Onboarding, Remarkup
epriestley moved T11138: mail is not delivered if metamta.placeholder-to-recipient is null from v3 to Stamps/Failover on the Mail board.
Jan 27 2018, 9:58 PM · Mail, Bug Report
epriestley updated the task description for T13053: Plans: Mail Tags and Failover.
Jan 27 2018, 9:58 PM · Plans, Must Encrypt Mail, Mail
epriestley moved T9141: Rename "MetaMTA" to "Mail" from v3 to Far Future on the Mail board.
Jan 27 2018, 9:57 PM · Restricted Project, Mail
epriestley moved T10448: Modularize mail tags from v3 to Stamps/Failover on the Mail board.
Jan 27 2018, 9:57 PM · Prioritized, Restricted Project, Mail, User Preferences, Owners, Feature Request
epriestley moved T7477: Handle inbound email with phabricator address in the CC from Backlog to Stamps/Failover on the Mail board.
Jan 27 2018, 9:57 PM · llvm, Restricted Project, Restricted Project, Mail
epriestley moved T4776: Notify users when someone takes an action that takes them off an object's recipient list from Backlog to Stamps/Failover on the Mail board.
Jan 27 2018, 9:56 PM · Mail, Restricted Project, Transactions
epriestley moved T11363: Differential "revision updated" mail preference is pretty useless from Backlog to Stamps/Failover on the Mail board.
Jan 27 2018, 9:56 PM · Customer Impact, Restricted Project, Differential, Mail
epriestley moved T11767: Unify email patch byte/line/time limit behaviors across Differential/Diffusion from Backlog to Future on the Mail board.
Jan 27 2018, 9:56 PM · Diffusion (v3), Differential, Mail
epriestley closed T12033: Large diffs can still repeatedly fail to insert as Resolved.

This is resolved as far as I'm aware. See also T11767 (improvements to these limits), PHI125 (similar issue with non-UTF8 attachments failing to insert) and T13053 (planning).

Jan 27 2018, 9:55 PM · Diffusion (v3), Restricted Project, Mail, Differential, Bug Report
epriestley closed T12033: Large diffs can still repeatedly fail to insert, a subtask of T11767: Unify email patch byte/line/time limit behaviors across Differential/Diffusion, as Resolved.
Jan 27 2018, 9:55 PM · Diffusion (v3), Differential, Mail
epriestley moved T12046: PHPMailer RCE [CVE-2016-10033 and CVE-2016-10045] from Backlog to Future on the Mail board.
Jan 27 2018, 9:54 PM · Mail, Security
epriestley moved T12404: Implement a first-party SMTP client from Backlog to Future on the Mail board.
Jan 27 2018, 9:54 PM · Infrastructure, Mail
epriestley moved T12491: Error reply emails which are generated before identifying the sender should no longer be sent, now that the "always require verification" rule is in place from Backlog to Stamps/Failover on the Mail board.
Jan 27 2018, 9:54 PM · Mail, Bug Report
epriestley moved T12630: Phabricator fails with a fatal PHP error if it receives a mail with no plain text part from Backlog to Stamps/Failover on the Mail board.
Jan 27 2018, 9:54 PM · Mail, Bug Report
epriestley moved T12677: Support multiple mail delivery services for automatic failover from Backlog to Stamps/Failover on the Mail board.
Jan 27 2018, 9:54 PM · Phacility, Mail
epriestley moved T12689: Mail is still received after resigning from a revision from Backlog to Stamps/Failover on the Mail board.
Jan 27 2018, 9:54 PM · Mail, Differential, Bug Report
epriestley moved T13037: An attacker gained staff access to Mailgun and was able to read customer API keys from Backlog to Stamps/Failover on the Mail board.
Jan 27 2018, 9:54 PM · Phacility, Security, Mail
epriestley moved T13053: Plans: Mail Tags and Failover from Backlog to Stamps/Failover on the Mail board.
Jan 27 2018, 9:53 PM · Plans, Must Encrypt Mail, Mail
epriestley closed T11337: Mail v3 as Wontfix.

I'm rolling this forward into T13053.

Jan 27 2018, 9:53 PM · llvm, Mail
epriestley triaged T13053: Plans: Mail Tags and Failover as Normal priority.
Jan 27 2018, 9:52 PM · Plans, Must Encrypt Mail, Mail
epriestley added a project to T13052: Differentiate "Waiting" from "Restarting after an error" on the daemon console: Daemons.
Jan 27 2018, 9:10 PM · Daemons
epriestley added a comment to T13052: Differentiate "Waiting" from "Restarting after an error" on the daemon console.

If that doesn't work, you might try adding some logging to PhabricatorDaemonEventListener->handleLogEvent() to see if the sequence of events you're receiving matches my expectations or not. It's hard to log out of this context because normal tools (var_dump(), phlog(), throw new Exception(), etc) won't necessarily end up anywhere very helpful, but you can do this to create an ad-hoc logfile:

Jan 27 2018, 8:54 PM · Daemons
epriestley added a comment to T13052: Differentiate "Waiting" from "Restarting after an error" on the daemon console.

The daemon log (at /daemon/) shows status icons for running (or recently-known) daemons. This stuff lives in PhabricatorDaemonLogViewController. The status PhabricatorDaemonLog::STATUS_WAIT represents two (slightly-more-than-two) different scenarios:

Jan 27 2018, 8:51 PM · Daemons
epriestley added a revision to T13051: Transaction edge storage is inefficient / an instance has 130GB of commit cross-refs: Unknown Object (Differential Revision).
Jan 27 2018, 3:33 PM · Transactions, Ops, Phacility
epriestley added a comment to T13051: Transaction edge storage is inefficient / an instance has 130GB of commit cross-refs.

My plan is to pick those to stable, then compact-edges here on secure, then compact-edges on the affected 130GB instance. There's some value in doing this sooner rather than later because the backups for 130GB of edge data are having some issues. The instance is a free test instance so this isn't a huge concern, but I'd sleep better if it was running smoothly. If you don't run compact-edges I think the worst those changes could really do is cause some kind of temporary display bug with new transactions, so the risk should be pretty small.

Jan 27 2018, 2:52 PM · Transactions, Ops, Phacility
epriestley added a revision to T13051: Transaction edge storage is inefficient / an instance has 130GB of commit cross-refs: Unknown Object (Differential Revision).
Jan 27 2018, 2:19 PM · Transactions, Ops, Phacility
epriestley added a revision to T13051: Transaction edge storage is inefficient / an instance has 130GB of commit cross-refs: Unknown Object (Differential Revision).
Jan 27 2018, 1:56 PM · Transactions, Ops, Phacility
epriestley added a revision to T13051: Transaction edge storage is inefficient / an instance has 130GB of commit cross-refs: Unknown Object (Differential Revision).
Jan 27 2018, 1:30 PM · Transactions, Ops, Phacility
epriestley added a comment to T13051: Transaction edge storage is inefficient / an instance has 130GB of commit cross-refs.

Bad news: data still has one reader/writer in the Asana-to-Revision linking implementation. So we can't completely get rid of that yet.

Jan 27 2018, 1:25 PM · Transactions, Ops, Phacility
epriestley updated the task description for T13051: Transaction edge storage is inefficient / an instance has 130GB of commit cross-refs.
Jan 27 2018, 1:21 PM · Transactions, Ops, Phacility
epriestley updated the task description for T13051: Transaction edge storage is inefficient / an instance has 130GB of commit cross-refs.
Jan 27 2018, 1:03 PM · Transactions, Ops, Phacility
epriestley triaged T13051: Transaction edge storage is inefficient / an instance has 130GB of commit cross-refs as Normal priority.
Jan 27 2018, 12:59 PM · Transactions, Ops, Phacility

Jan 26 2018

epriestley closed T13048: Implement a "take action only when this rule didn't match last time" repetition policy for Herald as Resolved.
Jan 26 2018, 11:29 PM · Herald
epriestley edited the content of Changelog.
Jan 26 2018, 11:24 PM
epriestley edited the content of Changelog.
Jan 26 2018, 11:23 PM
epriestley created 2018 Week 4 (Late January).
Jan 26 2018, 11:23 PM
epriestley closed T13050: 2018 Week 4 pre-release deploy issues on secure.phabricator.com as Resolved.

this can probably be figured out by examining the 2.16.0 release notes

Jan 26 2018, 9:43 PM · Phacility, Ops
epriestley added a revision to T13050: 2018 Week 4 pre-release deploy issues on secure.phabricator.com: Unknown Object (Differential Revision).
Jan 26 2018, 9:43 PM · Phacility, Ops
epriestley added a revision to T13050: 2018 Week 4 pre-release deploy issues on secure.phabricator.com: Unknown Object (Differential Revision).
Jan 26 2018, 9:37 PM · Phacility, Ops
epriestley triaged T13050: 2018 Week 4 pre-release deploy issues on secure.phabricator.com as Normal priority.
Jan 26 2018, 9:33 PM · Phacility, Ops
epriestley closed T11112: Herald allows setting multiple values in fields that should only accept one value as Resolved by committing Restricted Diffusion Commit.
Jan 26 2018, 9:23 PM · Herald, Bug Report
epriestley renamed T13046: Surface repository pull logs in the web UI from Improve repository pull logs to Surface repository pull logs in the web UI.
Jan 26 2018, 8:44 PM · Diffusion
epriestley renamed T13025: Turn bulk editor into a generic piece of EditEngine infrastructure with Custom Field support from Update bulk editor to Turn bulk editor into a generic piece of EditEngine infrastructure with Custom Field support.
Jan 26 2018, 6:18 PM · Transactions, ApplicationEditor
epriestley closed T13025: Turn bulk editor into a generic piece of EditEngine infrastructure with Custom Field support as Resolved.

T4675 survives this, because rendering policy controls on the client is very complicated.

Jan 26 2018, 6:17 PM · Transactions, ApplicationEditor
epriestley closed T13032: Upgrading: Git LFS as Resolved.
Jan 26 2018, 5:57 PM · Diffusion, Git, Installing & Upgrading
epriestley closed T13046: Surface repository pull logs in the web UI as Resolved.

T13049 is a followup about generalizing exporters.

Jan 26 2018, 5:57 PM · Diffusion
epriestley renamed T13047: Plans: Symbol Indexes from Figure out the state of affairs on symbol indexing to Plans: Symbol Indexes.
Jan 26 2018, 5:55 PM · Plans, Symbols
epriestley moved T13048: Implement a "take action only when this rule didn't match last time" repetition policy for Herald from Backlog to Next on the Herald board.
Jan 26 2018, 5:53 PM · Herald
epriestley moved T11112: Herald allows setting multiple values in fields that should only accept one value from Backlog to Next on the Herald board.
Jan 26 2018, 5:53 PM · Herald, Bug Report
epriestley moved T8644: Herald template / recipe book / examples / quick-create tool from Future to Guidance on the Herald board.
Jan 26 2018, 5:53 PM · Restricted Project, Herald
epriestley moved T11566: Provide more complete guidance on using regular expressions in Herald from Backlog to Guidance on the Herald board.
Jan 26 2018, 5:53 PM · Herald, Documentation
epriestley moved T13039: Support numeric fields in Herald from Backlog to Far Future on the Herald board.
Jan 26 2018, 5:52 PM · Herald
epriestley moved T4971: Add support for "Added filenames" and "Removed filenames" in Herald from Backlog to Future on the Herald board.
Jan 26 2018, 5:52 PM · Restricted Project, Herald
epriestley moved T8644: Herald template / recipe book / examples / quick-create tool from Backlog to Future on the Herald board.
Jan 26 2018, 5:52 PM · Restricted Project, Herald
epriestley moved T7804: Implement an "Acting User" Herald condition from Next to Far Future on the Herald board.
Jan 26 2018, 5:52 PM · Restricted Project, Herald, Mail
epriestley moved T6207: Add a "Can Use Herald Rules" application permission from Next to Far Future on the Herald board.
Jan 26 2018, 5:52 PM · Restricted Project, Policy, Herald
epriestley moved T9149: Flagging commits via Herald no longer generates flag note from Backlog to Future on the Herald board.
Jan 26 2018, 5:51 PM · Restricted Project, Flags, Herald
epriestley moved T13041: In Herald Commit rules, "Reviewer" and "On Autoclose Branch" have been deprecated from Backlog to Far Future on the Herald board.
Jan 26 2018, 5:50 PM · Diffusion, Herald
epriestley closed T11349: Herald has the wrong list of "Author's Projects" as Invalid.

This is quite old and I have no idea how to reproduce it and thus no idea how to fix it. Locally, I tried a few things and wasn't able to identify a case where the Herald field value differed from my actual project list.

Jan 26 2018, 5:49 PM · Restricted Project, Herald, Bug Report
epriestley added a revision to T11112: Herald allows setting multiple values in fields that should only accept one value: Unknown Object (Differential Revision).
Jan 26 2018, 5:42 PM · Herald, Bug Report
epriestley added a revision to T13048: Implement a "take action only when this rule didn't match last time" repetition policy for Herald: Unknown Object (Differential Revision).
Jan 26 2018, 5:42 PM · Herald
epriestley renamed T7961: In Herald, don't offer Fields and Actions from uninstalled applications to users writing rules from Herald should not show actions for uninstalled applications to In Herald, don't offer Fields and Actions from uninstalled applications to users writing rules.
Jan 26 2018, 5:33 PM · Applications, Herald
epriestley renamed T8952: Hide "Herald added subscribers..." stories from feed completely from herald showing as cause of feed item to Hide "Herald added subscribers..." stories from feed completely.
Jan 26 2018, 5:32 PM · Feed, Herald
epriestley closed T5954: Modularize "Export to Excel" as a feature of ApplicationSearch, a subtask of T5391: Support custom fields in 'Export to Excel', as Resolved.
Jan 26 2018, 4:59 PM · Data Exporters
epriestley closed T5954: Modularize "Export to Excel" as a feature of ApplicationSearch as Resolved.

I'm rolling this forward into T13049. See also D18919.

Jan 26 2018, 4:59 PM · Custom Fields, ApplicationSearch
epriestley closed T5954: Modularize "Export to Excel" as a feature of ApplicationSearch, a subtask of T3461: Improve documentation around Maniphest Excel exports, as Resolved.
Jan 26 2018, 4:59 PM · Maniphest
epriestley closed T5954: Modularize "Export to Excel" as a feature of ApplicationSearch, a subtask of T5151: Allow to export query results from Phrequent, as Resolved.
Jan 26 2018, 4:59 PM · Phrequent
epriestley edited projects for T5391: Support custom fields in 'Export to Excel', added: Data Exporters; removed Maniphest.
Jan 26 2018, 4:58 PM · Data Exporters