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.

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

include default ssh socket when given an ssh-based git url #581

Merged
merged 2 commits into from
Apr 2, 2021

Conversation

alexcb
Copy link
Contributor

@alexcb alexcb commented Apr 1, 2021

Signed-off-by: Alex Couture-Beil [email protected]

@alexcb
Copy link
Contributor Author

alexcb commented Apr 1, 2021

@tonistiigi here's my changes to make use of moby/buildkit#2056

I'm unsure how I should update go.mod; if I run go get -u github.com/moby/buildkit@df49b648c8bfef657ba99543390516a3f518d22a it updates 5 of so packages, is that the desired way to pull in the latest buildkit code? I can then run hack/update-vendor to pull in the deps. Just wanted to double if there's a different way to do this with more fine-control?

@tonistiigi
Copy link
Member

@alexcb You can just update to go.mod file to point to the sha and then run hack/update-vendor that will fix everything up. It is expected that it also updates records of the dependencies that have been updated.

- updated buildkit to current code in master via:

  go mod edit -require github.com/moby/buildkit@master && go mod tidy && ./hack/update-vendor

Signed-off-by: Alex Couture-Beil <[email protected]>
@alexcb
Copy link
Contributor Author

alexcb commented Apr 2, 2021

thanks for guiding me through the process @tonistiigi , I wasn't sure if it was expected for this PR to end up being 312 files changed.

@alexcb alexcb marked this pull request as ready for review April 2, 2021 00:10
@tonistiigi tonistiigi merged commit adc5f35 into docker:master Apr 2, 2021
alexcb added a commit to alexcb/cli-1 that referenced this pull request Apr 13, 2021
…d line

- fixes moby/buildkit#2040 with a similar fix
which was done in docker/buildx#581

Signed-off-by: Alex Couture-Beil <[email protected]>
alexcb added a commit to alexcb/cli-1 that referenced this pull request Apr 13, 2021
- this fixes moby/buildkit#2040 by enabling
ssh forwarding when a remote address is given on the command line, this
is a similar fix to docker/buildx#581

Signed-off-by: Alex Couture-Beil <[email protected]>
alexcb added a commit to alexcb/cli-1 that referenced this pull request Apr 21, 2021
- this fixes moby/buildkit#2040 by enabling
ssh forwarding when a remote address is given on the command line, this
is a similar fix to docker/buildx#581

Signed-off-by: Alex Couture-Beil <[email protected]>
alexcb added a commit to alexcb/cli-1 that referenced this pull request Apr 21, 2021
- this fixes moby/buildkit#2040 by enabling
ssh forwarding when a remote address is given on the command line, this
is a similar fix to docker/buildx#581

Signed-off-by: Alex Couture-Beil <[email protected]>
alexcb added a commit to alexcb/cli-1 that referenced this pull request Apr 21, 2021
- this fixes moby/buildkit#2040 by enabling
ssh forwarding when a remote address is given on the command line, this
is a similar fix to docker/buildx#581

Signed-off-by: Alex Couture-Beil <[email protected]>
alexcb added a commit to alexcb/cli-1 that referenced this pull request Aug 19, 2021
- this fixes moby/buildkit#2040 by enabling
ssh forwarding when a remote address is given on the command line, this
is a similar fix to docker/buildx#581

Signed-off-by: Alex Couture-Beil <[email protected]>
docker-jenkins pushed a commit to docker-archive/docker-ce that referenced this pull request Aug 19, 2021
- this fixes moby/buildkit#2040 by enabling
ssh forwarding when a remote address is given on the command line, this
is a similar fix to docker/buildx#581

Signed-off-by: Alex Couture-Beil <[email protected]>
Upstream-commit: af1bb80c3482d1395123af3037bb5ecf5feb5a7e
Component: cli
crazy-max pushed a commit to crazy-max/docker-cli that referenced this pull request Mar 24, 2023
- this fixes moby/buildkit#2040 by enabling
ssh forwarding when a remote address is given on the command line, this
is a similar fix to docker/buildx#581

Signed-off-by: Alex Couture-Beil <[email protected]>
(cherry picked from commit af1bb80)
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