Skip to content

train 1113173002: #2963 #2095 #2966 (CI only) - #3254

Closed
ashhart wants to merge 22 commits into
mainfrom
lead/train-1113173002
Closed

ashhart wants to merge 22 commits into
mainfrom
lead/train-1113173002

Conversation

@ashhart

@ashhart ashhart commented Nov 13, 2026

Copy link
Copy Markdown
Owner

CI for a landing train: #2963 #2095 #2966 stacked on main in that order. Nothing merges from this PR: each PR above lands on its own once this tip is green, and this PR is then closed.

ashhart and others added 22 commits November 13, 2026 17:06
…iles the page has

dashboard_test.zig was wired only into the macOS test step, so no CI job ran it. Since #2655's landing it has
expected a "Paged prefix cache" label that dashboard.html never had and nothing in /stats reports, so it failed
on every Mac running zig build test. The test now runs from build.zig's shared test step on every OS, and that
one string is gone; every other label it checks is on the page.
T0's group backend could not abort GLM's communicator before the
RoCE group was installed, because it only borrowed it. GLM now hands
the backend the group that owns the communicator from Engine.init,
so an abort always reaches it once, and the start-up unwind uses the
group's guarded destroy, which skips the destroy after an abort.
A failed spawn returned while the ranks already started still wrote to
the test's frame, the same pattern #2700 fixed in host.runWith. Both tests
now join the ranks they started on every return; in the gather test that
also covers rank 0's own steps between the spawns and the join. Every wait
in Bootstrap has a deadline, so the join is bounded.
Each app run uploads its ~230 MB AppImage, and with GitHub's default 90-day retention 2,069 copies (475 GB) had piled up
in Actions storage by 12 Nov, enough to hit the account's Actions limit. The build is smoke-tested in the job itself;
the upload is only for a quick download, which 3 days covers.
@ashhart

ashhart commented Nov 13, 2026

Copy link
Copy Markdown
Owner Author

Stopped: the train this one was stacked on did not land whole; its PRs go back to the queue.

@ashhart ashhart closed this Nov 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants