Page MenuHomePhabricator
Feed Advanced Search

Apr 21 2017

epriestley added a comment to T12612: Design (and possibly implement) production rate limiting controls.

Here's what we currently have:

Apr 21 2017, 2:02 PM · Phacility, Ops
epriestley lowered the priority of T12612: Design (and possibly implement) production rate limiting controls from High to Normal.
Apr 21 2017, 12:49 PM · Phacility, Ops
epriestley shifted T12612: Design (and possibly implement) production rate limiting controls from the Restricted Space space to the S1 Core space.
Apr 21 2017, 12:49 PM · Phacility, Ops
epriestley added a comment to T12612: Design (and possibly implement) production rate limiting controls.

I'm going to kick this out of the ops space since none of this should be sensitive.

Apr 21 2017, 12:49 PM · Phacility, Ops

Apr 20 2017

epriestley created T12612: Design (and possibly implement) production rate limiting controls.
Apr 20 2017, 9:22 PM · Phacility, Ops
epriestley created T12611: Write Phabricator HTTP and SSH logs in the production cluster.
Apr 20 2017, 9:20 PM · Phacility, Ops
epriestley added a comment to T12610: Audit behavior of LB healthchecks against *.phacility.com and secure.phabricator.com.

The current ruleset implements all those rules correctly, except that it 200's directly for /status/.

Apr 20 2017, 8:55 PM · Ops, Phacility
epriestley added a comment to T12610: Audit behavior of LB healthchecks against *.phacility.com and secure.phabricator.com.

redirect to Javelin

Apr 20 2017, 8:55 PM · Ops, Phacility
epriestley added a comment to T12610: Audit behavior of LB healthchecks against *.phacility.com and secure.phabricator.com.

On secure, two not-so-great fixes are:

Apr 20 2017, 8:54 PM · Ops, Phacility
amckinley created T12610: Audit behavior of LB healthchecks against *.phacility.com and secure.phabricator.com.
Apr 20 2017, 8:50 PM · Ops, Phacility

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 T12296: Improve Phacility repository import performance from Backlog to "Customers" on the Customer Impact board.
Apr 12 2017, 3:36 PM · Customer Impact, Diffusion, Ops
epriestley edited projects for T12296: Improve Phacility repository import performance, added: Customer Impact; removed Phacility.
Apr 12 2017, 3:36 PM · Customer Impact, Diffusion, Ops
epriestley edited projects for T12379: Instance reporting difficulty pushing the Linux kernel to a Phacility instance, added: Customer Impact; removed Phacility.
Apr 12 2017, 3:10 PM · Customer Impact, Diffusion, Ops

Apr 9 2017

epriestley closed T2794: Implement Phage (like Hypershell), a subtask of T12218: Reduce the operational cost of a larger Phacility cluster, as Resolved.
Apr 9 2017, 10:02 AM · Ops, Phacility

Apr 6 2017

epriestley added a revision to T12217: Reduce the hardware cost of Phacility free instances: Restricted Differential Revision.
Apr 6 2017, 10:40 PM · Ops, Phacility
epriestley added a revision to T12298: Allow daemon pools to autoscale down to 0 processes: Restricted Differential Revision.
Apr 6 2017, 10:26 PM · Daemons, Ops, Phacility

Mar 25 2017

epriestley closed T12448: Rotate 2017 SSL certificate for `secure.phabricator.com` as Resolved.

These are now up to date.

Mar 25 2017, 12:56 PM · Ops

Mar 24 2017

