Skip to content

Build multi-platform eden-sdn image #1082

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 1 commit into from
Jul 18, 2025

Conversation

milan-zededa
Copy link
Contributor

Refactored the sdn/Makefile to support building eden-sdn images for multiple platforms using Docker Buildx and LinuxKit. Each target architecture now produces a distinct sdn-<arch>-bios.img and is handled via dedicated Makefile targets (e.g., build-vm-img-linux-arm64).

Dockerfile.vm has been updated to use the correct image based on the target architecture at build time using the standard TARGETARCH argument.

With this patch it is now supported to specify multiple platforms, like we do in the publish workflow:

make DOCKER_TARGET=push LINUXKIT_TARGET=push DOCKER_PLATFORM=linux/arm64,linux/amd64 build-docker

(my previous PR broke the multi-platform building/publishing: https://github.com/lf-edge/eden/actions/runs/16353910243/job/46207393649)

Refactored the sdn Makefile to support building eden-sdn images for multiple
platforms using Docker Buildx and LinuxKit. Each target architecture
now produces a distinct sdn-<arch>-bios.img and is handled via dedicated
Makefile targets (e.g., build-vm-img-linux-arm64).

Dockerfile.vm has been updated to use the correct image based on the
target architecture at build time using the standard TARGETARCH argument.

Signed-off-by: Milan Lenco <[email protected]>
@milan-zededa
Copy link
Contributor Author

Merging to check if publish is fixed...

@milan-zededa milan-zededa merged commit c45832d into lf-edge:master Jul 18, 2025
41 of 45 checks passed
@milan-zededa
Copy link
Contributor Author

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.

2 participants