Skip to content

Conversation

@cclerget
Copy link
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings December 16, 2025 20:49
@cclerget cclerget merged commit e58ac6f into master Dec 16, 2025
3 checks passed
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR enables multi-architecture Docker image builds by adding QEMU emulation support to the GitHub Actions workflow. This allows the workflow to build images for both AMD64 and ARM64 architectures.

  • Adds QEMU setup step using docker/setup-qemu-action@v3 to enable cross-platform builds
  • Updates the build command to target both AMD64 and ARM64 Linux platforms

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.


- name: Build/Push to GitHub Container Registry
run:
make push-multiarch PULL_BASE_REF=$(git describe --tags --abbrev=0) REGISTRY_NAME=ghcr.io/ctrliq BUILD_PLATFORMS="linux amd64 amd64 -amd64; linux arm64 arm64 -arm64"
Copy link

Copilot AI Dec 16, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The BUILD_PLATFORMS parameter format appears incorrect. Based on the build.make file (line 162), the format expects space-separated fields per platform entry: "os arch buildx_platform suffix base_image addon_image".

The current format has duplicated "amd64 amd64" and "arm64 arm64" which doesn't match the expected pattern. According to the example in build.make (line 135), it should be "linux amd64" (os and arch) or "linux amd64 amd64 -amd64" when specifying buildx_platform and suffix.

Consider using: BUILD_PLATFORMS="linux amd64; linux arm64"

Suggested change
make push-multiarch PULL_BASE_REF=$(git describe --tags --abbrev=0) REGISTRY_NAME=ghcr.io/ctrliq BUILD_PLATFORMS="linux amd64 amd64 -amd64; linux arm64 arm64 -arm64"
make push-multiarch PULL_BASE_REF=$(git describe --tags --abbrev=0) REGISTRY_NAME=ghcr.io/ctrliq BUILD_PLATFORMS="linux amd64; linux arm64"

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants