Skip to content

Conversation

crazy-max
Copy link
Member

continue carrying changes from #3204

@crazy-max crazy-max marked this pull request as ready for review August 14, 2025 10:15
@crazy-max crazy-max requested a review from tonistiigi August 14, 2025 10:15
@crazy-max crazy-max force-pushed the bake-images branch 2 times, most recently from 8993f77 to e76584a Compare August 14, 2025 14:40
name: Build
uses: docker/bake-action@v6
with:
source: ${{ github.server_url }}/${{ github.repository }}.git#${{ github.ref }}
Copy link
Member Author

@crazy-max crazy-max Aug 14, 2025

Choose a reason for hiding this comment

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

Can't use Git context from our actions-toolkit: https://github.com/docker/actions-toolkit/blob/633bcf1936ffff0e05a4bbe2b23c71abf6f4111e/src/context.ts#L45-L60

Because currently it fetches by sha and not git ref so tags are not pulled: https://github.com/crazy-max/buildkit/actions/runs/16967780276/job/48096258733#step:7:440. That would therefore set a commit sha as BuildKit version: https://github.com/crazy-max/buildkit/actions/runs/16967780276/job/48096258733#step:7:1654

docker/actions-toolkit#677 would fix it but it's probably better to wait for #5974.

So in the meantime set Git context manually with full git ref as we currently do in our hack script:

currentcontext="https://github.com/$GITHUB_REPOSITORY.git#$GITHUB_REF"

@crazy-max crazy-max force-pushed the bake-images branch 2 times, most recently from 2c7f587 to a20221c Compare September 9, 2025 09:06
@crazy-max crazy-max added this to the v0.25.0 milestone Sep 9, 2025
@tonistiigi tonistiigi removed this from the v0.25.0 milestone Sep 23, 2025
@crazy-max crazy-max added this to the v0.26.0 milestone Oct 8, 2025
@crazy-max crazy-max merged commit e1b4261 into moby:master Oct 13, 2025
139 checks passed
@crazy-max crazy-max deleted the bake-images branch October 13, 2025 16:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/ci area/hack building buildkit itself

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants