Page MenuHomePhabricator
Feed Advanced Search

May 4 2022

epriestley triaged T13677: Drydock may grow resource pools too cautiously as Normal priority.
May 4 2022, 9:37 PM · Drydock
epriestley added a comment to T13675: Can Arcanist plausibly be compiled into a binary?.

What options exist for inter-process communication?

May 4 2022, 6:27 PM · Arcanist
epriestley added a comment to T13675: Can Arcanist plausibly be compiled into a binary?.

Here's how I'm thinking about overengineering this:

May 4 2022, 5:17 PM · Arcanist

May 3 2022

epriestley closed T11336: Provide a stable address range for Phacility outbound requests as Resolved.

This was resolved as part of T13630. I didn't actually put it in the UI anywhere, but all requests will now originate from 13.56.71.101, and this isn't likely to change given the wind-down of Phacility hosting.

May 3 2022, 11:29 PM · Customer Impact
epriestley added a project to T10522: Build plan page doesn't warn about unapproved Drydock blueprints: Drydock.
May 3 2022, 11:25 PM · Drydock, Restricted Project, Harbormaster (v3)
epriestley added a comment to T10508: Drydock WorkingCopy blueprints allow selection of invalid "Use Blueprints".

D21801, motivated by T13676, made this more practical.

May 3 2022, 11:19 PM · Drydock (v3)
epriestley added a revision to T10508: Drydock WorkingCopy blueprints allow selection of invalid "Use Blueprints": Unknown Object (Differential Revision).
May 3 2022, 11:18 PM · Drydock (v3)
epriestley closed T11694: Allow clients to generally reason about Drydock leases over the API as Resolved.

This is somewhat resolved and neither next steps or motivation are clear any longer, so I'm going to call it done until evidence to the contrary arises.

May 3 2022, 11:17 PM · Restricted Project, Drydock
epriestley closed T11694: Allow clients to generally reason about Drydock leases over the API, a subtask of T11693: Make drydock command interfaces accessible via SSH workflows, as Resolved.
May 3 2022, 11:17 PM · Restricted Project, Drydock
epriestley closed T13676: Drydock may reclaim recently-used resources as Resolved.

Perhaps a philosophical question here is: do we care about which repositories are checked out in a working copy resource?

May 3 2022, 11:15 PM · Drydock
epriestley added a comment to T13676: Drydock may reclaim recently-used resources.

Before, instant reclaim after lease destruction:

May 3 2022, 10:56 PM · Drydock
epriestley added a revision to T13676: Drydock may reclaim recently-used resources: Unknown Object (Differential Revision).
May 3 2022, 10:54 PM · Drydock
epriestley added a comment to T13676: Drydock may reclaim recently-used resources.

To create resource pressure, I'm now going to try this -- I guess I don't really need the --count flag, but it does make the terminal juggling slightly easier:

May 3 2022, 10:51 PM · Drydock
epriestley added a comment to T13676: Drydock may reclaim recently-used resources.

The blueprint thing was on the way toward creating allocation pressure, so D21802 allows you to select a blueprint (or a set of possible blueprints) with --blueprint. You can specify an ID or PHID:

May 3 2022, 10:07 PM · Drydock
epriestley added a revision to T13676: Drydock may reclaim recently-used resources: Unknown Object (Differential Revision).
May 3 2022, 10:03 PM · Drydock
epriestley added a revision to T13676: Drydock may reclaim recently-used resources: Unknown Object (Differential Revision).
May 3 2022, 9:28 PM · Drydock
epriestley added a revision to T13676: Drydock may reclaim recently-used resources: Unknown Object (Differential Revision).
May 3 2022, 9:10 PM · Drydock
epriestley added a revision to T13676: Drydock may reclaim recently-used resources: Unknown Object (Differential Revision).
May 3 2022, 9:09 PM · Drydock
epriestley added a comment to T13676: Drydock may reclaim recently-used resources.

That patch is reasonable, and shouldn't break anything as long as the list you provide is a subset of the possible list.

