feat(docker): Add multi-platform support for Docker build#541
feat(docker): Add multi-platform support for Docker build#541
Conversation
PR SummaryLow Risk Overview Reviewed by Cursor Bugbot for commit 09dfbca. Bugbot is set up for automated code reviews on this repo. Configure here. |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 09dfbca. Configure here.
| id: publish | ||
| with: | ||
| context: ${{ inputs.context }} | ||
| platforms: linux/amd64,linux/arm64 |
There was a problem hiding this comment.
Multi-platform builds produce empty imageid output
Low Severity
Adding platforms: linux/amd64,linux/arm64 causes docker/build-push-action to produce an empty imageid output, since multi-platform builds create a manifest list rather than a single image. The composite action still declares imageid as an output and displays it in the build summary, both of which will now always be empty strings. Any downstream consumer relying on this output will silently receive no value.
Additional Locations (2)
Reviewed by Cursor Bugbot for commit 09dfbca. Configure here.
| id: publish | ||
| with: | ||
| context: ${{ inputs.context }} | ||
| platforms: linux/amd64,linux/arm64 |
There was a problem hiding this comment.
Missing QEMU setup breaks arm64 platform build
High Severity
Adding platforms: linux/amd64,linux/arm64 requires QEMU emulation for arm64 on x86_64 GitHub Actions runners, but docker/setup-qemu-action is not present before docker/setup-buildx-action. The Dockerfile's final stage (FROM debian:bookworm-slim) runs apt-get under the target platform, which will fail for arm64 without QEMU's binfmt_misc handlers registered. The build stages avoid this by using --platform=$BUILDPLATFORM, but the runtime stage does not.
Reviewed by Cursor Bugbot for commit 09dfbca. Configure here.


No description provided.