fix(ci): serialize local tests — 4Gi dind runner OOMs under parallel browser suites#229
Closed
Jesssullivan wants to merge 1 commit into
Closed
fix(ci): serialize local tests — 4Gi dind runner OOMs under parallel browser suites#229Jesssullivan wants to merge 1 commit into
Jesssullivan wants to merge 1 commit into
Conversation
Owner
Author
|
Superseded by #228. The unique test:ci-cached --local_test_jobs=1 safeguard from this PR was folded into #228 at signed commit a58489c, together with regression coverage. #228 retains the already-proven Tectonic warmup, bounded local build parallelism, and shared-cache contract, so we have one canonical CI-hardening lane instead of two overlapping branches. |
Owner
Author
|
Closing as superseded by #228; no unique change was discarded. |
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
1 participant
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.
Post-#227, the gates lane executes the browser test suites locally. The jesssullivan-blog-dind runner container is limited to 4Gi memory; parallel local tests (vitest + puppeteer chromium + node suites concurrently, then playwright) intermittently OOM the pod → silent runner death after ~15m (job fails with zero failed steps). Observed on every run of the 4-PR stampede AND a solo rerun; #227's own green run fit by luck.
test:ci-cached --local_test_jobs=1caps peak memory at one browser at a time. Executor-backed lanes are unaffected (local_test_jobs governs local actions only).Durable follow-up (operator/GF-side): raise the runner-set memory limit for
jesssullivan-blog-dindin the GF overlay (currently limits.memory=4Gi) — tracked in the wave report, TIN-2677 family.