Page MenuHomePhabricator
Feed Advanced Search

Feb 14 2018

epriestley renamed T13076: Plans: Phacility cluster caching, renaming, and rebalance/compaction from Plans: Phacility cluster infrastructure improvements to Plans: Phacility cluster caching, renaming, and rebalance/compaction.
Feb 14 2018, 2:20 PM · Plans, Ops, Infrastructure, Phacility
epriestley triaged T13076: Plans: Phacility cluster caching, renaming, and rebalance/compaction as Normal priority.
Feb 14 2018, 2:10 PM · Plans, Ops, Infrastructure, Phacility
epriestley removed a project from T6960: Support %P in qsprintf(): Phacility.
Feb 14 2018, 1:09 PM · Security, Infrastructure

Feb 11 2018

epriestley closed T8475: Clean up remnants of old Differential hunk storage as Resolved by committing Restricted Diffusion Commit.
Feb 11 2018, 12:12 AM · Infrastructure, Restricted Project, Differential

Feb 10 2018

epriestley added a revision to T8475: Clean up remnants of old Differential hunk storage: Restricted Differential Revision.
Feb 10 2018, 8:11 PM · Infrastructure, Restricted Project, Differential
epriestley added a revision to T8475: Clean up remnants of old Differential hunk storage: Restricted Differential Revision.
Feb 10 2018, 7:54 PM · Infrastructure, Restricted Project, Differential

Feb 6 2018

epriestley added a revision to T12101: Phabricator PHP 7 Compatibility: Restricted Differential Revision.
Feb 6 2018, 1:00 PM · Infrastructure, Guides

Jan 31 2018

epriestley triaged T13056: Remove "data" from edge infrastructure as Low priority.
Jan 31 2018, 2:40 PM · Infrastructure

Jan 27 2018

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

Jan 25 2018

epriestley added a revision to T6203: Adjust nullability of various questionable columns: Restricted Differential Revision.
Jan 25 2018, 4:28 AM · Infrastructure

Jan 22 2018

epriestley added a revision to T12509: Plan the path forward from HMAC-SHA1: Restricted Differential Revision.
Jan 22 2018, 2:16 AM · Infrastructure, Security

Jan 4 2018

epriestley moved T10753: Remove Mercurial daemon working copy operations from Backlog to Clustering on the Mercurial board.
Jan 4 2018, 7:19 PM · Infrastructure, Daemons, Diffusion, Mercurial

Dec 22 2017

epriestley added a comment to T2312: Modifying PHP.

PHP allows and evaluates these constructions:

Dec 22 2017, 4:39 PM · Infrastructure

Nov 30 2017

epriestley added a revision to T12855: In PHP7, "Throwable" and "Error" are exciting new exception classes: Restricted Differential Revision.
Nov 30 2017, 7:29 PM · Infrastructure

Nov 28 2017

epriestley added a revision to T12101: Phabricator PHP 7 Compatibility: Restricted Differential Revision.
Nov 28 2017, 2:05 PM · Infrastructure, Guides
epriestley added a revision to T12855: In PHP7, "Throwable" and "Error" are exciting new exception classes: Restricted Differential Revision.
Nov 28 2017, 2:05 PM · Infrastructure
epriestley added a revision to T12190: In PHP7, include-time warnings from source files are ignored: Restricted Differential Revision.
Nov 28 2017, 2:05 PM · Infrastructure

Oct 23 2017

epriestley closed T12190: In PHP7, include-time warnings from source files are ignored, a subtask of T12101: Phabricator PHP 7 Compatibility, as Resolved.
Oct 23 2017, 5:54 PM · Infrastructure, Guides
epriestley closed T12190: In PHP7, include-time warnings from source files are ignored as Resolved by committing Restricted Diffusion Commit.
Oct 23 2017, 5:54 PM · Infrastructure
epriestley added a revision to T12190: In PHP7, include-time warnings from source files are ignored: Restricted Differential Revision.
Oct 23 2017, 5:17 PM · Infrastructure
epriestley added a revision to T12190: In PHP7, include-time warnings from source files are ignored: Restricted Differential Revision.
Oct 23 2017, 5:14 PM · Infrastructure

Sep 20 2017

epriestley added a comment to T12993: Datasource queries with many "JOIN ... LIKE" can have explosive complexity.

Some followup SHOW INDEXES from the case in PHI47 provided output which didn't have any evidence of key cardinality issues, so it seems less likely that this is a cardinality problem.

