integration: increase timeout to mitigate TestBasicGhaCacheImportExport flakiness #5575
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.
This PR adds an increased timeout to the
TestBasicGhaCacheImportExport
integration test in an effort to mitigate flakiness highlighted in the linked issue. Instead of an"ExtraTimeout"
of just15 minutes (3 * maxTimeout where maxTimeout equals 5 minutes)
, this now goes as high as 30 minutes by updating the multiplier to6
. This should theoretically be more than enough time to allow the test to complete successfully if the flake is truly rate-limited/time based(i.e. "We encounter this one when a bunch of jobs are running on CI and therefore we got rate-limited in GHA cache backend with this test.")
.Any more time would likely require deeper investigation into figuring out how to more adequately handle the rate limiting issue.
Related to #5494