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

Invalidate Linux build caches more agressively #8031

Merged
merged 1 commit into from
Feb 19, 2024
Merged

Conversation

SomeoneToIgnore
Copy link
Contributor

@SomeoneToIgnore SomeoneToIgnore commented Feb 19, 2024

We run Linux CI on regular GitHub Action runners, which have ~30GB of disk space. This is nothing for Rust builds and, due to Cargo.lock perturbations, we tend to accumulate enough artifacts to fill the disk entirely since restore-keys allowed to keep the cache for different lockfiles.

Instead, try to invalidate the cache more aggressively (which will cost us more frequent ~30min Linux CI runs) to see how this will work in comparison.

Release Notes:

  • N/A

We run Linux CI on regular GitHub Action runners, which have ~30GB of disk space.
This is nothing for Rust builds and, due to Cargo.lock perturbations, we tend to accumulate enough artifacts to fill the disk entirely since `restore-keys` alowed to keep the cache for different lockfiles.

Instead, try to invalidate the cache more aggressively (which will cost us more frequent ~30min Linux CI runs) to see how this will work in comparison.
@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Feb 19, 2024
@SomeoneToIgnore
Copy link
Contributor Author

Further options to consider:

@mikayla-maki mikayla-maki merged commit 37f6a70 into main Feb 19, 2024
7 checks passed
@mikayla-maki mikayla-maki deleted the kb/agressive-linux branch February 19, 2024 22:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed The user has signed the Contributor License Agreement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants