Page MenuHomePhabricator
Feed All Stories

Apr 12 2018

avivey added a comment to T3626: Detection of language for some file types isn't good.

Oh, good, it's about time this task be created!

image.png (141×844 px, 18 KB)

Apr 12 2018, 3:04 PM · Customer Impact, Restricted Project, Differential
epriestley renamed T3626: Detection of language for some file types isn't good from to Detection of language for some file types isn't good.
Apr 12 2018, 2:30 PM · Customer Impact, Restricted Project, Differential
epriestley renamed T6816: In diffs, when a file is both generated code and no changes were made, "Show File Contents" does not behave as expected from to In diffs, when a file is both generated code and no changes were made, "Show File Contents" does not behave as expected.
Apr 12 2018, 2:29 PM · Restricted Project, Differential
epriestley renamed T1101: Allow Show Raw File for updated added files from to Allow Show Raw File for updated added files.
Apr 12 2018, 2:29 PM · Differential
epriestley renamed T3626: Detection of language for some file types isn't good from Detection of language for some file types isn't good to .
Apr 12 2018, 2:28 PM · Customer Impact, Restricted Project, Differential
epriestley renamed T6816: In diffs, when a file is both generated code and no changes were made, "Show File Contents" does not behave as expected from In diffs, when a file is both generated code and no changes were made, "Show File Contents" does not behave as expected to .
Apr 12 2018, 2:28 PM · Restricted Project, Differential
epriestley renamed T1101: Allow Show Raw File for updated added files from Allow Show Raw File for updated added files to .
Apr 12 2018, 2:28 PM · Differential
n3v3rf411 added a comment to T4190: Rebuild remarkup image proxying.

I think https://secure.phabricator.com/book/phabricator/article/remarkup/ needs update to include this.

Apr 12 2018, 8:43 AM · Contributor Onboarding, Remarkup
epriestley added a comment to T13121: Remove "-q" from SSH commands executed by `bin/remote` and similar cluster commands.

I'm leaning toward:

Apr 12 2018, 12:11 AM · Phacility, Ops
epriestley triaged T13121: Remove "-q" from SSH commands executed by `bin/remote` and similar cluster commands as Low priority.
Apr 12 2018, 12:08 AM · Phacility, Ops

Apr 11 2018

amckinley created P2093 (An Untitled Masterwork).
Apr 11 2018, 11:25 PM
amckinley created P2092 (An Untitled Masterwork).
Apr 11 2018, 11:20 PM
epriestley updated the task description for T13081: Quicksand breaks the back button after a reload, et al.
Apr 11 2018, 9:48 PM · Quicksand
epriestley added a revision to T13105: Plans: Rich presentation and diff rendering pipelines for various file types: Unknown Object (Differential Revision).
Apr 11 2018, 9:47 PM · Plans, Files, Differential, Diffusion
epriestley added a revision to T13105: Plans: Rich presentation and diff rendering pipelines for various file types: Unknown Object (Differential Revision).
Apr 11 2018, 9:30 PM · Plans, Files, Differential, Diffusion
epriestley added a revision to T13105: Plans: Rich presentation and diff rendering pipelines for various file types: Unknown Object (Differential Revision).
Apr 11 2018, 9:15 PM · Plans, Files, Differential, Diffusion
amckinley created P2091 (An Untitled Masterwork).
Apr 11 2018, 8:45 PM
epriestley added a revision to T13105: Plans: Rich presentation and diff rendering pipelines for various file types: Unknown Object (Differential Revision).
Apr 11 2018, 8:45 PM · Plans, Files, Differential, Diffusion
epriestley added a comment to P2090 (An Untitled Masterwork).

Hmm, I can't reproduce the issue. Here's pre-comment:

Apr 11 2018, 7:09 PM
epriestley added a comment to T13105: Plans: Rich presentation and diff rendering pipelines for various file types.
  • Source view rendering in Paste has a missing background color.
Apr 11 2018, 7:06 PM · Plans, Files, Differential, Diffusion
amckinley created P2090 (An Untitled Masterwork).
Apr 11 2018, 7:04 PM
amckinley created P2089 (An Untitled Masterwork).
Apr 11 2018, 6:04 PM
epriestley closed T12414: Implement Almanac edit endpoints in Conduit, a subtask of T12218: Reduce the operational cost of a larger Phacility cluster, as Resolved.
Apr 11 2018, 5:54 PM · Ops, Phacility
epriestley closed T12414: Implement Almanac edit endpoints in Conduit as Resolved.

I think that's pretty much everything. There will be a little followup work in T10883 and maybe T13076 / T13120.

Apr 11 2018, 5:54 PM · Conduit, Almanac, Ops, Phacility
epriestley added a revision to T13120: Plans: 2018 Week 15 Bonus Content: Unknown Object (Differential Revision).
Apr 11 2018, 4:44 PM · Plans
epriestley updated the task description for T13120: Plans: 2018 Week 15 Bonus Content.
Apr 11 2018, 4:44 PM · Plans
epriestley added a revision to T12414: Implement Almanac edit endpoints in Conduit: Unknown Object (Differential Revision).
Apr 11 2018, 4:28 PM · Conduit, Almanac, Ops, Phacility
epriestley added a revision to T13120: Plans: 2018 Week 15 Bonus Content: Unknown Object (Differential Revision).
Apr 11 2018, 4:28 PM · Plans
amckinley closed T11476: Record transaction histories when users edit application capabilities as Resolved by committing Restricted Diffusion Commit.
Apr 11 2018, 3:55 PM · Contributor Onboarding, Restricted Project, Applications, Feature Request
epriestley added a revision to T13120: Plans: 2018 Week 15 Bonus Content: Unknown Object (Differential Revision).
Apr 11 2018, 3:41 PM · Plans
epriestley added a revision to T12414: Implement Almanac edit endpoints in Conduit: Unknown Object (Differential Revision).
Apr 11 2018, 3:41 PM · Conduit, Almanac, Ops, Phacility
epriestley added a revision to T13120: Plans: 2018 Week 15 Bonus Content: Unknown Object (Differential Revision).
Apr 11 2018, 2:17 PM · Plans
epriestley added a revision to T12414: Implement Almanac edit endpoints in Conduit: Unknown Object (Differential Revision).
Apr 11 2018, 2:17 PM · Conduit, Almanac, Ops, Phacility
epriestley added a comment to T12414: Implement Almanac edit endpoints in Conduit.

Everything here should pretty much work except:

Apr 11 2018, 2:08 PM · Conduit, Almanac, Ops, Phacility
epriestley added a revision to T12414: Implement Almanac edit endpoints in Conduit: Unknown Object (Differential Revision).
Apr 11 2018, 1:40 PM · Conduit, Almanac, Ops, Phacility
epriestley added a revision to T13120: Plans: 2018 Week 15 Bonus Content: Unknown Object (Differential Revision).
Apr 11 2018, 1:40 PM · Plans
DanielO added a comment to T9450: "abort: not a Mercurial bundle" with Mercurial 3.5>=.

Actually turns out that the work around isn't reliable - I still get that error occasionally (I'm using SSH)

Apr 11 2018, 2:48 AM · Mercurial
DanielO added a comment to T9450: "abort: not a Mercurial bundle" with Mercurial 3.5>=.

The same problem occurred on my site with the latest Phabricator and the latest Mercurial 3.7.2 installed on both sides.

Interestingly, it can be cured setting a special development debug flag for in the Mercurial configuration file:
in /etc/mercurial/hgrc or in vcs-user's ${HOME}/.hgrc adding the following snippet

[devel]
bundle2.debug = true

caused a hg push to a hosted Phabricator repository to work without a problem. But do not ask me, why.

Apr 11 2018, 2:14 AM · Mercurial

Apr 10 2018

amckinley added a revision to T11476: Record transaction histories when users edit application capabilities: Unknown Object (Differential Revision).
Apr 10 2018, 11:02 PM · Contributor Onboarding, Restricted Project, Applications, Feature Request
epriestley added a comment to T12999: Replace cluster magnetic volumes with SSD volumes.

I think all the T13076 stuff is blocked on me for now unless you're feeling especially ambitious about trying to get bin/host download working on 2GB+ files (T12907 / D19011). That might be a bit of a mess of a task though since I think I have a lot of secret cURL knowledge from over the years that is only somewhat-documented in HTTPSFuture.

Apr 10 2018, 10:47 PM · Phacility, Ops
amckinley added a comment to T12999: Replace cluster magnetic volumes with SSD volumes.

Oh, I'm happy to kick this down the road until we go through the Big Compaction. I just saw this task as a dependency for T13076, which is on this week's planning board and figured I'd tackle it. Is there a different task you think I should work on instead to move T13076 forward?

Apr 10 2018, 7:47 PM · Phacility, Ops
amckinley reassigned T12414: Implement Almanac edit endpoints in Conduit from amckinley to epriestley.
Apr 10 2018, 7:32 PM · Conduit, Almanac, Ops, Phacility
epriestley added a comment to T12999: Replace cluster magnetic volumes with SSD volumes.

