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

ci: run memory intensive check targets sequentially #3513

Merged
merged 1 commit into from
Dec 2, 2024

Conversation

daniel-weisse
Copy link
Member

Context

Our tidy/check/generate CI job fails intermittently during bazel run //:check due to memory constraints on our CI runners, when more than one job runs at the same time on our cluster.

Proposed change(s)

  • Run the two most memory intensive targets of check sequentially to avoid issues with single jobs taking up more than 20GB of RAM
    • govulncheck requires around 12GB of RAM on my local machine
    • golang-ci-lint requires 8-10GB on my local machine

Signed-off-by: Daniel Weiße <dw@edgeless.systems>
@daniel-weisse daniel-weisse added the no changelog Change won't be listed in release changelog label Dec 2, 2024
Copy link

netlify bot commented Dec 2, 2024

Deploy Preview for constellation-docs canceled.

Name Link
🔨 Latest commit d2edcd8
🔍 Latest deploy log https://app.netlify.com/sites/constellation-docs/deploys/674d69fcddf0610008033a27

@daniel-weisse daniel-weisse merged commit 5b1e362 into main Dec 2, 2024
7 checks passed
@daniel-weisse daniel-weisse deleted the dw/ci/sequential-check branch December 2, 2024 08:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no changelog Change won't be listed in release changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants