Repository navigation
ci: raise the time limits of five jobs that run near them - #895
Open
LeeroyHannigan wants to merge 1 commit into
Open
LeeroyHannigan wants to merge 1 commit into
LeeroyHannigan wants to merge 1 commit into
Conversation
On main's last seven push runs, run-integration took up to 39.1 minutes, run-integration-sqlite up to 39.2 and mongodb-rust up to 38.7, against a 40-minute limit. cassandra-pytest took up to 41.6 minutes against 45. A run that restores no Rust cache builds for longer. The bulk put items pull request (#874) had the three 40-minute jobs cancelled at the limit. Every test that had run in them passed. These jobs now stop at 60 minutes. run-tests stops at 25 minutes instead of 15. On #874 it took 12 to 15 minutes. Its build took 5.5 minutes without a cache, where main's takes 2, and the engine's unit tests took 5 minutes on the 4-core runner. A job that hangs now holds its runner up to 20 minutes longer.
LeeroyHannigan
requested review from
amrith,
c33howard,
eliseong,
hitesh96db,
jcshepherd,
navareamol,
pdf-amzn and
robinnsc
as code owners
November 6, 2026 17:47
1 of 8 tasks
This branch has not been deployed
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.
What
This pull request raises the
timeout-minutesof five CI jobs. Nothing else changes.run-tests(both toolchains)test.ymlrun-integrationintegration.ymlrun-integration-sqliteintegration.ymlmongodb-rustintegration-mongodb.ymlcassandra-pytestintegration-cassandra.ymlWhy
No issue tracks this change. On main's last seven push runs, four of these jobs came within one to four minutes of their limits:
run-integrationrun-integration-sqlitemongodb-rustcassandra-pytestA pull request run that restores no Rust cache builds for longer. A pull request that adds tests to these jobs also takes them over the limit. On #874 (bulk put items), the run on
4a825f6fhadrun-integration,run-integration-sqliteandmongodb-rustcancelled at 40 minutes. Every test that had run in them passed.run-teststook 12 to 15 minutes on #874. Its build took 5.5 minutes without a cache, where main's took 2. The engine's unit tests then took 5 minutes on the 4-core runner. One run hit the 15-minute limit, and its re-run passed in 12m38s.The cost is that a job that hangs holds its runner up to 20 minutes longer, and
run-testsup to 10 minutes longer.Testing done
actionlintreports the same findings for the four workflow files before and after the change. The findings are the custom runner labels that it does not know.yaml.safe_loadreads each file.Checklist
cargo test --workspace)cargo fmt --check)cargo clippy -- -W clippy::pedantic)Storagetrait, auth model, on-diskformat, or public CLI surface, an RFC has been accepted or is linked
below. Otherwise, an ADR captures the decision (link below).
ADR / RFC: n/a
By submitting this pull request, I confirm that my contribution is made under
the terms of the Apache License 2.0 and I agree to the Developer Certificate of
Origin (DCO). See CONTRIBUTING.md for details.