epriestley created T12448: Rotate 2017 SSL certificate for `secure.phabricator.com`.
Mar 24 2017, 8:47 PM · Ops
epriestley added a revision to T12298: Allow daemon pools to autoscale down to 0 processes: Restricted Differential Revision.
Mar 24 2017, 8:44 PM · Daemons, Ops, Phacility
epriestley added a revision to T12298: Allow daemon pools to autoscale down to 0 processes: Restricted Differential Revision.
Mar 24 2017, 8:28 PM · Daemons, Ops, Phacility
epriestley added a revision to T12298: Allow daemon pools to autoscale down to 0 processes: Restricted Differential Revision.
Mar 24 2017, 3:56 PM · Daemons, Ops, Phacility
epriestley added a revision to T12298: Allow daemon pools to autoscale down to 0 processes: Restricted Differential Revision.
Mar 24 2017, 3:23 PM · Daemons, Ops, Phacility
epriestley added a revision to T12298: Allow daemon pools to autoscale down to 0 processes: Restricted Differential Revision.
Mar 24 2017, 3:21 PM · Daemons, Ops, Phacility

Mar 23 2017

epriestley added a revision to T12217: Reduce the hardware cost of Phacility free instances: Restricted Differential Revision.
Mar 23 2017, 2:35 PM · Ops, Phacility
epriestley added a revision to T12298: Allow daemon pools to autoscale down to 0 processes: Restricted Differential Revision.
Mar 23 2017, 2:02 AM · Daemons, Ops, Phacility
epriestley added a revision to T12298: Allow daemon pools to autoscale down to 0 processes: Restricted Differential Revision.
Mar 23 2017, 1:57 AM · Daemons, Ops, Phacility
epriestley added a revision to T12298: Allow daemon pools to autoscale down to 0 processes: Restricted Differential Revision.
Mar 23 2017, 1:17 AM · Daemons, Ops, Phacility

Mar 21 2017

20after4 added a comment to T12296: Improve Phacility repository import performance.

While the git ls-remote change isn't really motivated as a performance improvement, it does seem to have reduced CPU usage a measurable amount (deployed on the morning of 3/18), maybe 15%:

Mar 21 2017, 2:19 PM · Customer Impact, Diffusion, Ops
epriestley added a comment to T12296: Improve Phacility repository import performance.

While the git ls-remote change isn't really motivated as a performance improvement, it does seem to have reduced CPU usage a measurable amount (deployed on the morning of 3/18), maybe 15%:

Mar 21 2017, 2:13 PM · Customer Impact, Diffusion, Ops

Mar 18 2017

epriestley closed T12355: Daemons are having a rough time starting on `admin` as Resolved.

The setsid() appears to have resolved the issue, in the sense that daemons are now starting cleanly every time without receiving any inexplicable signals.

Mar 18 2017, 12:35 PM · Phacility, Daemons, Ops
epriestley added a comment to T12355: Daemons are having a rough time starting on `admin`.

The MySQL stuff seems to be in the old daemons (them trying to do logging while MySQL restarts), not the new daemons, and not the cause of the issue.

Mar 18 2017, 12:23 PM · Phacility, Daemons, Ops
chad added a comment to T12217: Reduce the hardware cost of Phacility free instances.

Even with a very conservative policy, we're going to drop 80% of the instances it sounds like.

Mar 18 2017, 4:33 AM · Ops, Phacility
chad added a comment to T12217: Reduce the hardware cost of Phacility free instances.

Well, I think it's reasonable to possibly not log in for a while like I keep dormant projects on GH. I don't want them to disappear, but I prefer to keep them in the cloud.

Mar 18 2017, 4:32 AM · Ops, Phacility
epriestley added a comment to T12217: Reduce the hardware cost of Phacility free instances.

We can also do other stuff like "never archive an instance with 2+ users", "after an instance emails us to unsuspend, have a flag to mark it as exempt from cleanup in the future", "don't destroy data for 365 days, instead of 90, if an instance had hosted repositories", or other sorts of things like that, but I suspect measures like that will be largely unnecessary for years to come and that "no one loaded a page for 45 days" is already a nearly perfect signal for "no one cares about this instance".

Mar 18 2017, 4:30 AM · Ops, Phacility
chad added a comment to T12217: Reduce the hardware cost of Phacility free instances.

I'd say 180 day destruction window, and mostly that's just for us to feel good about deleting stuff.

Mar 18 2017, 4:29 AM · Ops, Phacility
chad added a comment to T12217: Reduce the hardware cost of Phacility free instances.

Fair points.