May 3 2022, 8:41 PM · Drydock
epriestley added a revision to T13588: PHP 8 Compatibility: Unknown Object (Differential Revision).
May 3 2022, 7:00 PM · Infrastructure
epriestley added a revision to T13676: Drydock may reclaim recently-used resources: Unknown Object (Differential Revision).
May 3 2022, 7:00 PM · Drydock
epriestley added a revision to T13676: Drydock may reclaim recently-used resources: Unknown Object (Differential Revision).
May 3 2022, 6:57 PM · Drydock
epriestley added a comment to T13676: Drydock may reclaim recently-used resources.

After D21796:

May 3 2022, 6:54 PM · Drydock
epriestley added a comment to T13676: Drydock may reclaim recently-used resources.

(one orthogonal bug I found is that bin/drydock lease discards any blueprints provided in an attributes JSON)

May 3 2022, 6:46 PM · Drydock
epriestley added a revision to T13676: Drydock may reclaim recently-used resources: Unknown Object (Differential Revision).
May 3 2022, 6:36 PM · Drydock
epriestley added a comment to T13676: Drydock may reclaim recently-used resources.

Grab a test lease on the host with:

May 3 2022, 6:31 PM · Drydock
epriestley closed T12103: No active Drydock blueprint exists which can ever allocate a resource for lease, when there is one available as Resolved.

Harbormaster needs a "Working Copy" blueprint, not just a "Host" blueprint.

May 3 2022, 6:18 PM · Bug Report
epriestley added a revision to T13676: Drydock may reclaim recently-used resources: Unknown Object (Differential Revision).
May 3 2022, 6:10 PM · Drydock
epriestley added a revision to T13588: PHP 8 Compatibility: Unknown Object (Differential Revision).
May 3 2022, 6:10 PM · Infrastructure
epriestley added a comment to T13676: Drydock may reclaim recently-used resources.

Here's a fairly simple way to reproduce this:

May 3 2022, 5:55 PM · Drydock
epriestley triaged T13676: Drydock may reclaim recently-used resources as Normal priority.
May 3 2022, 5:35 PM · Drydock

May 2 2022

epriestley added a comment to T13675: Can Arcanist plausibly be compiled into a binary?.

How do modern servers written in C/C++ handle parallelizing requests?

May 2 2022, 11:06 PM · Arcanist
epriestley added a comment to T13675: Can Arcanist plausibly be compiled into a binary?.

Just for my own notes:

May 2 2022, 8:14 PM · Arcanist

Apr 29 2022

epriestley added a comment to T13675: Can Arcanist plausibly be compiled into a binary?.

What are the practical limits of multi-system or multi-architecture binaries?

Apr 29 2022, 8:32 PM · Arcanist
mormegil awarded T13675: Can Arcanist plausibly be compiled into a binary? a Love token.
Apr 29 2022, 8:26 AM · Arcanist

Apr 28 2022

epriestley added a comment to T13675: Can Arcanist plausibly be compiled into a binary?.

I generated D21794 with a native binary that has no dependency on system PHP (but does depend on system cURL).

Apr 28 2022, 10:41 PM · Arcanist
epriestley added a revision to T13675: Can Arcanist plausibly be compiled into a binary?: Unknown Object (Differential Revision).
Apr 28 2022, 10:25 PM · Arcanist
epriestley added a revision to T13556: Setting "phd.trace" may cause TTY testing with "posix_isatty()" against closed pipes: Unknown Object (Differential Revision).
Apr 28 2022, 10:25 PM · Daemons
epriestley added a comment to T13556: Setting "phd.trace" may cause TTY testing with "posix_isatty()" against closed pipes.

Another open question is: before we call posix_isatty(STDOUT), how do we test whether STDOUT is a valid handle?

