-
Notifications
You must be signed in to change notification settings - Fork 1
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: Add cache for Micromamba installation #57
Conversation
7581173
to
da17dd2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for figuring out a way to reduce failure rate.
This way we might never get updates from mamba.
Should we maybe use the cache only if the install fails?
Or run the cache first, then install second even if the cache hit so we pull changes?
da17dd2
to
15d7723
Compare
This endpoint sometimes fails to resolve, so caching as a fallback should help.
15d7723
to
6aead8f
Compare
- name: Save Micromamba to cache | ||
uses: actions/cache/save@v4 | ||
with: | ||
path: ~/.local/bin/micromamba | ||
key: ${{ runner.os }}-micromamba |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This needs some more work - it didn't save.
Failed to save: Unable to reserve cache with key Linux-micromamba, another job may be creating this cache. More details: Cache already exists. Scope: refs/pull/57/merge, Key: Linux-micromamba, Version: 53670150c936a15bf347fc758f8d4ffb00bfaed7e1251039dfd6945875b46746
Warning: Cache save failed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh no, that's a shame :(
Caching this way seems error prone to me, so I'd personally avoid it. If resolving |
Description of proposed changes
This endpoint sometimes fails to resolve, so caching as a fallback should help.
Related issue(s)
Checklist