Backups should run continuously (starting 12 hours after the instance launches, then every 24 hours after that):

Apr 10 2018, 7:30 PM · Phacility, Ops
epriestley added a revision to T13120: Plans: 2018 Week 15 Bonus Content: Unknown Object (Differential Revision).
Apr 10 2018, 7:22 PM · Plans
epriestley added a revision to T12414: Implement Almanac edit endpoints in Conduit: Unknown Object (Differential Revision).
Apr 10 2018, 7:22 PM · Conduit, Almanac, Ops, Phacility
amckinley added a comment to T12999: Replace cluster magnetic volumes with SSD volumes.

I'm going to go through the volumes type-by-type instead of host-by-host, starting with the backup volumes (because those should be fine to detach as long as the backup isn't running). It looks like backups run daily at 2300, so that should give me plenty of time.

Apr 10 2018, 7:05 PM · Phacility, Ops
amckinley moved T12847: A Pathway Towards Private Clusters from Soon to Future on the Plans board.
Apr 10 2018, 6:45 PM · Plans, Ops, Phacility
epriestley added a revision to T12414: Implement Almanac edit endpoints in Conduit: Unknown Object (Differential Revision).
Apr 10 2018, 6:41 PM · Conduit, Almanac, Ops, Phacility
epriestley added a revision to T13120: Plans: 2018 Week 15 Bonus Content: Unknown Object (Differential Revision).
Apr 10 2018, 6:41 PM · Plans
epriestley added a revision to T13120: Plans: 2018 Week 15 Bonus Content: Unknown Object (Differential Revision).
Apr 10 2018, 6:22 PM · Plans
epriestley added a revision to T12414: Implement Almanac edit endpoints in Conduit: Unknown Object (Differential Revision).
Apr 10 2018, 6:22 PM · Conduit, Almanac, Ops, Phacility
epriestley added a revision to T13120: Plans: 2018 Week 15 Bonus Content: Unknown Object (Differential Revision).
Apr 10 2018, 6:19 PM · Plans
epriestley added a revision to T12414: Implement Almanac edit endpoints in Conduit: Unknown Object (Differential Revision).
Apr 10 2018, 6:19 PM · Conduit, Almanac, Ops, Phacility
epriestley added a revision to T13120: Plans: 2018 Week 15 Bonus Content: Unknown Object (Differential Revision).
Apr 10 2018, 6:03 PM · Plans
epriestley added a revision to T12414: Implement Almanac edit endpoints in Conduit: Unknown Object (Differential Revision).
Apr 10 2018, 6:03 PM · Conduit, Almanac, Ops, Phacility
epriestley added a revision to T12414: Implement Almanac edit endpoints in Conduit: Unknown Object (Differential Revision).
Apr 10 2018, 5:10 PM · Conduit, Almanac, Ops, Phacility
epriestley added a revision to T13120: Plans: 2018 Week 15 Bonus Content: Unknown Object (Differential Revision).
Apr 10 2018, 5:10 PM · Plans
epriestley added a revision to T12414: Implement Almanac edit endpoints in Conduit: Unknown Object (Differential Revision).
Apr 10 2018, 5:08 PM · Conduit, Almanac, Ops, Phacility
epriestley added a revision to T13120: Plans: 2018 Week 15 Bonus Content: Unknown Object (Differential Revision).
Apr 10 2018, 5:08 PM · Plans
amckinley added a comment to T13076: Plans: Phacility cluster caching, renaming, and rebalance/compaction.

Hmmm, then maybe the narrowest way to solve this is to just setup disk space alarms. That's a thing we'd want independent of how we eventually build the One True Logging System. The CloudWatch agent collects disk stats out of the box, and takes a list of mountpoints to track. Details here: https://docs.aws.amazon.com/AmazonCloudWatch/latest/monitoring/CloudWatch-Agent-Configuration-File-Details.html

