The gate is one suite's clock, not a grouping problem - #24
Merged
Merged
Conversation
factory's Tests job is 129s and its `bats` step is 118s of it, which made it look like haus's shell half before that was cut four ways. It is not the same shape, and the step list's own need boundary — the snug checkout only `agent-surface` and `presentation` read — is the split that proves it: the painter half is 9s, under the ~11s a Linux job costs, and the half left behind is still a 121s job. Eight seconds for a second runner request. Timed per file from the per-line timestamps `gh api .../jobs/<id>/logs` carries, five PR runs: factory-watchdog is 88s of the 118, and 55s of that 88 is wall clock on purpose — 36s in the two suspend cases, which stub `sleep` so awake-seconds and wall-seconds can be told apart, and 19s in seven waits for an absence, which cannot poll. A Mac 2.8x slower on every other suite runs those two suspend cases in 35s against the runner's 36, which is the tell. No split, then; the arithmetic goes in the job's banner instead, so the next reader inherits the measurement rather than the impression. Also checked what haus turned up: nothing here is held open past its last `ok` — summed per-case timestamps come to 118.1s against the step's own 117.8s. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
factory's Tests job is 129s and its
batsstep is 118s of it, which made itlook like haus's shell half before that was cut four ways. It is not the same
shape, and the step list's own need boundary — the snug checkout only
agent-surfaceandpresentationread — is the split that proves it: thepainter half is 9s, under the ~11s a Linux job costs, and the half left behind
is still a 121s job. Eight seconds for a second runner request.
Timed per file from the per-line timestamps
gh api .../jobs/<id>/logscarries, five PR runs: factory-watchdog is 88s of the 118, and 55s of that 88
is wall clock on purpose — 36s in the two suspend cases, which stub
sleepsoawake-seconds and wall-seconds can be told apart, and 19s in seven waits for an
absence, which cannot poll. A Mac 2.8x slower on every other suite runs those
two suspend cases in 35s against the runner's 36, which is the tell.
No split, then; the arithmetic goes in the job's banner instead, so the next
reader inherits the measurement rather than the impression. Also checked what
haus turned up: nothing here is held open past its last
ok— summed per-casetimestamps come to 118.1s against the step's own 117.8s.
Co-Authored-By: Claude Opus 5 (1M context) noreply@anthropic.com