Repository navigation
chore(ci): use a 600s system-tests scheduling budget - #6447
Conversation
|
✅ All CI checks and tests passed. 🎉 All green!🧪 All tests passed 🎯 Code Coverage (details) 🔗 Commit SHA: c9d842b | Docs | View more details | Give us feedback! |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
More details
The 600-second scheduling target preserves the expected numeric input, and the reusable workflow and checkout pins match. Independent verification of the linked CI runs was blocked by GitHub HTTP 403.
🤖 Bits Code Review · Commit c9d842b · @DataDog review to ask questions
What does this PR do?
Raises the Ruby system-tests scheduling target from 300 to 600 seconds.
Pins system-tests to
52cecf6f2c74f3d41e57e4352cf41daf0f7a51b5, which refreshes the Ruby p75 timing estimates used to balance end-to-end shards.Motivation:
The 300-second target is shorter than several Ruby weblog build estimates and leaves too little execution budget for their scenarios.
The refreshed estimates and 600-second target reduce the workflow's long tail:
The remaining shards over the target finished within 35 seconds of it, indicating substantially better scheduling balance.
Change log entry
None. This is an internal CI scheduling change.
Additional Notes:
The pinned system-tests commit is merged into
DataDog/system-tests'smainbranch, as confirmed by the workflow guard.How to test the change?
The updated run completed successfully.