Skip to content

Commit daeeb32

Browse files
committed
ci(test): raise Test Core shard timeout-minutes 30 -> 45, temporarily
Shard 5/6 keeps hitting the 30-minute wall, so every tail observation is a reading of the wall rather than of the shard: the true duration is censored and unbounded above. Raising the backstop both unblocks the job and un-censors the durations the shard rebalance needs before it can be derived. The new value carries its revert condition inline, next to the value. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01T6HeZvT9wdSJD1ZxJb5Eno
1 parent bf45e62 commit daeeb32

1 file changed

Lines changed: 22 additions & 1 deletion

File tree

.github/workflows/ci.yml

Lines changed: 22 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -292,7 +292,28 @@ jobs:
292292
# ~5× a normal sharded run (~4-6 min), with margin for a cold Turbo cache;
293293
# the old 45 left a hung job "running" for half an hour past any plausible
294294
# healthy finish.
295-
timeout-minutes: 30
295+
#
296+
# ⚠ TEMPORARY RAISE, 30 -> 45 (#16173). Shard 5/6 is being killed at the
297+
# 30-minute wall — 12+ observations at 30:16-30:21 with every other job in
298+
# every one of those runs green — so the paragraph above describes the
299+
# value this line must RETURN to, not the value it currently carries.
300+
#
301+
# Two reasons, both #16173's:
302+
# 1. Unblock. The only uncensored shard-5 readings are 25:32 and 27:36,
303+
# and shard 6/6 has been seen at 28:56 — a ~1 minute margin at the top
304+
# of the observed green band against a 30:00 wall.
305+
# 2. Un-censor. Every tail observation is a reading of THIS number, not
306+
# of the shard: the true duration is >= 30:16, unbounded above. So the
307+
# shard timings cannot be re-derived from CI history while the wall
308+
# stands here. Raising it produces the durations #16173 needs before
309+
# anyone touches `scripts/test-shard-timings.json`.
310+
#
311+
# ⛔ REVERT CONDITION, explicit: back to `30` once #16173 lands its shard
312+
# rebalance. This value is temporary and carries no other expiry — a raise
313+
# with no revert condition written beside it becomes permanent by
314+
# forgetting. The backstop stays loose only for that window; the stall
315+
# guard named above remains the primary hang detector throughout.
316+
timeout-minutes: 45
296317
permissions:
297318
contents: read
298319
strategy:

0 commit comments

Comments
 (0)