-
Notifications
You must be signed in to change notification settings - Fork 71
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
Github-hosted runners for OSX no longer carry a bundled conda installation #344
Comments
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
I'm a bit lost here
Is this change live? The blog says it'll happen at some point in the next 12 weeks from April 1, but no updates there. This page indicates it's already happened to |
Yes, I'm observing it in some repos already. See the linked CI run above. |
Must have happened overnight? Wonder if there'll be an announcement today then |
I'm not seeing this change live: https://github.com/mattwthompson/gha-sandbox/actions/runs/8803326872/job/24161162575#step:3:5 |
Yep, you got macos-12 there (check first step). This will take 12 weeks, and we are currently starting the 4th-ish. It might hit you latee, but it will 😁 |
Thanks for that - I totally forgot to see what that label actually pulls down |
… found at !" (#801) * Test activate base to fix failure * Try without miniconda * Try manually using conda to install * Try forcing macos runner to 13 instead of latest see: conda-incubator/setup-miniconda#344 and https://github.com/conda/constructor/pull/777/files * Update run_tests.yaml
Looking forward, is it likely there will be a time when only MacOS/ARM-based runners are available? If so, should there be a way to specify that one wants to use a x86_64 miniconda even if running on ARM? I can imagine that I will want to build for x86_64 for several more years. It seems to me that tying the CPU architecture to a specific runner is going to be a problem down the road. |
There's an |
Hi @jaimergp - good catch! And we've literally just noticed this by upgrading to v4, see ESMValGroup/ESMValCore#2411 (comment) - the problem is that in the future there will be fewer and fewer nodes that will run v3 (GHA already toss warnings that one should update the action to latest version, since such and such node are the only ones able to execute the older version). BTW using |
I found out that forcing the Intel architecture as: osx:
runs-on: "macos-latest"
strategy:
matrix:
python-version: ["3.9", "3.10", "3.11"]
architecture: ["x64"]
fail-fast: false
name: OSX Python ${{ matrix.python-version }}
steps:
- uses: actions/checkout@v4
with:
fetch-depth: 0
- uses: conda-incubator/setup-miniconda@v3
with:
architecture: ${{ matrix.architecture }}
activate-environment: esmvalcore
environment-file: environment.yml
python-version: ${{ matrix.python-version }}
miniforge-version: "latest"
miniforge-variant: Mambaforge
use-mamba: true works biutifully even with the ARM64 processor and v4, well at least for now, before they diverge completely 🥳 |
* Remove log file redirect * Add missing quotation * Add miniconda version * Try v3 * Add latest * try miniforge? * fix architecture conda-incubator/setup-miniconda#344 * Pin macos-13 * Remove latest * ok try macos-12...
See https://github.com/conda/conda-standalone/actions/runs/8796238715/job/24143815312?pr=67.
macos-latest
runners will now default to macOS 14, which means they run on ARM64 Apple Silicon, not Intel! This breaks the assumption ofmacos-latest == x86_64
. Just a heads up!We should document this.
The current recommendation is to pin directly to
macos-13
to ensure you get an Intel runner. Make sure you are usingsetup-miniconda@v3
.The text was updated successfully, but these errors were encountered: