Skip to content
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.

Sign up for GitHub

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

Jump to bottom

default to git context #193

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

default to git context #193

wants to merge 2 commits into from
+103 −53

Conversation

crazy-max
Copy link
Member

follow-up #181

@crazy-max crazy-max force-pushed the git-context-optout branch from b8e03e3 to 3a10d09 Compare March 14, 2024 10:08
@crazy-max crazy-max requested a review from tonistiigi March 14, 2024 10:30
@crazy-max crazy-max marked this pull request as ready for review March 14, 2024 10:30
@crazy-max crazy-max mentioned this pull request Mar 26, 2024
Merged
@crazy-max
Copy link
Member Author

crazy-max commented Apr 2, 2024 •
edited
Loading

I put this one back in draft while we wait for auth support before defaulting: docker/buildx#2363

@crazy-max crazy-max marked this pull request as draft April 2, 2024 17:16
@crazy-max crazy-max force-pushed the git-context-optout branch 2 times, most recently from 9447836 to f9b39de Compare June 18, 2024 10:08
@crazy-max crazy-max marked this pull request as ready for review June 18, 2024 10:10
tonistiigi
tonistiigi reviewed Jul 1, 2024
View reviewed changes
README.md Show resolved Hide resolved
@crazy-max crazy-max force-pushed the git-context-optout branch 2 times, most recently from e9a067d to 6b5d6f3 Compare July 3, 2024 18:05
@crazy-max crazy-max requested a review from tonistiigi July 3, 2024 18:06
@crazy-max crazy-max force-pushed the git-context-optout branch 2 times, most recently from f2d34fa to 1dbd507 Compare August 22, 2024 15:53
@crazy-max crazy-max mentioned this pull request Sep 20, 2024
Merged
@crazy-max
default to git context
f944cfa 
Signed-off-by: CrazyMax <1951866+crazy-max@users.noreply.github.com>
@crazy-max
chore: update generated content
98ab4bc 
Signed-off-by: CrazyMax <1951866+crazy-max@users.noreply.github.com>
@crazy-max
Copy link
Member Author

https://github.com/docker/bake-action/actions/runs/12300037568/job/34336154468?pr=193#step:5:255 is not related

/usr/bin/docker buildx bake --allow fs=* --set *.output=/tmp/bake-build --set *.cache-from=type=gha,scope=attests-binary --set *.cache-to=type=gha,scope=attests-binary,mode=max --metadata-file /home/runner/work/_temp/docker-actions-toolkit-si9yJo/bake-metadata-3093f93292.json --provenance mode=max,builder-id=https://github.com/docker/bake-action/actions/runs/12300037568/attempts/3 --sbom true binary
#0 building with "builder-f2c9d3e0-f8b5-4a52-b04c-877d7b196b0a" instance using docker-container driver

#1 [internal] load local bake definitions
#1 reading docker-bake.hcl 242B / 242B done
#1 DONE 0.0s
ERROR: failed to evaluate path "/tmp/bake-build": lstat /tmp/bake-build: no such file or directory

Being fixed in docker/buildx#2860

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tonistiigi tonistiigi Awaiting requested review from tonistiigi

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@crazy-max @tonistiigi