Sep 20 2017, 7:43 PM · Infrastructure, Typeahead, Search
epriestley created T12994: Add a setup warning if MYSQLI_ASYNC is not defined (support for MySQLi async).
Sep 20 2017, 7:28 PM · Database, Infrastructure
epriestley created T12993: Datasource queries with many "JOIN ... LIKE" can have explosive complexity.
Sep 20 2017, 7:22 PM · Infrastructure, Typeahead, Search

Sep 11 2017

epriestley closed T10560: Find ways to detect/fix/warn about MySQL indexes with poor cardinality estimates as Resolved.

bin/storage adjust and bin/storage upgrade now run ANALYZE, unless users opt-out with flags.

Sep 11 2017, 12:04 PM · Infrastructure

Aug 28 2017

epriestley closed T12855: In PHP7, "Throwable" and "Error" are exciting new exception classes, a subtask of T12101: Phabricator PHP 7 Compatibility, as Resolved.
Aug 28 2017, 7:25 PM · Infrastructure, Guides
epriestley closed T12855: In PHP7, "Throwable" and "Error" are exciting new exception classes as Resolved.

I think there will be a trickle of these for a while, but we seem to have found the most common cases.

Aug 28 2017, 7:25 PM · Infrastructure

Aug 15 2017

epriestley closed T6861: PHP sort function is unpredictable to human minds when dealing with mixed types as Resolved.

Practically, msortv() and PhutilSortVector are now the solution here. They provide high-performance (no usort*()), multi-dimensional, stable, explicitly typed sort.

Aug 15 2017, 6:17 PM · Infrastructure
epriestley closed T10967: Move Differential reviewers back to a dedicated database storage table as Resolved.

We appear to have survived this. D18398 removed the double writes. There's a bit more cleanup (we could remove the edge type and actually destroy the older data) but that can happen whenever we get around to it.

Aug 15 2017, 5:51 PM · Restricted Project, Infrastructure, Differential

Aug 11 2017

epriestley added a revision to T10967: Move Differential reviewers back to a dedicated database storage table: Restricted Differential Revision.
Aug 11 2017, 4:47 PM · Restricted Project, Infrastructure, Differential

Jul 27 2017

epriestley added a comment to T12404: Implement a first-party SMTP client.

This doesn't affect us, but more fuel on the fire -- XSS in example code which ships with PHPMailer (we don't distribute this code):

Jul 27 2017, 2:55 PM · Infrastructure, Mail

Jul 18 2017

epriestley closed T12926: Document the existence of the LB `/status/` endpoint as Resolved by committing Restricted Diffusion Commit.
Jul 18 2017, 4:08 PM · Infrastructure, Setup, Documentation
epriestley added a revision to T12926: Document the existence of the LB `/status/` endpoint: Restricted Differential Revision.
Jul 18 2017, 3:58 PM · Infrastructure, Setup, Documentation
epriestley created T12926: Document the existence of the LB `/status/` endpoint.
Jul 18 2017, 9:41 AM · Infrastructure, Setup, Documentation

Jul 9 2017

chad set the icon for Infrastructure to Infrastructure.
Jul 9 2017, 7:36 PM
chad updated the image for Infrastructure from F5039288: profile to F5039289: profile.
Jul 9 2017, 7:36 PM
chad set the image for Infrastructure to F5039288: profile.
Jul 9 2017, 7:36 PM

Jun 27 2017

epriestley closed T12870: Configuration check for "maniphest.priorities" does not validate that keywords are unique across priorities, a subtask of T12845: Convert remaining ancient configuration type classes to modern modular type classes, as Resolved.
Jun 27 2017, 7:35 PM · Infrastructure, Config
epriestley renamed T12845: Convert remaining ancient configuration type classes to modern modular type classes from Setting and removing a config for maniphest.priorities results in a null config instead of reverting to the default to Convert remaining ancient configuration type classes to modern modular type classes.
Jun 27 2017, 7:12 PM · Infrastructure, Config

Jun 23 2017

epriestley added a revision to T12855: In PHP7, "Throwable" and "Error" are exciting new exception classes: Restricted Differential Revision.
Jun 23 2017, 4:21 PM · Infrastructure
epriestley added a project to T12855: In PHP7, "Throwable" and "Error" are exciting new exception classes: Infrastructure.
Jun 23 2017, 3:54 PM · Infrastructure

Jun 19 2017

epriestley added a subtask for T12101: Phabricator PHP 7 Compatibility: T12855: In PHP7, "Throwable" and "Error" are exciting new exception classes.
Jun 19 2017, 4:24 PM · Infrastructure, Guides

May 26 2017

