Skip to content

release: pass GOARCH build-arg to arm64 docker builds#839

Merged
javuto merged 2 commits into
jmpsec:mainfrom
alvarofraguas:pr/fix-goreleaser-arm64-docker
May 19, 2026
Merged

release: pass GOARCH build-arg to arm64 docker builds#839
javuto merged 2 commits into
jmpsec:mainfrom
alvarofraguas:pr/fix-goreleaser-arm64-docker

Conversation

@alvarofraguas
Copy link
Copy Markdown
Collaborator

Summary

  • The four Dockerfiles in deploy/cicd/docker/ declare ARG GOARCH=amd64 as the default
  • The arm64 goreleaser blocks pass --platform=linux/arm64 but never override the GOARCH build arg
  • The COPY osctrl-${COMPONENT}-${GOOS}-${GOARCH} instruction therefore looks for the amd64 binary inside an arm64 build context and fails with "/osctrl-admin-linux-amd64": not found

This was masked by the manifest-list export error fixed in #828. After that merge, the main CI run (26082810103) got past the manifest step but hit this binary name mismatch on the arm64 docker images.

Fix: add --build-arg=GOARCH=arm64 to each of the four arm64 docker blocks in .goreleaser.yml.

Test plan

  • goreleaser check passes
  • CI main workflow passes after merge (arm64 docker images should now find the correct binary)

uname -i returns x86_64 on Intel but both yq and Go release
binaries use amd64 in their filenames, causing 404 downloads.

Closes jmpsec#836
The Dockerfiles use ARG GOARCH=amd64 as default. The arm64 goreleaser
blocks set --platform=linux/arm64 but never override the GOARCH arg,
so the COPY instruction looks for the amd64 binary in an arm64 build
context and fails with "not found".

Fixes the CI failure surfaced after jmpsec#828 landed — the manifest-list
fix let the build proceed far enough to hit this second bug.
@javuto javuto added 🚧 bugfix Fix for an existing bug cicd CICD related issues labels May 19, 2026
@javuto javuto merged commit 1aadde3 into jmpsec:main May 19, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🚧 bugfix Fix for an existing bug cicd CICD related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants