-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Update dependency packages for sdk package #23913
Conversation
7809b83
to
a900abd
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! Thanks for updating these. I have gone ahead and updated the branch with main
. I will merge once all the checks pass.
@@ -7,7 +7,7 @@ require ( | |||
github.com/armon/go-metrics v0.4.1 | |||
github.com/armon/go-radix v1.0.0 | |||
github.com/cenkalti/backoff/v3 v3.2.2 | |||
github.com/docker/docker v24.0.5+incompatible | |||
github.com/docker/docker v24.0.7+incompatible |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The CI setup is failing due to a missing go.sum entry for this dependency. You can resolve with the following:
github.com/docker/docker@v24.0.7+incompatible: missing go.sum entry for go.mod file; to add it:
go mod download github.com/docker/docker
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for some reason go mod download github.com/docker/docker
did nothing for me. So I performed global update:
go mod download
go mod tidy
#23912