Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

mitigate CI timeouts by increasing parallelism #10837

Merged
merged 1 commit into from
May 16, 2024

Conversation

alexrashed
Copy link
Member

Motivation

The run time of our pipeline increased dramatically somewhat around the time of our latest bump of moto (with #10742).
This is causing some timeouts in the pipeline due to the hard limit of 60 minutes for a job in CircleCI.
While we are still investigating the issue, this PR tries to unblock the pipeline by adding another parallel runner in the meantime.

Changes

  • Adds another runner to the CircleCI integration tests for until we identified the issue.

/cc @viren-nadkarni

@alexrashed alexrashed added the semver: patch Non-breaking changes which can be included in patch releases label May 16, 2024
@alexrashed alexrashed added this to the 3.5 milestone May 16, 2024
@alexrashed alexrashed self-assigned this May 16, 2024
Copy link

LocalStack Community integration with Pro

    2 files  ±0      2 suites  ±0   1h 35m 50s ⏱️ +48s
2 950 tests ±0  2 647 ✅ ±0  303 💤 ±0  0 ❌ ±0 
2 952 runs  ±0  2 647 ✅ ±0  305 💤 ±0  0 ❌ ±0 

Results for commit 6ce2f7b. ± Comparison against base commit b59f520.

@alexrashed alexrashed requested a review from silv-io May 16, 2024 11:42
Copy link
Member

@silv-io silv-io left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Running independent tests should be embarrassingly parallel so increasing the runners is a good shout :)

Copy link
Member

@dominikschubert dominikschubert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, good call unblocking us for now

@alexrashed alexrashed merged commit 780b6cb into master May 16, 2024
35 checks passed
@alexrashed alexrashed deleted the fix-timeouts-increase-gs branch May 16, 2024 11:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver: patch Non-breaking changes which can be included in patch releases
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants