Skip to content

[v0.8] vendor: update buildkit to 10e6f94b#1042

Merged
tonistiigi merged 1 commit into
docker:v0.8from
tonistiigi:update-buildkit-220403-v0.8
Apr 3, 2022
Merged

[v0.8] vendor: update buildkit to 10e6f94b#1042
tonistiigi merged 1 commit into
docker:v0.8from
tonistiigi:update-buildkit-220403-v0.8

Conversation

@tonistiigi

@tonistiigi tonistiigi commented Apr 3, 2022

Copy link
Copy Markdown
Member

Signed-off-by: Tonis Tiigi [email protected]

@crazy-max

crazy-max commented Apr 3, 2022

Copy link
Copy Markdown
Member

I think we should also bump the replace: https://github.com/moby/buildkit/blob/10e6f94bf90d5224a32816a0ac923320c82a8c87/go.mod#L127

@tonistiigi

tonistiigi commented Apr 3, 2022

Copy link
Copy Markdown
Member Author

That is quite a lot of unrelated changes to bring into patch release(it was also not taken to buildkit patch release). Some of it could be potentially breaking (temporary error handling and TLS updates).

commit 3b32055aaa210928c4abfc618fd94dce8f28424a (HEAD -> update-buildkit-220403-v0.8)
Author: Tõnis Tiigi <[email protected]>
Date:   Sun Apr 3 15:21:27 2022 -0700

    vendor: update docker to match buildkit v0.10

    Signed-off-by: Tonis Tiigi <[email protected]>

 go.mod                                                                |  2 +-
 go.sum                                                                |  4 ++--
 vendor/github.com/docker/docker/api/swagger.yaml                      | 31 ++++++++++++++++++++----
 vendor/github.com/docker/docker/client/client.go                      |  3 ---
 vendor/github.com/docker/docker/client/container_wait.go              |  4 +++-
 vendor/github.com/docker/docker/client/options.go                     | 87 ++++++++++++++++++++++++++++++++++++++++++++++++--------------------
 vendor/github.com/docker/docker/client/request.go                     |  6 ++---
 vendor/github.com/docker/docker/pkg/archive/archive.go                | 70 +++++++++++++++++++++++++++++++++++++++++++++++-------
 vendor/github.com/docker/docker/pkg/archive/archive_linux.go          |  2 +-
 vendor/github.com/docker/docker/pkg/archive/archive_unix.go           | 29 +++++++++++++++--------
 vendor/github.com/docker/docker/pkg/archive/diff.go                   |  2 ++
 vendor/github.com/docker/docker/pkg/namesgenerator/names-generator.go | 11 +++++++++
 vendor/github.com/docker/docker/pkg/system/syscall_windows.go         | 52 ++++-------------------------------------
 vendor/modules.txt                                                    |  4 ++--
 14 files changed, 198 insertions(+), 109 deletions(-)

Is there anything specific we would need from there?

crazy-max
crazy-max approved these changes Apr 3, 2022
View reviewed changes

@crazy-max crazy-max left a comment •
edited
Loading

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Yes good point and no don't think there is anything specific necessary for us.

@tonistiigi
tonistiigi merged commit 6224def into docker:v0.8 Apr 3, 2022
@crazy-max crazy-max added this to the v0.8.2 milestone Apr 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@crazy-max crazy-max crazy-max approved these changes
@AkihiroSuda AkihiroSuda Awaiting requested review from AkihiroSuda

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

v0.8.2

Development

Successfully merging this pull request may close these issues.

2 participants

@tonistiigi @crazy-max