-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
ci: update buildkit to latest rc #2669
Conversation
288e6ab
to
159e58f
Compare
@tonistiigi This one is odd: https://github.com/moby/buildkit/runs/5299617535?check_suite_focus=true#step:6:1003
Could be linked to latest QEMU? (#2634) |
@@ -27,7 +27,7 @@ FROM --platform=$BUILDPLATFORM golang:1.17-alpine AS golatest | |||
# gobuild is base stage for compiling go/cgo | |||
FROM golatest AS gobuild-base | |||
RUN apk add --no-cache file bash clang lld pkgconfig git make | |||
COPY --from=xx / / | |||
COPY --link --from=xx / / |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For the problem w/ cross, is this line wrong?
Lines 42 to 43 in 9bc409d
pop, md := MarshalConstraints(constraints, &m.constraints) | |
pop.Platform = nil // merge op is not platform specific |
I may have misunderstood how platform constraints work.
159e58f
to
d155e19
Compare
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
d155e19
to
8b1a1a0
Compare
Updated frontend to master to get the fix from #2672 |
Let's switch to the latest test release to make sure CI doesn't catch anything.
Signed-off-by: Tonis Tiigi tonistiigi@gmail.com