Skip to content

build: improve caching. #210

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

Merged
merged 6 commits into from
Jan 6, 2022
Merged

Conversation

PiotrSikora
Copy link
Member

@PiotrSikora PiotrSikora commented Jan 4, 2022

There is a lot of unnecessary data in Bazel's cache, so this commit
removes Bazel's repository cache (~700 MiB), install base (150 MiB)
and tarballs from the Rust toolchain (350 MiB) before saving cache.

This decreases the cache size for each OS+runtime combination from
1.8 GiB to 700 MiB, which means that we can save cache from 2 runs
with 7 combinations using GitHub's 10 GiB total storage.

Additionally, Wasmtime builds so fast that we can disable cache to
free those slots for future OS+runtime combinations.

While there, fix caching on macOS, which uses a different location.

Signed-off-by: Piotr Sikora [email protected]

There is a lot of unnecessary data in Bazel's cache, so this commit
removes Bazel's repository cache (~700 MiB), install base (150 MiB)
and tarballs from the Rust toolchain (350 MiB) before saving cache.

This decreases the cache size for each OS+runtime combination from
1.8 GiB to 600 MiB, which means that we can save cache from 2 runs
with 8 combinations using GitHub's 10 GiB total storage.

Additionally, Wasmtime builds so fast that we can disable cache to
free 2 slots for future OS+runtime combinations.

While there, fix caching on macOS, which uses a different location.

Signed-off-by: Piotr Sikora <[email protected]>
@PiotrSikora PiotrSikora requested a review from mathetake as a code owner January 4, 2022 10:44
@PiotrSikora PiotrSikora merged commit af1728e into proxy-wasm:master Jan 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants