Skip to content

Commit 2e796dc

Browse files
committed
ci: mark the job-cap exhibit as mechanism, not verdict
Run 30724385719 ran under the retired 26/30 pair and would have passed under #131's 40:00. It demonstrates that two steps sharing one step_timeout lets the job cap fire behind a green step; it is not itself evidence that 40:00 is too tight. That case rests on the arithmetic, which is arithmetic -- no job has been observed hitting 40:00.
1 parent a6aabff commit 2e796dc

1 file changed

Lines changed: 4 additions & 0 deletions

File tree

.github/workflows/ci.yml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -307,6 +307,10 @@ jobs:
307307
# does NOT happen when a step is killed (that ends the job and skips what follows, so those
308308
# durations never sum); it happens when the first step PASSES near its budget.
309309
#
310+
# READ THAT EXHIBIT FOR ITS MECHANISM, NOT AS A VERDICT ON THE CAP IT REPLACED: it ran under the
311+
# retired 26/30 pair and would have passed under #131's 40:00. What condemns 40:00 is the
312+
# arithmetic below -- which is arithmetic, not an observed kill; no job has yet hit 40:00.
313+
#
310314
# job_timeout is therefore sized to hold BOTH gated steps plus setup -- not step_timeout plus a
311315
# constant, which is what the previous +4 relationship was. Each addend below is that addend's
312316
# MEASURED MAXIMUM over the same pool. An earlier revision used 0:41 for setup, which is the

0 commit comments

Comments
 (0)