Mar 18 2017, 4:27 AM · Ops, Phacility
epriestley added a comment to T12217: Reduce the hardware cost of Phacility free instances.

Picking a random shard, 80% of the active instances have zero sessions in the last 30 days -- so no user has loaded a single page in at least a month. Most (maybe all?) of these have only one account. These include (anonymized) free instances like "alsdknfasf", "test2", "test2017", "testinstance9", "testforme", "mytest", "myinstance", "bbbbbbb", "ddeemmoo". Of the 20% of active instances which do have activity in the last 30 days, many are recently-launched free instances with only one user session, and I'd guess many of those users are never coming back and these instances will join the boneyard shortly.

Mar 18 2017, 4:25 AM · Ops, Phacility
epriestley added a comment to T12217: Reduce the hardware cost of Phacility free instances.

My thinking is that v1 would work like this:

Mar 18 2017, 4:09 AM · Ops, Phacility
chad added a comment to T12217: Reduce the hardware cost of Phacility free instances.

I suppose we could get pretty far changing the language from "Disabled" to "Inactive". That is, send a warning after n days, then after 3 warnings, set it to "Inactive" but have a means they can contact us or click a button to re-activate it?

Mar 18 2017, 3:27 AM · Ops, Phacility
chad added a comment to T12217: Reduce the hardware cost of Phacility free instances.

So I've read this a few times and I'm still a little unclear what happens if "activity requirement" isn't met. It sounds like we'll just disable them like we do non-paying instances - which I don't think users expect and feels like a crumby experience. Would it just be possible to "hibernate/sleep" the instance (no daemons run, user has to wait and spin them up to log back in if 90 days later they get back to using the instance)?

Mar 18 2017, 1:49 AM · Ops, Phacility

Mar 17 2017

epriestley added a project to T12414: Implement Almanac edit endpoints in Conduit: Conduit.
Mar 17 2017, 2:46 PM · Conduit, Almanac, Ops, Phacility
epriestley created T12414: Implement Almanac edit endpoints in Conduit.
Mar 17 2017, 2:46 PM · Conduit, Almanac, Ops, Phacility

Mar 14 2017

epriestley added a revision to T12296: Improve Phacility repository import performance: Restricted Differential Revision.
Mar 14 2017, 10:25 PM · Customer Impact, Diffusion, Ops

Mar 10 2017

epriestley created T12379: Instance reporting difficulty pushing the Linux kernel to a Phacility instance.
Mar 10 2017, 1:06 AM · Customer Impact, Diffusion, Ops

Mar 8 2017

epriestley added a comment to T12355: Daemons are having a rough time starting on `admin`.

I haven't been able to reproduce this locally by artificially making services unavailable or connections fail. Debugging this in production will require momentary service interruptions on admin until I can narrow things down so I'm going to wait until off-peak to poke at it.

Mar 8 2017, 12:00 AM · Phacility, Daemons, Ops

Mar 5 2017

eliaspro added a comment to T12355: Daemons are having a rough time starting on `admin`.

In terms of application/code stability, it would be good to handle this properly.

Mar 5 2017, 9:02 PM · Phacility, Daemons, Ops
epriestley added a comment to T12355: Daemons are having a rough time starting on `admin`.

Specifically, I suspect it won't affect repo because db restarts separately, so the database won't be unavailable when daemons on repo restart.

Mar 5 2017, 12:32 PM · Phacility, Daemons, Ops
epriestley added a comment to T12355: Daemons are having a rough time starting on `admin`.

It looks like this might be an issue with MySQL not being available yet when the daemons start. The deployment script restarts MySQL, then immediately restarts the daemons, and a bunch of this stuff is ending up in the log in the 1-2 seconds after the restart:

Mar 5 2017, 12:32 PM · Phacility, Daemons, Ops
epriestley added a project to T12355: Daemons are having a rough time starting on `admin`: Phacility.
Mar 5 2017, 12:24 PM · Phacility, Daemons, Ops
epriestley created T12355: Daemons are having a rough time starting on `admin`.
Mar 5 2017, 12:24 PM · Phacility, Daemons, Ops

