-
Notifications
You must be signed in to change notification settings - Fork 7
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
Use native ARM runners when testing on ARM architecture #214
Conversation
afd5051
to
b10d648
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ARMazing! 🚀
Can you please note somewhere in this PR why the Fedora 40 and 41 tests are failing?
Note that the Fedora 40 and 41 checks are currently failing because of fedora-cloud/docker-brew-fedora#117. |
@cisagov/vm-dev - Do you think I should remove the QEMU and buildx jazz from this workflow? We're not using it, but it isn't doing any harm. |
If it's not causing any issues, I'm fine leaving it in. |
These failures are fixed by #216. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems reasonable enough 🔥
This provides better and faster testing of ARM architectures than testing under QEMU emulation.
We are no longer emulating ARM under QEMU, so using this GitHub Action no longer causes the runners to run out of resources and crash.
b10d648
to
7a3c694
Compare
🗣 Description
This pull request:
build.yml
workflow to use native ARM runners when testing on ARM architecture💭 Motivation and context
Resolves #211.
🧪 Testing
All automated tests pass. I also verified that the expected platforms are indeed tested via native ARM runners.
✅ Pre-approval checklist