Repository navigation
Commit c30d5c9
ci: wait long enough for a Cloudflare preview, and say what a red one means
PR #163 went red on `Cloudflare Pages` and its preview smoke. Re-pushing the
same commit turned it green, so it was not the code -- but "retry fixed it"
locates nothing. What does: `E2E (Cloudflare Pages semantics)` was green on all
five shards, and that job runs the same bin/build.sh Cloudflare runs. Build
script fine on CI's Ubuntu, failure only on Cloudflare's own machine.
Why this link flakes at all: bin/build.sh takes about seven seconds; everything
else is haulage. The vendored editor lives in git, so every preview clones 616
MiB, checks out 625 MB and uploads 697 MB of output. That check took 36 minutes
to conclude. The distribution is already up against Cloudflare's build ceiling,
so crossing it is a matter of the day's luck.
The flake also exposed something deterministic: the gate waited `seq 1 75` x 20s
= 25 minutes, against a preview measured at 36. So even a Cloudflare deployment
that goes on to SUCCEED gets reported red if it is merely slow -- and the loop
count and the "25 minutes" in the message were two numbers kept in step by hand.
Now: 45 minutes, with ATTEMPTS derived from a single WAIT_MINUTES that the
message also quotes; the job's 60-minute timeout still leaves the smoke room.
Poll lines carry status= (absent/queued/in_progress) so a queue is readable as a
queue. And the failure message says how to tell the two cases apart -- green
Pages-semantics checks mean the build is fine -- and that a Git-integration
deployment has no retry API, so pushing again is the way. None of that was
written down anywhere; I had to infer it.
Pinned in workflow-contract: the window must exceed the measured 36 minutes, the
job must outlast it by 10, ATTEMPTS must be derived, and the message must carry
both hints. Reverse-verified: 25 minutes turns it red, dropping the hints turns
it red.
The real fix is getting 625 MB of vendor out of git, which every build path pays
for. That is its own project with its own risk, and is written up rather than
attempted here.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>1 parent c541ffd commit c30d5c9
3 files changed
Lines changed: 155 additions & 4 deletions
File tree
- .github/workflows
- docs/explorations
- test/unit
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
34 | 34 | | |
35 | 35 | | |
36 | 36 | | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
37 | 48 | | |
38 | 49 | | |
39 | 50 | | |
40 | 51 | | |
41 | 52 | | |
| 53 | + | |
42 | 54 | | |
43 | | - | |
| 55 | + | |
| 56 | + | |
44 | 57 | | |
45 | 58 | | |
46 | 59 | | |
47 | 60 | | |
48 | 61 | | |
49 | 62 | | |
50 | | - | |
| 63 | + | |
| 64 | + | |
| 65 | + | |
| 66 | + | |
| 67 | + | |
| 68 | + | |
| 69 | + | |
| 70 | + | |
51 | 71 | | |
52 | 72 | | |
53 | 73 | | |
54 | 74 | | |
55 | 75 | | |
56 | 76 | | |
57 | 77 | | |
58 | | - | |
| 78 | + | |
59 | 79 | | |
60 | | - | |
| 80 | + | |
61 | 81 | | |
62 | 82 | | |
63 | 83 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
| 55 | + | |
| 56 | + | |
| 57 | + | |
| 58 | + | |
| 59 | + | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
| 63 | + | |
| 64 | + | |
| 65 | + | |
| 66 | + | |
| 67 | + | |
| 68 | + | |
| 69 | + | |
| 70 | + | |
| 71 | + | |
| 72 | + | |
| 73 | + | |
| 74 | + | |
| 75 | + | |
| 76 | + | |
| 77 | + | |
| 78 | + | |
| 79 | + | |
| 80 | + | |
| 81 | + | |
| 82 | + | |
| 83 | + | |
| 84 | + | |
| 85 | + | |
| 86 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
292 | 292 | | |
293 | 293 | | |
294 | 294 | | |
| 295 | + | |
| 296 | + | |
| 297 | + | |
| 298 | + | |
| 299 | + | |
| 300 | + | |
| 301 | + | |
| 302 | + | |
| 303 | + | |
| 304 | + | |
| 305 | + | |
| 306 | + | |
| 307 | + | |
| 308 | + | |
| 309 | + | |
| 310 | + | |
| 311 | + | |
| 312 | + | |
| 313 | + | |
| 314 | + | |
| 315 | + | |
| 316 | + | |
| 317 | + | |
| 318 | + | |
| 319 | + | |
| 320 | + | |
| 321 | + | |
| 322 | + | |
| 323 | + | |
| 324 | + | |
| 325 | + | |
| 326 | + | |
| 327 | + | |
| 328 | + | |
| 329 | + | |
| 330 | + | |
| 331 | + | |
| 332 | + | |
| 333 | + | |
| 334 | + | |
| 335 | + | |
| 336 | + | |
| 337 | + | |
| 338 | + | |
| 339 | + | |
295 | 340 | | |
296 | 341 | | |
297 | 342 | | |
| |||
0 commit comments