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

fix: use qemu 8.1.5 instead of latest #535

Merged
merged 1 commit into from
Feb 5, 2025

Conversation

holyspectral
Copy link
Contributor

@holyspectral holyspectral commented Jan 31, 2025

We noticed recently that our release build would fail when building using emulated arm64 environment. When it happens, it can fail in zypper, gcc or other commands. For example:

#63 127.6 # github.com/mattn/go-sqlite3
#63 127.6 gcc: internal compiler error: Segmentation fault signal terminated program cc1
#63 127.6 Please submit a full bug report, with preprocessed source (by using -freport-bug).
#63 127.6 See <https://bugs.opensuse.org/> for instructions.

And https://github.com/neuvector/scanner/actions/runs/13068955842/attempts/1

After invesitation, this is because tonistiigi/binfmt:latest is not up-to-date, and it might get accidentally broken when GitHub migrate to Ubuntu 24.04 runners.

This PR fixes the issue by specifying qemu-v8.1.5 to be used in publish-image action.

More discussions and affected software can be found in this upstream issue: tonistiigi/binfmt#215

tonistiigi/binfmt:latest is not up-to-date and broke when used with
Ubuntu 24.04 runners.

This commit specifies qemu-v8.1.5 to be used in publish-image action.
@holyspectral
Copy link
Contributor Author

cc: @pjbgf

@pjbgf
Copy link
Member

pjbgf commented Feb 1, 2025

cc: @tashima42 @briandowns

@tashima42
Copy link
Contributor

We might have to eventually mirror this image to our registry to prevent docker hub rate limits, as we have experienced before.

@holyspectral
Copy link
Contributor Author

Thanks everyone! I don't have the permission to merge this PR. Is there anything that we have to do before this can be merged?

@briandowns briandowns merged commit 7384e1a into rancher:master Feb 5, 2025
1 check passed
@holyspectral holyspectral deleted the use-qemu-8-1-5 branch February 6, 2025 21:27
@holyspectral
Copy link
Contributor Author

The last two runs after the fix work like a charm - no crash or anything. Thanks!

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.

4 participants