[POC] Multi-platform in dagger europa#1467
[POC] Multi-platform in dagger europa#1467TomChv wants to merge 6 commits intodagger:mainfrom TomChv:poc/push-multi-platform
Conversation
|
✔️ Deploy Preview for devel-docs-dagger-io ready! 🔨 Explore the source changes: 055669adf7867eded823353edf6bc2e611dd4fc3 🔍 Inspect the deploy log: https://app.netlify.com/sites/devel-docs-dagger-io/deploys/61f2c4a974231a000863c4fe 😎 Browse the preview: https://deploy-preview-1467--devel-docs-dagger-io.netlify.app |
|
@aluzzardi Actually the PR will always fail because our buildkit image do not support Here is logs # 10:32PM INF actions."test-linux/arm64".printArch | computing
646
# 10:32PM INF actions."test-linux/amd64".printArch | completed duration=100ms
647
# 10:32PM INF actions."test-linux/arm64".printArch | #7 0.113 standard_init_linux.go:228: exec user process caused: exec format error
648
# 10:32PM INF actions."test-linux/amd64".test | computing
649
# 10:32PM INF actions."test-linux/amd64".test | completed duration=0s
650
# 10:32PM ERR actions."test-linux/arm64".printArch | failed: process "/bin/sh -c uname -m >> /arch.txt" did not complete successfully: exit code: 1 duration=200ms
651
# 10:32PM ERR actions."test-linux/s390x".image | canceled duration=300ms
652
# 10:32PM FTL system | failed to up environment: task failed: actions."test-linux/arm64".printArch: process "/bin/sh -c uname -m >> /arch.txt" did not complete successfully: exit code: 1We get the troubleshooting explained : here @dubo-dubon-duponey Already has a working Dockerfile but it's huge so I'll try to create an simpler image |
Signed-off-by: Vasek - Tom C <[email protected]>
… the inputs Signed-off-by: Vasek - Tom C <[email protected]>
Signed-off-by: Vasek - Tom C <[email protected]>
Signed-off-by: Vasek - Tom C <[email protected]>
Signed-off-by: Vasek - Tom C <[email protected]>
|
CI fixed thanks QEMU action (more details here) |
Signed-off-by: Vasek - Tom C <[email protected]>
|
This PR is stale because it has been open 14 days with no activity. Remove stale label or comment or this will be closed in 7 days. |
|
This PR was closed because it has been stalled for 7 days with no activity. |
2 participants
Changes
Related to #1371, this PR is a working PoC of multi-platform in Europa that we can use to choose a final design
This PR improve
#Pushto push multi-platform image using newinputsfield.It will push all image one by one and then return the digest of the manifest list. The user can then use it to pull a specific platform using the new
platformfield in#Pull.Indeed, I've also update
#Dockerfileto supports only one platform instead of an array.@shykes @aluzzardi IMO we should maybe create a specific definition to push multi-platform image, this way we can split the code and improve the maintainability.
List of improvement
Push
#Pushdefinition to supports multipleinputsImageinterface that store every data required to push an imageExportsmethod to push multi-platformDockerfile
#Dockerfileto supports only one platform instead of an arrayPull
#Pullwith a fieldplatformto specify a specific platform to pull for multi-platform digestTests
Should we test failure case?