Skip to content

[K9VULN-4521] ci(ghcr): fix broken build by unpinning qemu #685

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

Merged
merged 2 commits into from
Apr 14, 2025

Conversation

amaanq
Copy link
Contributor

@amaanq amaanq commented Apr 14, 2025

What problem are you trying to solve?

Our GHCR workflow is failing for the latest release, see here

What is your solution?

Removing the pinning of the image works, which means we're using QEMU v9 for ARM emulation.

This PR reverts the following PRs that downgraded the runner and pinned QEMU to v7:

Verification

A successful run of this build can be observed in my fork here

Alternatives considered

What the reviewer should know

I bumped to Ubuntu latest because that, along with the unpinning, had succeeded in my fork. Ideally, we should be on the latest stable release of Ubuntu, but we had to downgrade to fix the segfaults in the past.

@amaanq amaanq requested a review from a team as a code owner April 14, 2025 14:33
@datadog-datadog-prod-us1
Copy link

Datadog Summary

✅ Code Quality    ✅ Code Security

@amaanq amaanq changed the title ci(ghcr): fix broken build by unpinning qemu [K9VULN-4521] ci(ghcr): fix broken build by unpinning qemu Apr 14, 2025
@amaanq amaanq requested a review from a team April 14, 2025 15:24
@jasonforal
Copy link
Collaborator

Please indicate in the description that this PR reverts:

@amaanq
Copy link
Contributor Author

amaanq commented Apr 14, 2025

Please indicate in the description that this PR reverts:

Done

@amaanq amaanq merged commit 1d74f1b into main Apr 14, 2025
75 checks passed
@amaanq amaanq deleted the amaan.qureshi/fix-ghcr branch April 14, 2025 17:24
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.

3 participants