Allow slow CI runners to finish runtime and CLI tests - #1137
Merged
Merged
Conversation
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.
A runtime integration shard was canceled at the 75-minute CI limit after 59 passing tests, with no assertion failures. Identical coverage passed on other runners, but the slow runner took roughly three times as long; its remaining tests project a total of about 86 minutes. The completed local CLI suite also took 76.3 minutes before CI setup time.
Give all four runtime integration shards and the CLI lane a bounded 120-minute budget, matching the existing runtime support lane. Preserve every test, shard, selection rule, validation command, and failure gate. Package and benchmark-history/typecheck lane budgets stay at 45 minutes.
Validation: nine lane-policy tests (70 assertions), frozen dependency installation, strict lint, formatting, and independent review passed. The policy tests cover the budgets and their use in the workflow matrix. Full CI on the final combined main commit will verify completion across every lane.
The PR appears safe to merge; the intended lanes receive the larger bounded budget without weakening coverage or changing unrelated lane budgets.
Summary
This PR increases the bounded timeout for the four sharded runtime integration lanes and the CLI lane from 75 to 120 minutes while preserving the existing lane selection and failure gates.
Reviews (1) · Last reviewed commit: "Allow complete integration and CLI check..."