Skip to content

add missing param Init in ContainerSpec#1927

Merged
eddumelendez merged 3 commits intodocker-java:masterfrom
ziminghua:AddUtilParam
Sep 19, 2022
Merged

add missing param Init in ContainerSpec#1927
eddumelendez merged 3 commits intodocker-java:masterfrom
ziminghua:AddUtilParam

Conversation

@ziminghua
Copy link
Copy Markdown
Contributor

@ziminghua ziminghua commented Aug 22, 2022

When I use update service cmd,find missing Init param in ContainerSpec

@ziminghua
Copy link
Copy Markdown
Contributor Author

@bsideup Can you review this PR? Dose this project still alive?

Copy link
Copy Markdown
Member

@eddumelendez eddumelendez left a comment

Choose a reason for hiding this comment

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

Thanks for the PR @ziminghua ! Just a small comment about the version where this attribute is available

Comment thread docker-java-api/src/main/java/com/github/dockerjava/api/model/ContainerSpec.java Outdated
…ContainerSpec.java

Co-authored-by: Eddú Meléndez Gonzales <[email protected]>
@ziminghua
Copy link
Copy Markdown
Contributor Author

Thank you for reminding me

@eddumelendez eddumelendez merged commit 3ea46f9 into docker-java:master Sep 19, 2022
@eddumelendez
Copy link
Copy Markdown
Member

Thanks for the PR @ziminghua ! this is now in master branch

@ziminghua
Copy link
Copy Markdown
Contributor Author

Thanks for the PR @ziminghua ! this is now in master branch

Thanks

@eddumelendez eddumelendez added this to the next milestone Sep 19, 2022
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.

2 participants