amckinley closed T12104: Clean up remnants of old Differential draft storage as Resolved by committing Restricted Diffusion Commit.
May 26 2017, 8:59 PM · Infrastructure, Differential
amckinley added a revision to T12104: Clean up remnants of old Differential draft storage: Restricted Differential Revision.
May 26 2017, 8:51 PM · Infrastructure, Differential
epriestley added a comment to T12104: Clean up remnants of old Differential draft storage.

That sounds right to me.

May 26 2017, 8:39 PM · Infrastructure, Differential
amckinley claimed T12104: Clean up remnants of old Differential draft storage.

This looks pretty easy. No mentions of DifferentialDraft in the codebase other than the class itself and a single migration. Steps here (I think) are:

May 26 2017, 8:25 PM · Infrastructure, Differential

May 5 2017

epriestley created T12678: Bring sanity to the world of all MySQL data being string-typed.
May 5 2017, 4:11 PM · Database, Infrastructure

May 4 2017

epriestley closed T12546: "bin/remote deploy" picks up deployment keys locally instead of on the bastion as Resolved by committing Restricted Diffusion Commit.
May 4 2017, 10:29 PM · Ops, Infrastructure, Phacility

Apr 14 2017

epriestley closed T12554: PhabricatorAuthSSHKeyQuery cache is not dirtied on username changes as Resolved by committing Restricted Diffusion Commit.
Apr 14 2017, 3:03 PM · Customer Impact, Infrastructure, Diffusion, Bug Report
epriestley added a comment to T12554: PhabricatorAuthSSHKeyQuery cache is not dirtied on username changes.

Totally my fault, I just didn't read half that sentence. 🐈

Apr 14 2017, 2:59 PM · Customer Impact, Infrastructure, Diffusion, Bug Report
edward added a comment to T12554: PhabricatorAuthSSHKeyQuery cache is not dirtied on username changes.

Thanks for looking into this so quickly.

Apr 14 2017, 2:59 PM · Customer Impact, Infrastructure, Diffusion, Bug Report
epriestley claimed T12554: PhabricatorAuthSSHKeyQuery cache is not dirtied on username changes.
Apr 14 2017, 12:59 PM · Customer Impact, Infrastructure, Diffusion, Bug Report
epriestley added a comment to T12554: PhabricatorAuthSSHKeyQuery cache is not dirtied on username changes.

And thanks for the report! Let us know if you run into anything else.

Apr 14 2017, 12:10 PM · Customer Impact, Infrastructure, Diffusion, Bug Report
epriestley added a comment to T12554: PhabricatorAuthSSHKeyQuery cache is not dirtied on username changes.

D17687 should dirty the cache properly on username changes. It should promote to stable within the next ~12 hours, and deploy to Phacility production about 24 hours from now.

Apr 14 2017, 12:10 PM · Customer Impact, Infrastructure, Diffusion, Bug Report
epriestley added a revision to T12554: PhabricatorAuthSSHKeyQuery cache is not dirtied on username changes: Restricted Differential Revision.
Apr 14 2017, 12:09 PM · Customer Impact, Infrastructure, Diffusion, Bug Report
epriestley added a comment to T12554: PhabricatorAuthSSHKeyQuery cache is not dirtied on username changes.

I was able to resolve the issue by deleting my user's ssh key, and uploading the original ssh key material as a new Phabricator sshkey.

Apr 14 2017, 12:02 PM · Customer Impact, Infrastructure, Diffusion, Bug Report
epriestley added a comment to T12554: PhabricatorAuthSSHKeyQuery cache is not dirtied on username changes.

See also somewhat-related task T12397.

Apr 14 2017, 11:55 AM · Customer Impact, Infrastructure, Diffusion, Bug Report
epriestley renamed T12554: PhabricatorAuthSSHKeyQuery cache is not dirtied on username changes from Git push failed, "there is no user with this username" to PhabricatorAuthSSHKeyQuery cache is not dirtied on username changes.
Apr 14 2017, 11:50 AM · Customer Impact, Infrastructure, Diffusion, Bug Report

Apr 12 2017

epriestley added a revision to T12546: "bin/remote deploy" picks up deployment keys locally instead of on the bastion: Restricted Differential Revision.
Apr 12 2017, 8:12 PM · Ops, Infrastructure, Phacility
epriestley created T12546: "bin/remote deploy" picks up deployment keys locally instead of on the bastion.
Apr 12 2017, 7:08 PM · Ops, Infrastructure, Phacility
epriestley moved T11741: Quickstart can fail to initialize databases if MyISAM is not available (currently, only in Google Cloud) from Backlog to v2 on the Search board.
Apr 12 2017, 4:27 PM
epriestley added a project to T11741: Quickstart can fail to initialize databases if MyISAM is not available (currently, only in Google Cloud): Search.
Apr 12 2017, 4:27 PM