Mar 1 2017

epriestley added a comment to T12237: Amazon SES has suspended outbound mail from secure.phabricator.com.

Earlier, I predicted:

Mar 1 2017, 12:56 PM · Mail, Phacility, Ops

Feb 28 2017

epriestley closed T12331: Taskmasters on `admin.phacility.com` are not scaling up correctly as Resolved.

I've deployed everything so I believe this is now resolved. I'll keep an eye on it going forward.

Feb 28 2017, 4:37 PM · Daemons, Ops, Phacility
epriestley added a comment to T12331: Taskmasters on `admin.phacility.com` are not scaling up correctly.

The pool appears to be scaling up and down properly now. I'm going to deploy the fix to the secure and repo tiers since it may affect all instances.

Feb 28 2017, 4:21 PM · Daemons, Ops, Phacility
epriestley added a comment to T12331: Taskmasters on `admin.phacility.com` are not scaling up correctly.

I'm picking D17433 now and sending it to admin.

Feb 28 2017, 4:14 PM · Daemons, Ops, Phacility
epriestley added a revision to T12331: Taskmasters on `admin.phacility.com` are not scaling up correctly: Restricted Differential Revision.
Feb 28 2017, 4:10 PM · Daemons, Ops, Phacility
epriestley added a comment to T12331: Taskmasters on `admin.phacility.com` are not scaling up correctly.

I'm picking those to stable and sending them to production now.

Feb 28 2017, 3:43 PM · Daemons, Ops, Phacility
epriestley added a revision to T12331: Taskmasters on `admin.phacility.com` are not scaling up correctly: Restricted Differential Revision.
Feb 28 2017, 3:37 PM · Daemons, Ops, Phacility
epriestley added a revision to T12331: Taskmasters on `admin.phacility.com` are not scaling up correctly: Restricted Differential Revision.
Feb 28 2017, 3:31 PM · Daemons, Ops, Phacility
epriestley created T12331: Taskmasters on `admin.phacility.com` are not scaling up correctly.
Feb 28 2017, 12:06 PM · Daemons, Ops, Phacility
joshuaspence added a revision to T12298: Allow daemon pools to autoscale down to 0 processes: Restricted Differential Revision.
Feb 28 2017, 3:49 AM · Daemons, Ops, Phacility

Feb 27 2017

epriestley closed T12316: Phacility Deploy Log 2017 Week 8, a subtask of T12217: Reduce the hardware cost of Phacility free instances, as Resolved.
Feb 27 2017, 6:46 PM · Ops, Phacility
epriestley closed T12316: Phacility Deploy Log 2017 Week 8 as Resolved.
Feb 27 2017, 6:46 PM · Phacility, Ops
20after4 added a comment to T12218: Reduce the operational cost of a larger Phacility cluster.

For wikimedia's "scap" tool we built a feature called "checks" which are configurable commands that run on each target machine and confirm if the desired commands were successful.

Feb 27 2017, 6:00 PM · Ops, Phacility
epriestley added a comment to T12218: Reduce the operational cost of a larger Phacility cluster.

I did the last deploy (T12316) with Phage again and things went much more smoothly after the various bugfixes and improvements.

Feb 27 2017, 3:10 PM · Ops, Phacility

Feb 26 2017

Mnkras added a comment to T12296: Improve Phacility repository import performance.

Cool, now I don't feel so bad since I know you only recently added the option 😄

Feb 26 2017, 4:44 AM · Customer Impact, Diffusion, Ops
epriestley added a comment to T12296: Improve Phacility repository import performance.

(That said, my expectation is that behaviors around unreachable commits have not changed -- D17279 added the "Unreachable" option to solve this problem.)

Feb 26 2017, 4:43 AM · Customer Impact, Diffusion, Ops
epriestley added a comment to T12296: Improve Phacility repository import performance.

Likely somewhere in T9430 / T10978, which rewrote a big chunk of the audit code.

