ci: run the front's tests in one job rather than three shards #511
No reviewers
Labels
No labels
bug
documentation
duplicate
enhancement
good first issue
help wanted
invalid
question
spec
wayfinder:grilling
wayfinder:map
wayfinder:prototype
wayfinder:research
wayfinder:task
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
amaurycoudr/chess-project!511
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "ci/front-test-one-job"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Summary
The three legs ran on the same 4 CPUs, so sharding bought no parallelism. Each leg still paid its own container start, cache restore and
pnpm install, plus about 10 s rebuilding the front's dependencies. The legs' Turbo hashes differ, so no leg reused another's builds. Two fewer jobs also free two runner slots per pipeline. Decision logged ininfra/gitlab/DECISIONS.md, "CI speed-up".Evidence
After: pipeline 14 (this MR), job 146: 180 s of runner time instead of 263 s (−32%), one slot instead of three. Vitest took 106 s for all 54 files (
54 passed), against 146 s summed over the three legs; setup took 38 s, slowed by another job running beside it.Trade-off: the one job (180 s) now runs as long as
e2e(167 s) and thebuild-workspace→build-image→reviewchain (about 170 s), so it no longer finishes well ahead of them. To recheck oncee2eruns on 2 workers (!512): iffront-testis then the only long job, two legs is a one-line change.Merge Danger
Door: two-way
parallel: 3comes back in one line.Blast Radius: front-test
The one job runs the whole suite. It now has to finish within the 25-minute timeout; GitHub ran it in under 2 minutes.
added 4 commits
mainCompare with previous version
changed the description
added 2 commits
9bd3ffd6- 1 commit from branchmainCompare with previous version
added 2 commits
a84b494b- 1 commit from branchmain4f61cf6b- ci: run the front's tests in one job rather than three shardsCompare with previous version