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

Switch to using native arm runner #561

Merged
merged 1 commit into from
Feb 10, 2025

Conversation

nosracd
Copy link
Contributor

@nosracd nosracd commented Feb 9, 2025

Currently, the aarch64 wheel builds are failing due to a qemu bug:

Relevant issue on cibuildwheel project discussing the bug: pypa/cibuildwheel#2257

This PR bypasses the issue by switching to the native ARM GitHub runner. The advantages of this approach are:

Faster build times. For example, the Python 3.7 jobs that used qemu took:

But the new, equivalent jobs take:

Also, this approach should result in a simpler build chain. By avoiding qemu we also avoid emulation errors, like the one currently affecting jobs.

The downside of this approach is that the ubuntu-42.04-arm runners are currently is public preview so there may be stability issues when using this image. If we'd rather not go this approach then the alternative workaround would probably be to pin qemu to a version without the bug.

@nosracd nosracd requested a review from ihilt February 9, 2025 15:39
@nosracd nosracd self-assigned this Feb 9, 2025
@nosracd nosracd mentioned this pull request Feb 9, 2025
@nosracd nosracd merged commit ed363bc into lcm-proj:master Feb 10, 2025
54 checks passed
@nosracd nosracd deleted the fix/arm64_ci_builds branch February 10, 2025 13:48
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