Feb 26 2017, 4:42 AM · Customer Impact, Diffusion, Ops
Mnkras added a comment to T12296: Improve Phacility repository import performance.

That works, Im curious when this changed, as I didn't need to filter since the last few weeks.

Feb 26 2017, 4:37 AM · Customer Impact, Diffusion, Ops
epriestley added a comment to T12296: Improve Phacility repository import performance.

If you don't want to see unreachable commits, use a query which hides them:

Feb 26 2017, 4:32 AM · Customer Impact, Diffusion, Ops
Mnkras added a comment to T12296: Improve Phacility repository import performance.

The reason I think something isn't working correctly is in my dashboard, I list recent commits, and since I updated my install a few weeks ago my commit list looks like this a lot:

pasted_file (1×1 px, 470 KB)

Feb 26 2017, 4:29 AM · Customer Impact, Diffusion, Ops
epriestley added a comment to T12296: Improve Phacility repository import performance.

It is expected that some deleted commits will import and others will not.

Feb 26 2017, 4:05 AM · Customer Impact, Diffusion, Ops
Mnkras added a comment to T12296: Improve Phacility repository import performance.

@epriestley all other commits that have the message "This commit has been deleted in the repository: it is no longer reachable from any branch, tag, or ref." have imported, so I don't think the reachable code is working correctly, as until within the last few weeks I wasn't seeing these "This commit has been deleted in the repository: it is no longer reachable from any branch, tag, or ref." commits showing up, but now I see them all the time.

Feb 26 2017, 1:30 AM · Customer Impact, Diffusion, Ops

Feb 25 2017

epriestley added a comment to T12297: Make Conduit API calls on `admin.phacility.com` reasonably easy to profile.

Total cost is about 1.6s. Of this:

Feb 25 2017, 4:09 PM · XHProf, Ops, Phacility
epriestley added a comment to T12297: Make Conduit API calls on `admin.phacility.com` reasonably easy to profile.

This is still a little bit of a pain but far more reasonable now, here's an actual profile:

Feb 25 2017, 4:03 PM · XHProf, Ops, Phacility
epriestley added a comment to T12316: Phacility Deploy Log 2017 Week 8.

I cycled the rest of the tier and things look better now.

Feb 25 2017, 4:00 PM · Phacility, Ops
epriestley added a comment to T12316: Phacility Deploy Log 2017 Week 8.

After manually deploying that to repo001, I'm now seeing a consistent Taskmaster/PullLocal count. I'm redeploying the rest of the repo hosts.

Feb 25 2017, 3:05 PM · Phacility, Ops
epriestley added a comment to T12316: Phacility Deploy Log 2017 Week 8.

I think the issue is that the --reserve flag for reserving system memory now applies to even the first daemon in a pool, because all daemons are now part of pools. Previously, it only applied only to the 2nd..Nth daemon.

Feb 25 2017, 3:02 PM · Phacility, Ops
epriestley added a comment to T12316: Phacility Deploy Log 2017 Week 8.

Some taskmasters failed to start on repo001 and perhaps elsewhere.

Feb 25 2017, 2:46 PM · Phacility, Ops
epriestley added a comment to T12316: Phacility Deploy Log 2017 Week 8.

I suspended and destroyed the largest instance on the shard, a free instance with one user and no logins for >30 days (the user appears to have imported a huge repository and then never logged back in). We still have backups, but this instance was way over reasonable resource use anyway. The volume is now 78% full.

Feb 25 2017, 2:45 PM · Phacility, Ops
epriestley added a comment to T12316: Phacility Deploy Log 2017 Week 8.

I pruned some very old backups (backup pruning is currently very conservative in several cases). The volume is now 72% full. This could easily be reduced much further by allowing bin/host prune to remove backups older than 90 days for suspended/terminated instances that have been out of service for 90 days, but backup utilization is generally relatively stable so I'm not concerned about this.

Feb 25 2017, 2:26 PM · Phacility, Ops
epriestley added a comment to T12316: Phacility Deploy Log 2017 Week 8.

I synchronized services for this shard and destroyed old instances. The volume is now 68% full.

