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

Enable caching in conda-forge workflow #717

Open
glatterf42 opened this issue Jun 6, 2023 · 0 comments
Open

Enable caching in conda-forge workflow #717

glatterf42 opened this issue Jun 6, 2023 · 0 comments
Assignees
Labels
ci Continuous integration

Comments

@glatterf42
Copy link
Member

During the work on #715, I discovered that our conda-forge CI tests intend to use the cache action, but never do so successfully. The whole issue is described in detail in actions/cache/issues/1177.

After finding a way to fix the conda-forge CI tests, I reset the branch to throw out all unwanted commits, including all attempts of fixing the cache path. This also prevents scope creep.

One further source of scope creep might be this: while looking into mamba as a replacement for conda, I found that micromamba in particular comes with a setup action that has caching integrated. Pending @khaeru's opinion, we might switch to mamba/micromamba entirely rather than just using the mamba solver in conda. In that case, the paths here don't need fixing; instead, we can replace the whole cache action with the one integrated in the setup-micromamba action.

If we do want to keep conda and need to fix the paths, it is worth noting that our own setup-conda action provides an output called cache-path, which might in theory allow us to automatically use the correct path for caching our conda installation. In practice, we currently run the cache action before setup-conda, so we don't receive the path information in time. Currently, that output is not used anywhere in the workflow, so it might not work out of the box.

@glatterf42 glatterf42 added the ci Continuous integration label Jun 6, 2023
@glatterf42 glatterf42 self-assigned this Jun 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Continuous integration
Projects
None yet
Development

No branches or pull requests

1 participant