Skip to content

ci: pin buildx to v0.18.0#48964

Closed
crazy-max wants to merge 1 commit intomoby:masterfrom
crazy-max:ci-pin-buildx
Closed

ci: pin buildx to v0.18.0#48964
crazy-max wants to merge 1 commit intomoby:masterfrom
crazy-max:ci-pin-buildx

Conversation

@crazy-max
Copy link
Member

Relates to https://github.com/moby/moby/actions/runs/12046370143/job/33590391297?pr=48960#step:8:201

image

Seems to be a bug in latest Buildx 0.19.0 related to bake composable attributes. Let's pin to previous stable to unblock the pipeline

@crazy-max crazy-max marked this pull request as draft November 27, 2024 10:04
DOCKER_GRAPHDRIVER: ${{ inputs.storage == 'snapshotter' && 'overlayfs' || 'overlay2' }}
TEST_INTEGRATION_USE_SNAPSHOTTER: ${{ inputs.storage == 'snapshotter' && '1' || '' }}
SETUP_BUILDX_VERSION: latest
SETUP_BUILDX_VERSION: v0.18.0
Copy link
Member

Choose a reason for hiding this comment

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

Not for this PR, but now wondering if we should set a global version, and also override the one we use in the Dockerfile?

ARG BUILDX_VERSION=0.18.0

Although that would mean that we would not get early warnings on failures in rc's; is there a value other than latest that also includes betas and rc's?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes having some kind of latest-unstable would be good to catch this early

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