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: update rust cache action v1 -> v2 #2460

Merged
merged 1 commit into from
May 3, 2023
Merged

Conversation

conorsch
Copy link
Contributor

@conorsch conorsch commented May 3, 2023

We've gone over the 10GB per-repo limit for workflow caches on GH. GH will automatically evict caches not used for a week or longer, but we've had a bunch of active PRs with plenty of rebases, and went over, ~22GB/10GB. Manually cleaned up old caches in the GitHub UI [1] to unbreak workflow runs. This change brings us up to speed with the latest release for the associated GH action helper, which has some updated caching settings in the changelog. May need to follow up with refinements to ensure that we don't hit the cache limit again.

N.B. GH does not allow us to purchase a cache increase per repo, which is unfortunate.

[0] https://docs.github.com/en/actions/using-workflows/caching-dependencies-to-speed-up-workflows#usage-limits-and-eviction-policy
[1] https://github.com/penumbra-zone/penumbra/actions/caches

We've gone over the 10GB per-repo limit for workflow caches on GH.
GH will automatically evict caches not used for a week or longer, but
we've had a bunch of active PRs with plenty of rebases, and went over,
~22GB/10GB. Manually cleaned up old caches in the GitHub UI [1] to
unbreak workflow runs. This change brings us up to speed with the latest
release for the associated GH action helper, which has some updated
caching settings in the changelog. May need to follow up with
refinements to ensure that we don't hit the cache limit again.

N.B. GH does not allow us to purchase a cache increase per repo, which
is unfortunate.

[0] https://docs.github.com/en/actions/using-workflows/caching-dependencies-to-speed-up-workflows#usage-limits-and-eviction-policy
[1] https://github.com/penumbra-zone/penumbra/actions/caches
@conorsch
Copy link
Contributor Author

conorsch commented May 3, 2023

Smoke test is failing on swap integration test:

error: test failed, to rerun pass `-p pcli --test network_integration`
failures:

failures:
    swap

test result: FAILED. 6 passed; 1 failed; 0 ignored; 0 measured; 0 filtered out; finished in 164.19s

Rerunning...

@redshiftzero
Copy link
Member

Heads up the smoke test CI failure is unrelated to the changes in this PR and is occurring on main so I think it's safe to ignore here

@conorsch
Copy link
Contributor Author

conorsch commented May 3, 2023

Missed that it was on main already, tks!

@conorsch conorsch merged commit 7b81604 into main May 3, 2023
@conorsch conorsch deleted the fix-smoke-test-rust-cache branch May 3, 2023 20:49
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