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

vendor: github.com/docker/docker cfdca8dc1d4c (v25.0.0-dev) #4673

Merged
merged 1 commit into from
Nov 24, 2023

Conversation

thaJeztah
Copy link
Member

@thaJeztah thaJeztah commented Nov 23, 2023

full diff: moby/moby@v25.0.0-beta.1...cfdca8d

- What I did

- How I did it

- How to verify it

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

@thaJeztah
Copy link
Member Author

No significant changes, so I'll keep this in draft for now.

@codecov-commenter
Copy link

codecov-commenter commented Nov 23, 2023

Codecov Report

Merging #4673 (72a0f94) into master (6d820e8) will not change coverage.
The diff coverage is 33.33%.

❗ Current head 72a0f94 differs from pull request most recent head 539537d. Consider uploading reports for the commit 539537d to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4673   +/-   ##
=======================================
  Coverage   59.75%   59.75%           
=======================================
  Files         287      287           
  Lines       24821    24821           
=======================================
  Hits        14832    14832           
  Misses       9103     9103           
  Partials      886      886           

@thaJeztah
Copy link
Member Author

thaJeztah commented Nov 23, 2023

60.33 cli/command/service/update.go:1298:18: SA1019: spec.Networks is deprecated: This field is deprecated since v1.44. The Networks field in TaskSpec should be used instead. (staticcheck)
60.33 		specNetworks = spec.Networks
60.33 		               ^
60.33 cli/command/service/update.go:1300:2: SA1019: spec.Networks is deprecated: This field is deprecated since v1.44. The Networks field in TaskSpec should be used instead. (staticcheck)
60.33 	spec.Networks = nil
60.33 	^
60.33 cli/compose/convert/service.go:177:3: SA1019: serviceSpec.Networks is deprecated: This field is deprecated since v1.44. The Networks field in TaskSpec should be used instead. (staticcheck)
60.33 		serviceSpec.Networks = networks
60.33 		^

⚠️ if we update, we probably may have to do so based on API version, or otherwise make sure that the apiClient itself takes care of that.

also added nolint:staticcheck to spec.Networks refs

full diff: moby/moby@v25.0.0-beta.1...cfdca8d

Co-authored-by: Albin Kerouanton <[email protected]>
Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah
Copy link
Member Author

⚠️ if we update, we probably may have to do so based on API version, or otherwise make sure that the apiClient itself takes care of that.

Chatted with @akerouanton and conversion is already there, so we only needed nolint comments 👍

@thaJeztah thaJeztah marked this pull request as ready for review November 24, 2023 08:52
Copy link
Member

@akerouanton akerouanton left a comment

Choose a reason for hiding this comment

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

LGTM!

@thaJeztah thaJeztah merged commit 5aa3077 into docker:master Nov 24, 2023
76 checks passed
@thaJeztah thaJeztah deleted the update_engine branch November 24, 2023 09:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants