Skip to content

Switch from mambaforge to miniforge in CI#11409

Merged
jrbourbeau merged 5 commits into
dask:mainfrom
jrbourbeau:pin-mamba
Oct 1, 2024
Merged

Switch from mambaforge to miniforge in CI#11409
jrbourbeau merged 5 commits into
dask:mainfrom
jrbourbeau:pin-mamba

Conversation

@jrbourbeau

Copy link
Copy Markdown
Member

We're seeing CI jobs fail during setup with

Warning: ERROR: executing pre_install.sh failed

Here's a similar upstream issue conda-incubator/setup-miniconda#366 where they suggest pinning to mamba<2 (mamba=2 just came out).

cc @jhamman

@jrbourbeau jrbourbeau mentioned this pull request Oct 1, 2024
3 tasks

@jrbourbeau jrbourbeau left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay, CI is running again. I'll plan to merge this after CI finishes. Adding some additional context in comments below.

- name: Setup Conda Environment
uses: conda-incubator/[email protected]
with:
miniforge-variant: Mambaforge

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removing this was also needed, but switching to miniforge instead of mambaforge is fine as they're now identical and mambaforge is actually being sunsetted https://conda-forge.org/news/2024/07/29/sunsetting-mambaforge/

Comment thread .github/workflows/additional.yml Outdated
miniforge-variant: Mambaforge
miniforge-version: latest
use-mamba: true
mamba-version: "1.5.10"

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can undo this pin once conda-incubator/setup-miniconda#366 is resolved

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually, it turns out this pin wasn't needed. We just needed to switch to miniforge. I've removed these pins.

@jrbourbeau jrbourbeau changed the title Pin mamba in CI Switch from mambaforge to miniforge in CI Oct 1, 2024

@hendrikmakait hendrikmakait left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, @jrbourbeau!

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