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

deps: update buildkit, vendor changes #2175

Merged
merged 1 commit into from
Jan 5, 2024

Conversation

jsternberg
Copy link
Collaborator

No description provided.

@jedevc
Copy link
Collaborator

jedevc commented Jan 5, 2024

cc @thaJeztah since updates to docker/cli 😄

@thaJeztah
Copy link
Member

Overall looks fine, but before merging, you could consider having a look at moby/buildkit#4523 - if that's merged, you could update moby and docker/cli to v25.0.0-rc.1

The tricky bit is that neither docker/cli nor moby/moby are a module yet, so currently it requires "manual" compare of the vendor.mod from these repositories; one dependency that looks to be "behind" is Cobra; in docker/cli we updated to 1.8. CI looks green so it's not critical, and I'm happy to do a follow-up to align some f the remaining modules.

@thaJeztah
Copy link
Member

@jsternberg moby/buildkit#4523 was merged, so you can consider updating to current master 👍

Signed-off-by: Jonathan A. Sternberg <[email protected]>
Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

Thanks!

LGTM (nb)

@tonistiigi tonistiigi merged commit 9060cab into docker:master Jan 5, 2024
61 checks passed
@jsternberg jsternberg deleted the vendor-update branch January 5, 2024 20:50
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.

4 participants