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

GitHub test.yml: add ubuntu-24.04-arm to matrix #221

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

msgilligan
Copy link

GitHub has enabled arm64 runners for Open Source projects:
https://github.blog/changelog/2025-01-16-linux-arm64-hosted-runners-now-available-for-free-in-public-repositories-public-preview/

This is a naive attempt to turn it on for this project.

@sandydoo
Copy link
Member

Neat! Looks like someone at GitHub made a typo in XDG_CONFIG_HOME 👀 I'll give them some time to roll out a new image and we'll try again.

@msgilligan
Copy link
Author

Is there an open issue upstream for this?

@sandydoo
Copy link
Member

Yeah, there's a couple with the same problem across multiple different actions.

Probably the correct one to upvote: actions/partner-runner-images#27

@msgilligan
Copy link
Author

Issue #222 is how I discovered the problem.

@msgilligan
Copy link
Author

The CI failure for this issue is currently:

https://github.com/cachix/install-nix-action/actions/runs/12817083173/job/35739671809?pr=221

Run cachix use cachix
  cachix use cachix
  shell: /usr/bin/bash -e {0}
  env:
    nixpkgs_channel: nixpkgs=channel:nixos-24.11
    oldest_supported_installer: nix-2.8.0
    NIX_PATH: nixpkgs=channel:nixos-24.11
    TMPDIR: /home/runner/work/_temp
cachix: /home/runneradmin/.config/nix: createDirectory: permission denied (Permission denied)
Error: Process completed with exit code 1.

@msgilligan
Copy link
Author

I added a second commit with the suggested workaround from the upstream issue:

7e7bccb

@sandydoo
Copy link
Member

There's no need to add a workaround. GitHub needs to update the image.

@msgilligan msgilligan marked this pull request as draft January 17, 2025 04:58
@msgilligan
Copy link
Author

There's no need to add a workaround. GitHub needs to update the image.

Well, it would be nice to verify that the job runs to completion with the workaround. Then we can remove it before merging. I marked the PR as draft for now...

@msgilligan msgilligan force-pushed the msgilligan/github-test-aarch64-linux branch from 7e7bccb to 265a04a Compare January 18, 2025 01:03
@msgilligan msgilligan marked this pull request as ready for review January 18, 2025 01:04
@msgilligan
Copy link
Author

@sandydoo Upstream is fixed. I removed the workaround and force-pushed. I also marked as ready-for-review. 🤞

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