ci: parallelize isolated test shards and separate lint - #57
Merged
Merged
Conversation
Owner
Author
|
Validation completed on 55756a3: run 34269949546 passed all 9 jobs in 5m42s, compared with baseline 34264761652 at 13m28s (57.7% less wall time in this sample). Logs confirm full=true on all six shards and all 13 suites executed exactly once per OS. Both legacy-named aggregate checks passed. Local inert-fixture regression tests also passed. This is a single-run comparison; runner variation contributes (Ubuntu MOTD 325s vs 384s baseline). No assertions or production scripts changed; main and branch protection remain untouched. Ready for review; not merged. |
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
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.
Summary
Evidence and validation
Baseline main run 34264761652: 13m28s overall; Ubuntu tests 713s, including MOTD 384s and push suites 289s; ShellCheck 83s. Debian tests 616s, including MOTD 331s and push suites 251s. Dependency installation took 8s, so caching is not the primary opportunity. Sharding should bring the critical path near the slowest suite plus runner overhead, not guarantee 4–6 minutes.
Local inert-fixture tests passed: test-shards.sh, test-selection.sh, test-selection-git.sh, test-lint-coverage.sh; Bash syntax and git diff --check passed. Production suites are intentionally validated only on isolated CI runners, without local credentials. This workflow change forces full PR coverage. More concurrent jobs and modest setup overhead are expected.
Rollback
Revert this commit to restore the previous two-job workflow and serial test runner. No branch protection or production configuration changes.