Apr 10 2018, 4:09 PM · Plans, Ops, Infrastructure, Phacility
epriestley added a revision to T12414: Implement Almanac edit endpoints in Conduit: Unknown Object (Differential Revision).
Apr 10 2018, 3:43 PM · Conduit, Almanac, Ops, Phacility
epriestley added a revision to T13120: Plans: 2018 Week 15 Bonus Content: Unknown Object (Differential Revision).
Apr 10 2018, 3:43 PM · Plans
epriestley added a revision to T12414: Implement Almanac edit endpoints in Conduit: Unknown Object (Differential Revision).
Apr 10 2018, 3:27 PM · Conduit, Almanac, Ops, Phacility
epriestley added a revision to T13120: Plans: 2018 Week 15 Bonus Content: Unknown Object (Differential Revision).
Apr 10 2018, 3:27 PM · Plans
epriestley added a revision to T12414: Implement Almanac edit endpoints in Conduit: Unknown Object (Differential Revision).
Apr 10 2018, 2:24 PM · Conduit, Almanac, Ops, Phacility
epriestley added a revision to T12414: Implement Almanac edit endpoints in Conduit: Unknown Object (Differential Revision).
Apr 10 2018, 1:38 PM · Conduit, Almanac, Ops, Phacility
epriestley added a comment to T12414: Implement Almanac edit endpoints in Conduit.

Before I can get rid of AlmanacDeviceTransaction::TYPE_INTERFACE, we have two meaningful callsites in rSERVICES and one unit test in rSAAS to clean up.

Apr 10 2018, 1:30 PM · Conduit, Almanac, Ops, Phacility
epriestley added a revision to T12414: Implement Almanac edit endpoints in Conduit: Unknown Object (Differential Revision).
Apr 10 2018, 1:28 PM · Conduit, Almanac, Ops, Phacility
epriestley added a revision to T13120: Plans: 2018 Week 15 Bonus Content: Unknown Object (Differential Revision).
Apr 10 2018, 1:28 PM · Plans
epriestley added a revision to T12414: Implement Almanac edit endpoints in Conduit: Unknown Object (Differential Revision).
Apr 10 2018, 1:17 PM · Conduit, Almanac, Ops, Phacility
epriestley added a revision to T13120: Plans: 2018 Week 15 Bonus Content: Unknown Object (Differential Revision).
Apr 10 2018, 1:17 PM · Plans
epriestley added a revision to T12414: Implement Almanac edit endpoints in Conduit: Unknown Object (Differential Revision).
Apr 10 2018, 12:57 PM · Conduit, Almanac, Ops, Phacility
epriestley added a revision to T13120: Plans: 2018 Week 15 Bonus Content: Restricted Differential Revision.
Apr 10 2018, 12:57 PM · Plans
epriestley added a comment to T12414: Implement Almanac edit endpoints in Conduit.

There's a bit of a mess with AlmanacInterface and AlmanacDevice. Currently, AlmanacInterface does not use transactions, and is edited purely as a side effect of INTERFACE transactions applying to AlmanacDevice. I'm going to change how this works so that AlmanacInterface is a normal transactional object and can use the same rules and infrastructure as everything else.

Apr 10 2018, 12:34 PM · Conduit, Almanac, Ops, Phacility
epriestley added a revision to T12414: Implement Almanac edit endpoints in Conduit: Unknown Object (Differential Revision).
Apr 10 2018, 12:21 PM · Conduit, Almanac, Ops, Phacility
epriestley added a revision to T13120: Plans: 2018 Week 15 Bonus Content: Unknown Object (Differential Revision).
Apr 10 2018, 12:21 PM · Plans
epriestley added a revision to T12414: Implement Almanac edit endpoints in Conduit: Unknown Object (Differential Revision).
Apr 10 2018, 12:15 PM · Conduit, Almanac, Ops, Phacility
epriestley added a revision to T13120: Plans: 2018 Week 15 Bonus Content: Unknown Object (Differential Revision).
Apr 10 2018, 12:15 PM · Plans
epriestley added a revision to T12414: Implement Almanac edit endpoints in Conduit: Unknown Object (Differential Revision).
Apr 10 2018, 11:51 AM · Conduit, Almanac, Ops, Phacility
epriestley added a revision to T13120: Plans: 2018 Week 15 Bonus Content: Unknown Object (Differential Revision).
Apr 10 2018, 11:51 AM · Plans
epriestley added a comment to T13076: Plans: Phacility cluster caching, renaming, and rebalance/compaction.

I think "run weekly with the deploy" already works pretty well for normal log rotation with logs on local disk, and in a perfect world logs would be buffering on disk only briefly before being sent over the network to dedicated log service so we wouldn't need cron there either.

Apr 10 2018, 11:36 AM · Plans, Ops, Infrastructure, Phacility
EddieGP added a member for Wikimedia: EddieGP.
Apr 10 2018, 9:33 AM
NDKilla added a comment to T13097: Plans: Path of Exile 3.2.0 (Bestiary League).

i feel unethical

Apr 10 2018, 3:22 AM · Phacility

Apr 9 2018

amckinley added a comment to T13076: Plans: Phacility cluster caching, renaming, and rebalance/compaction.