Apr 10 2017

epriestley closed T12526: parse_url() behavior has changed with PHP7, causing libphutil unit tests to fail and possibly creating security concerns, a subtask of T12101: Phabricator PHP 7 Compatibility, as Resolved.
Apr 10 2017, 5:34 PM · Infrastructure, Guides
epriestley added a subtask for T12101: Phabricator PHP 7 Compatibility: T12526: parse_url() behavior has changed with PHP7, causing libphutil unit tests to fail and possibly creating security concerns.
Apr 10 2017, 3:46 PM · Infrastructure, Guides

Apr 9 2017

epriestley moved T11701: Add a lint rule for `if (CONSTANT) { ... }` from Ready to Intermediate on the Contributor Onboarding board.
Apr 9 2017, 10:53 AM · Contributor Onboarding, Infrastructure, Arcanist, Lint
epriestley closed T2794: Implement Phage (like Hypershell) as Resolved.

I've been doing production deployments with this for a while without hitting any meaningful issues. It's not general-purpose yet but followups can live in Phage from here on out.

Apr 9 2017, 10:02 AM · Phage, Infrastructure

Apr 7 2017

epriestley placed T9690: Remove calls to methods deprecated by newPage() up for grabs.
Apr 7 2017, 1:04 PM · Contributor Onboarding, Modernization, Infrastructure
epriestley closed T12196: PHP7, particularly before 7.1.2, segfaults a whole lot while running CLI scripts, a subtask of T12101: Phabricator PHP 7 Compatibility, as Resolved.
Apr 7 2017, 12:55 PM · Infrastructure, Guides

Apr 6 2017

epriestley added a revision to T12509: Plan the path forward from HMAC-SHA1: Restricted Differential Revision.
Apr 6 2017, 5:10 PM · Infrastructure, Security
epriestley added a revision to T12509: Plan the path forward from HMAC-SHA1: Restricted Differential Revision.
Apr 6 2017, 5:03 PM · Infrastructure, Security
epriestley added a revision to T12509: Plan the path forward from HMAC-SHA1: Restricted Differential Revision.
Apr 6 2017, 4:38 PM · Infrastructure, Security
epriestley added a comment to T12509: Plan the path forward from HMAC-SHA1.

(That said, the hash cost may become very relevant when we eventually ship "Phabricator Valuable Golden Coin Money", our blockchain-based virtual currency.)

Apr 6 2017, 2:01 PM · Infrastructure, Security
epriestley added a comment to T12509: Plan the path forward from HMAC-SHA1.

I get these rough per-hash costs locally (Macbook Pro), with a 64-byte key:

Apr 6 2017, 1:05 PM · Infrastructure, Security
siepkes added a comment to T12509: Plan the path forward from HMAC-SHA1.

Piece of info (you guys might already be aware of it) which might be of interest when implementing this; SHA512 is often faster then SHA256 on x64. See for example: https://crypto.stackexchange.com/questions/26336/sha512-faster-than-sha256

Apr 6 2017, 12:54 PM · Infrastructure, Security

Apr 5 2017

epriestley added a comment to T12509: Plan the path forward from HMAC-SHA1.

I'm maybe going to try to do this in the short term:

Apr 5 2017, 8:26 PM · Infrastructure, Security
epriestley created T12509: Plan the path forward from HMAC-SHA1.
Apr 5 2017, 8:21 PM · Infrastructure, Security

Apr 4 2017

epriestley removed a project from T7954: Make it easier to upload XHProf profiles: Files.