Feb 25 2017, 2:13 PM · Phacility, Ops
epriestley added a comment to T12316: Phacility Deploy Log 2017 Week 8.

I synchronized services for this shard and destroyed old instances. The volume is now 44% full.

Feb 25 2017, 1:44 PM · Phacility, Ops
epriestley added a comment to T12316: Phacility Deploy Log 2017 Week 8.

This was a bunch of repository file data in /tmp. Each file was unique, so this doesn't look like any sort of buggy loop or anything. There's some actual bug here but since there's a long list of other issues here I just "fixed" this by clearing /tmp.

Feb 25 2017, 1:08 PM · Phacility, Ops
epriestley added a comment to T12316: Phacility Deploy Log 2017 Week 8.

Some reserved instances ended on Feb 19th; I should make sure reserved instances match production instances.

Feb 25 2017, 1:02 PM · Phacility, Ops
epriestley added a comment to T12316: Phacility Deploy Log 2017 Week 8.

Shards 017 through 020 are now in production.

Feb 25 2017, 12:57 PM · Phacility, Ops
epriestley added a comment to T12316: Phacility Deploy Log 2017 Week 8.

The re-launched hosts seem OK, I'm waiting for DNS to update and then redeploying them.

Feb 25 2017, 12:10 PM · Phacility, Ops
epriestley added a comment to T12316: Phacility Deploy Log 2017 Week 8.

The hosts launched into an "Auto-Assign Public IPv4" subnet but didn't get IPv4 addresses. Restarting them didn't help, and there's no apparent way to assign public addresses after the fact. I'm going to throw them away and try again.

Feb 25 2017, 12:06 PM · Phacility, Ops
epriestley added a comment to T12316: Phacility Deploy Log 2017 Week 8.

I'm going to put more hosts into production immediately before pursuing other issues.

Feb 25 2017, 11:59 AM · Phacility, Ops
epriestley added a comment to T12316: Phacility Deploy Log 2017 Week 8.
  • Some reserved instances ended on Feb 19th; I should make sure reserved instances match production instances.
Feb 25 2017, 11:29 AM · Phacility, Ops
epriestley added a comment to T12316: Phacility Deploy Log 2017 Week 8.
  • The tier as a whole is 88% full.
Feb 25 2017, 11:12 AM · Phacility, Ops
epriestley added a parent task for T12316: Phacility Deploy Log 2017 Week 8: T12217: Reduce the hardware cost of Phacility free instances.
Feb 25 2017, 11:11 AM · Phacility, Ops
epriestley added a subtask for T12217: Reduce the hardware cost of Phacility free instances: T12316: Phacility Deploy Log 2017 Week 8.
Feb 25 2017, 11:11 AM · Ops, Phacility
epriestley created T12316: Phacility Deploy Log 2017 Week 8.
Feb 25 2017, 11:10 AM · Phacility, Ops

Feb 24 2017

20after4 added a comment to T12296: Improve Phacility repository import performance.

The commit cache should be a fairly large performance win.

Feb 24 2017, 7:22 PM · Customer Impact, Diffusion, Ops
epriestley added a comment to T12298: Allow daemon pools to autoscale down to 0 processes.

Hibernating daemons currently show as "Waiting" in the Daemon console, but I'm not going to worry about that for now.

Feb 24 2017, 6:49 PM · Daemons, Ops, Phacility
epriestley added a comment to T12296: Improve Phacility repository import performance.

Ah, okay. This is the relevant part:

Feb 24 2017, 6:46 PM · Customer Impact, Diffusion, Ops
Mnkras added a comment to T12296: Improve Phacility repository import performance.

@epriestley
https://test-3uoifmrptebb.phacility.com/R2:80233a27d7b72bb03710d358e12a531ce46808a9

Feb 24 2017, 6:25 PM · Customer Impact, Diffusion, Ops
epriestley added a revision to T12296: Improve Phacility repository import performance: Restricted Differential Revision.
Feb 24 2017, 6:05 PM · Customer Impact, Diffusion, Ops