How often does the size directive in a logrotate script get evaluated?

Apr 9 2018, 11:48 PM · Plans, Ops, Infrastructure, Phacility
epriestley added a revision to T13118: Code block headers wrap in the dumbest way in Safari: Unknown Object (Differential Revision).
Apr 9 2018, 11:34 PM · Remarkup
cteffetalor added a comment to T13118: Code block headers wrap in the dumbest way in Safari.

Maybe vertical-align: top? Or display: inline-flex?

Apr 9 2018, 10:27 PM · Remarkup
epriestley added a comment to T13118: Code block headers wrap in the dumbest way in Safari.

yikes

Apr 9 2018, 10:05 PM · Remarkup
cteffetalor added a comment to T13118: Code block headers wrap in the dumbest way in Safari.

FYI Chrome/Windows after the change (extra whitespace added?)

Apr 9 2018, 10:04 PM · Remarkup
epriestley added a revision to T12414: Implement Almanac edit endpoints in Conduit: Unknown Object (Differential Revision).
Apr 9 2018, 9:20 PM · Conduit, Almanac, Ops, Phacility
epriestley added a revision to T13120: Plans: 2018 Week 15 Bonus Content: Unknown Object (Differential Revision).
Apr 9 2018, 9:20 PM · Plans
epriestley added a revision to T12414: Implement Almanac edit endpoints in Conduit: Unknown Object (Differential Revision).
Apr 9 2018, 9:10 PM · Conduit, Almanac, Ops, Phacility
epriestley added a revision to T13120: Plans: 2018 Week 15 Bonus Content: Unknown Object (Differential Revision).
Apr 9 2018, 9:10 PM · Plans
epriestley added a comment to T13076: Plans: Phacility cluster caching, renaming, and rebalance/compaction.

Yeah, I think "solve it forever" is probably "first-party Sentry-like application", this just seems like a problem that should have a 15-minute "solve it for now" solution in the form of something that gives us "a thing that looks like a disk but never fills up" that will hold for a year or two until we build "Phentry".

Apr 9 2018, 9:01 PM · Plans, Ops, Infrastructure, Phacility
amckinley added a comment to T13076: Plans: Phacility cluster caching, renaming, and rebalance/compaction.

That makes sense. I was coming at it more from the direction of “if we’re going to touch logs, lets solve it forever”. How about we make everything log locally through syslog and use logrotate to avoid filling disks? Then we can come back and point syslog at something more elaborate later without having to touch all the things that currently generate logs.

Apr 9 2018, 8:40 PM · Plans, Ops, Infrastructure, Phacility
epriestley added a comment to T13076: Plans: Phacility cluster caching, renaming, and rebalance/compaction.

I believe there are zero or nearly-zero actionable errors in the log today and that if we had a perfect Sentry-like system it wouldn't actually help us identify or fix any errors -- we'd just ignore it after a week like we currently ignore all the other production error logs.

Apr 9 2018, 7:49 PM · Plans, Ops, Infrastructure, Phacility
amckinley added a comment to T13076: Plans: Phacility cluster caching, renaming, and rebalance/compaction.

There's some flavor of bin/remote ssh <some-host> -- tail -f <some-log_location>.

Apr 9 2018, 7:33 PM · Plans, Ops, Infrastructure, Phacility
epriestley added a comment to T13076: Plans: Phacility cluster caching, renaming, and rebalance/compaction.

The "logs fill up disks" problem is more "sudden bursts of unusual error/activity that cause log rates to increase 1000x+ fill up disks", not "logs don't rotate / normal log volumes fill up disks". Most logs do already rotate today, I think with the exception of the aphlict log being a bit of a weird case, maybe.

Apr 9 2018, 7:14 PM · Plans, Ops, Infrastructure, Phacility
epriestley added a comment to T13076: Plans: Phacility cluster caching, renaming, and rebalance/compaction.

phage tail <some-instance-name> <some-log-type>

Apr 9 2018, 7:10 PM · Plans, Ops, Infrastructure, Phacility
amckinley added a comment to T13076: Plans: Phacility cluster caching, renaming, and rebalance/compaction.

But I don't think this is too valuable if it doesn't solve the "logs fill up disks" problem -- does the agent rotate/reap the logs?

Apr 9 2018, 7:00 PM · Plans, Ops, Infrastructure, Phacility
amckinley added a comment to T13076: Plans: Phacility cluster caching, renaming, and rebalance/compaction.

In my mind, the only real problem here is...

Apr 9 2018, 6:54 PM · Plans, Ops, Infrastructure, Phacility