(This isn't really very closely related to the Files application anymore, just cleaning up that board a little.)

Apr 4 2017, 1:01 PM · Infrastructure, XHProf

Apr 3 2017

epriestley closed T12496: "differential.sticky-accept" isn't working because of overbroad accept downgrading, a subtask of T10967: Move Differential reviewers back to a dedicated database storage table, as Resolved.
Apr 3 2017, 4:55 PM · Restricted Project, Infrastructure, Differential
epriestley added a subtask for T10967: Move Differential reviewers back to a dedicated database storage table: T12496: "differential.sticky-accept" isn't working because of overbroad accept downgrading.
Apr 3 2017, 11:10 AM · Restricted Project, Infrastructure, Differential

Mar 28 2017

epriestley closed T12271: When users accept a revision, allow them to accept for a subset of package/project reviewers they could accept for, a subtask of T10967: Move Differential reviewers back to a dedicated database storage table, as Resolved.
Mar 28 2017, 1:31 PM · Restricted Project, Infrastructure, Differential
epriestley added a revision to T10967: Move Differential reviewers back to a dedicated database storage table: Restricted Differential Revision.
Mar 28 2017, 1:24 PM · Restricted Project, Infrastructure, Differential
epriestley added a revision to T10967: Move Differential reviewers back to a dedicated database storage table: Restricted Differential Revision.
Mar 28 2017, 1:10 PM · Restricted Project, Infrastructure, Differential
epriestley added a revision to T10967: Move Differential reviewers back to a dedicated database storage table: Restricted Differential Revision.
Mar 28 2017, 12:52 PM · Restricted Project, Infrastructure, Differential
epriestley added a comment to T10967: Move Differential reviewers back to a dedicated database storage table.

The workflow thing is somewhat nontrivial. Previously, we would explicitly downgrade "Accept" into "Accepted Older" when an author used "Request Review".

Mar 28 2017, 11:57 AM · Restricted Project, Infrastructure, Differential
epriestley added a comment to T10967: Move Differential reviewers back to a dedicated database storage table.

From D17564, there's a minor workflow issue with "request review" incorrectly putting accepted changes back into an "accepted" state from this.

Mar 28 2017, 11:32 AM · Restricted Project, Infrastructure, Differential

Mar 22 2017

ruslan_a added a comment to T8510: Typeahead project proposals in Maniphest advanced search do not always include exact matches.

Hmm. Ok. Deleted.

Mar 22 2017, 7:30 PM · Typeahead, Prioritized, Wikimedia, Infrastructure, Search
epriestley closed T11050: Allow resigning from a review you are not directly responsible for, a subtask of T10967: Move Differential reviewers back to a dedicated database storage table, as Resolved.
Mar 22 2017, 4:50 PM · Restricted Project, Infrastructure, Differential
chad added a comment to T8510: Typeahead project proposals in Maniphest advanced search do not always include exact matches.

Specifically, this task is about how the javascript in typeaheads rank and match choices.

Mar 22 2017, 2:56 PM · Typeahead, Prioritized, Wikimedia, Infrastructure, Search
chad added a comment to T8510: Typeahead project proposals in Maniphest advanced search do not always include exact matches.

Not similar.

Mar 22 2017, 2:55 PM · Typeahead, Prioritized, Wikimedia, Infrastructure, Search
ruslan_a added a comment to T8510: Typeahead project proposals in Maniphest advanced search do not always include exact matches.
Mar 22 2017, 2:48 PM · Typeahead, Prioritized, Wikimedia, Infrastructure, Search
epriestley added a revision to T10967: Move Differential reviewers back to a dedicated database storage table: Restricted Differential Revision.
Mar 22 2017, 2:00 PM · Restricted Project, Infrastructure, Differential

Mar 20 2017

epriestley added a revision to T10967: Move Differential reviewers back to a dedicated database storage table: Restricted Differential Revision.
Mar 20 2017, 10:09 PM · Restricted Project, Infrastructure, Differential
epriestley added a revision to T10967: Move Differential reviewers back to a dedicated database storage table: Restricted Differential Revision.
Mar 20 2017, 9:46 PM · Restricted Project, Infrastructure, Differential
epriestley added a revision to T10967: Move Differential reviewers back to a dedicated database storage table: Restricted Differential Revision.
Mar 20 2017, 9:10 PM · Restricted Project, Infrastructure, Differential
epriestley added a revision to T10967: Move Differential reviewers back to a dedicated database storage table: Restricted Differential Revision.
Mar 20 2017, 8:24 PM · Restricted Project, Infrastructure, Differential
epriestley added a revision to T10967: Move Differential reviewers back to a dedicated database storage table: Restricted Differential Revision.
Mar 20 2017, 6:34 PM · Restricted Project, Infrastructure, Differential
epriestley added a revision to T10967: Move Differential reviewers back to a dedicated database storage table: Restricted Differential Revision.
Mar 20 2017, 5:01 PM · Restricted Project, Infrastructure, Differential
epriestley added a revision to T10967: Move Differential reviewers back to a dedicated database storage table: Restricted Differential Revision.
Mar 20 2017, 4:37 PM · Restricted Project, Infrastructure, Differential
epriestley added a comment to T10967: Move Differential reviewers back to a dedicated database storage table.

Downgrading accepts and rejects (changing "Accepted" into "Accepted Older") uses raw edge transactions.

Mar 20 2017, 4:23 PM · Restricted Project, Infrastructure, Differential
epriestley added a revision to T10967: Move Differential reviewers back to a dedicated database storage table: Restricted Differential Revision.
Mar 20 2017, 4:16 PM · Restricted Project, Infrastructure, Differential