Apr 28 2022, 10:23 PM · Daemons
epriestley added a comment to T13675: Can Arcanist plausibly be compiled into a binary?.
  1. PHP doesn't link cURL statically and doesn't appear (?) to provide any easy way to link it statically.
      • Possible solution: build static cURL.
      • Possible solution: link cURL dynamically.
      • Possible solution: replace cURL with mbedTLS + a first-party HTTP client, since we don't care about 99% of what cURL does. Or implement first-party TLS hahaha except I'm half-serious? If HTTP/HTTPS is going to happen from an embedded context on my coffee maker I need a healthy embeddable TLS + HTTP stack anyway.
    1. The STDERR and STDOUT constants are defined by the CLI SAPI, and not present in the embed SAPI.
      • Possible solution: define them in the C wrapper.
      • Possible solution: polyfill them at startup in PHP (I'm not entirely sure this is possible).
      • Possible solution: abstract around them and use php://stderr and php://stdout instead. See also T13556. This is likely desirable anyway.
    2. When arc tries to load PHP code, it needs to read data out of the executable binary in some set of conditions (e.g., "if we miss on disk").
      • Possible solution: hook zend_compile_file(), which seems to be the expected way to approach this. The flow in phar_compile_file() seems similar.
    3. To run unit tests, arc depends on the presence of a php binary on the system.
      • Possible solution: accept that you must also have PHP to run arc's tests (and that system PHP may differ from arc php).
      • Possible solution: also include the PHP CLI wrapper and invoke it when arc is executed as php, providing what is essentially a fallback toolset.
      • Possible solution: provide a "unit test helper" fallback toolset.
      • Possible solution: rewrite the 5 trivial cases where we need this (support/unit/*) in shell script (but: the reason to do these in PHP in the first place was so that they're portable to Windows).
Apr 28 2022, 9:44 PM · Arcanist

Apr 27 2022

epriestley added a comment to T13675: Can Arcanist plausibly be compiled into a binary?.

Can we build a single binary with a bunch of data in it (e.g., a picture of a cat) without breaking anything?

Apr 27 2022, 4:51 PM · Arcanist

Apr 26 2022

cburroughs awarded T13675: Can Arcanist plausibly be compiled into a binary? a Y So Serious token.
Apr 26 2022, 12:50 AM · Arcanist

Apr 25 2022

epriestley closed T5619: Duplicate entry 'PHID-PHRF-n6jez7lb5eyyll77k7re-338' for key 'key_version'' in Phragment as Wontfix.

See T5479.

Apr 25 2022, 11:50 PM · Phragment
epriestley closed T4684: Asynchronously calculate or pre-calculate patches for Phragment files as Wontfix.

See T5479.

Apr 25 2022, 11:50 PM · Phragment
epriestley closed T5065: Allow Phragment to serve up repositories in e.g. NuGet and Maven formats, a subtask of T5479: Unbeta Phragment, as Wontfix.
Apr 25 2022, 11:50 PM · Phragment
epriestley closed T5065: Allow Phragment to serve up repositories in e.g. NuGet and Maven formats as Wontfix.

See T5479.

Apr 25 2022, 11:50 PM · Phragment
epriestley closed T5479: Unbeta Phragment as Resolved.

I removed Phragment in D21793.

Apr 25 2022, 11:50 PM · Phragment
epriestley added a comment to T13658: How to rebrand Phabricator.

My initial linter implementation didn't actually catch all of these so there were a few more than I expected, but I think I got essentially all of them.

Apr 25 2022, 11:48 PM
epriestley added a revision to T5479: Unbeta Phragment: Unknown Object (Differential Revision).
Apr 25 2022, 11:45 PM · Phragment
epriestley added a revision to T13658: How to rebrand Phabricator: Unknown Object (Differential Revision).
Apr 25 2022, 11:45 PM
epriestley added a revision to T9530: Release Server / Workflow app / Future of Releeph : Unknown Object (Differential Revision).
Apr 25 2022, 11:40 PM · Restricted Project, Harbormaster
epriestley added a revision to T13658: How to rebrand Phabricator: Unknown Object (Differential Revision).
Apr 25 2022, 11:40 PM
epriestley added a revision to T13658: How to rebrand Phabricator: Unknown Object (Differential Revision).
Apr 25 2022, 11:30 PM
epriestley added a revision to T13658: How to rebrand Phabricator: Unknown Object (Differential Revision).
Apr 25 2022, 11:26 PM
epriestley added a revision to T13658: How to rebrand Phabricator: Unknown Object (Differential Revision).
Apr 25 2022, 11:24 PM
epriestley added a revision to T13658: How to rebrand Phabricator: Unknown Object (Differential Revision).
Apr 25 2022, 11:19 PM
epriestley added a revision to T13658: How to rebrand Phabricator: Unknown Object (Differential Revision).
Apr 25 2022, 11:16 PM
epriestley added a revision to T13658: How to rebrand Phabricator: Unknown Object (Differential Revision).
Apr 25 2022, 11:12 PM
epriestley added a revision to T13658: How to rebrand Phabricator: Unknown Object (Differential Revision).
Apr 25 2022, 11:08 PM
epriestley added a revision to T13658: How to rebrand Phabricator: Unknown Object (Differential Revision).
Apr 25 2022, 11:05 PM
epriestley added a revision to T13658: How to rebrand Phabricator: Unknown Object (Differential Revision).
Apr 25 2022, 9:50 PM
epriestley added a revision to T13658: How to rebrand Phabricator: Unknown Object (Differential Revision).
Apr 25 2022, 9:44 PM
epriestley added a revision to T13658: How to rebrand Phabricator: Unknown Object (Differential Revision).
Apr 25 2022, 9:39 PM
epriestley added a revision to T13658: How to rebrand Phabricator: Unknown Object (Differential Revision).
Apr 25 2022, 9:34 PM
epriestley added a revision to T13658: How to rebrand Phabricator: Unknown Object (Differential Revision).
Apr 25 2022, 9:28 PM
epriestley added a revision to T13658: How to rebrand Phabricator: Unknown Object (Differential Revision).
Apr 25 2022, 9:21 PM
epriestley added a revision to T13658: How to rebrand Phabricator: Unknown Object (Differential Revision).
Apr 25 2022, 9:14 PM
epriestley added a revision to T13658: How to rebrand Phabricator: Unknown Object (Differential Revision).
Apr 25 2022, 8:04 PM
epriestley added a revision to T13588: PHP 8 Compatibility: Unknown Object (Differential Revision).
Apr 25 2022, 7:28 PM · Infrastructure
epriestley added a revision to T13658: How to rebrand Phabricator: Unknown Object (Differential Revision).
Apr 25 2022, 7:19 PM
epriestley added a revision to T13658: How to rebrand Phabricator: Unknown Object (Differential Revision).
Apr 25 2022, 7:16 PM
epriestley added a revision to T13658: How to rebrand Phabricator: Unknown Object (Differential Revision).
Apr 25 2022, 7:09 PM
epriestley added a revision to T13658: How to rebrand Phabricator: Unknown Object (Differential Revision).
Apr 25 2022, 7:03 PM
epriestley added a revision to T13658: How to rebrand Phabricator: Unknown Object (Differential Revision).
Apr 25 2022, 6:37 PM
epriestley added a revision to T13658: How to rebrand Phabricator: Unknown Object (Differential Revision).
Apr 25 2022, 6:24 PM
epriestley added a revision to T13658: How to rebrand Phabricator: Unknown Object (Differential Revision).
Apr 25 2022, 6:19 PM
epriestley added a comment to T13658: How to rebrand Phabricator.

Some general guidelines I'm following here:

Apr 25 2022, 5:56 PM
epriestley added a revision to T13658: How to rebrand Phabricator: Unknown Object (Differential Revision).
Apr 25 2022, 5:46 PM
epriestley added a revision to T13658: How to rebrand Phabricator: Unknown Object (Differential Revision).
Apr 25 2022, 5:30 PM
epriestley added a revision to T13658: How to rebrand Phabricator: Unknown Object (Differential Revision).
Apr 25 2022, 5:18 PM
epriestley added a comment to T13658: How to rebrand Phabricator.

I don't want to get embroiled in this too much, but:

Apr 25 2022, 5:04 PM
epriestley added a revision to T13658: How to rebrand Phabricator: Unknown Object (Differential Revision).
Apr 25 2022, 4:44 PM
epriestley added a revision to T13658: How to rebrand Phabricator: Unknown Object (Differential Revision).
Apr 25 2022, 4:41 PM

Apr 23 2022

epriestley added a comment to T13675: Can Arcanist plausibly be compiled into a binary?.

Is the purpose of implementing introspection of libraries to allow for building native components that would get picked up and auto-loaded by arcanist?

Apr 23 2022, 10:08 PM · Arcanist

Apr 22 2022

cspeckmim awarded T13675: Can Arcanist plausibly be compiled into a binary? a Manufacturing Defect? token.
Apr 22 2022, 2:08 AM · Arcanist

Apr 21 2022

epriestley added a comment to T13675: Can Arcanist plausibly be compiled into a binary?.

How can PHP be built statically? How hard is this?

Apr 21 2022, 11:01 PM · Arcanist
epriestley added a comment to T13675: Can Arcanist plausibly be compiled into a binary?.
$ cat random.c
int get_random_number() {
  return 5;
}
$ gcc -c -o random.o random.c
$ gcc -shared -o random.so random.o
Apr 21 2022, 10:23 PM · Arcanist
epriestley triaged T13675: Can Arcanist plausibly be compiled into a binary? as Normal priority.
Apr 21 2022, 8:24 PM · Arcanist

Apr 20 2022

epriestley lowered the priority of T11132: New Phabricator NUX from High to Wishlist.
Apr 20 2022, 10:43 PM · Design, Phacility, NUX
epriestley lowered the priority of T11456: Don't lose user in NUX flow because of Timezone issues from High to Wishlist.
Apr 20 2022, 10:43 PM · Design, Phacility, NUX
epriestley closed T10847: 30GB Phacility instance caused a series of cascading failures which left web services unreachable as Resolved.

There's nothing particularly useful or actionable here now, so closing it out. (I believe this was the most severe incident Phacility ever experienced while actively maintained.)

Apr 20 2022, 10:43 PM · Ops, Phacility
epriestley closed T12610: Audit behavior of LB healthchecks against *.phacility.com and secure.phabricator.com as Wontfix.

This hasn't caused any more problems in like 4 years, so I guess it's kind of whatever.

Apr 20 2022, 10:30 PM · Ops, Phacility
epriestley closed T12674: Update developer docs to include rCORGI setup as Wontfix.

This is moot given the Phacility wind-down.

Apr 20 2022, 10:26 PM
epriestley closed T12727: Mail handler unable to parse e-mail from MS Outlook 12, Failed to JSON encode value as Wontfix.

This is almost certainly "PhabricatorMetaMTAReceivedMail incorrectly assumes email is valid UTF8", and the solution begins by making all the associated storage binary, but it's hard to imagine that ever happening now.

Apr 20 2022, 10:25 PM · Bug Report
epriestley closed T12886: Cannot upload file from Firefox (Android) as Wontfix.

Difficult to imagine this ever getting fixed in the upstream since I don't have any Android devices.

Apr 20 2022, 10:21 PM · Android, Firefox, Bug Report
epriestley closed T12984: Support issues don't have hovercards as Wontfix.

This only impacts the "Support" application in Phacility and it's hard to imagine building a Doorkeeper bridge for it now.

Apr 20 2022, 10:18 PM · Support
epriestley closed T13402: Owner audit flag got flipped in update? as Wontfix.

Discourse got nuked, so just throwing this away until another report somehow surfaces.

Apr 20 2022, 10:14 PM · Audit
epriestley closed T13493: JIRA API has changed identifiers from "key" to "accountId" as Resolved.

This hasn't cropped up again and is presumably resolved.

Apr 20 2022, 9:18 PM · Auth
epriestley closed T13537: Support local port forwarding through Phacility cluster bastion hosts as Wontfix.

This isn't really resolved, but almost certainly does not make sense to pursue given the Phacility wind-down.

Apr 20 2022, 9:09 PM · Phacility
epriestley closed T13555: In "ExecFuture", "proc_get_status()" may be passed NULL, possibly leading to a daemon stall as Resolved.

I believe the above changes resolved this, and it hasn't cropped up again.

Apr 20 2022, 9:03 PM · Daemons