TST: Use free Linux ARM64 runner#17645
Conversation
(in public beta now)
|
Thank you for your contribution to Astropy! 🌌 This checklist is meant to remind the package maintainers who will review this pull request of some common things to look for.
|
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
|
The permissions are actually right - the environment variables If you need a temporary workaround for now, this is what I'm using. It's based off of actions/partner-runner-images#25 (comment), but adapted to fix both of the affected variables. # Insert this at the beginning of the `steps` array:
- name: Fix environment for ARM64 Linux runners (actions/partner-runner-images#25)
run: for var in PATH XDG_CONFIG_HOME; do sed -Ee "s/^/${var}=/" -e 's/(runner)admin/\1/g' <<< "${!var}"; done | tee -a "$GITHUB_ENV"
|
|
This seems to have been fixed on GitHub's end now. The runner now gets the correct |
|
Thank you. It is green! 🎉 Took 5m ish vs 9m for the paid runner. 🧐 |
|
The other jobs were cancelled on purpose because they are irrelevant here. What do you think, @Cadair ? Merge? |
|
|
||
| # Native arm64 testing - we keep this in the weekly cron as we need to pay for it so we should | ||
| # minimize how many jobs we do. | ||
| # Native arm64 testing - we keep this in the weekly cron to avoid exceeding free quota. |
There was a problem hiding this comment.
I mean we have never exceeded the free quota on x86 jobs ;)
Seriously tho we could make one of the main jobs aarch64.
There was a problem hiding this comment.
I think moving it to CI proper would be much simpler if I wait for OpenAstronomy?
I can open follow-up issue here (#17656). Thanks!
There was a problem hiding this comment.
Re: quota -- since it is still in "public beta," I am playing safe for now...
|
But yes |
…645-on-v7.0.x Backport PR #17645 on branch v7.0.x (TST: Use free Linux ARM64 runner)
Description
This pull request is to switch from paid to free Linux ARM64 runner (in public beta now). Also see:
TODO