-
Notifications
You must be signed in to change notification settings - Fork 18.6k
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
Explicitly add --detach to service CLI calls #35007
Conversation
The behavior of service (create/update/scale) was changed in a recent PR to docker/cli. This commit serves to remedy test failures experienced when attempting to use service calls. Should not affect current behavior. Signed-off-by: Eli Uriegas <eli.uriegas@docker.com>
Relates to: docker/cli#525 |
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.
LGTM
Also related: docker-archive/docker-ce#255 (for testing purposes) |
LGTM |
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.
LGTM when tests go green
@seemethere fyi we'll need to adjust the PR to not have |
Signed-off-by: Victor Vieux <victorvieux@gmail.com>
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.
LGTM 🐮
Thanks for making this change. |
Signed-off-by: Eli Uriegas eli.uriegas@docker.com
- What I did
The behavior of service (create/update/
scale) was changed in a recent PRto docker/cli. This commit serves to remedy test failures experienced
when attempting to use service calls.
Should not affect current behavior.
- How I did it
- How to verify it
- Description for the changelog
--detach
flags- A picture of a cute animal (not mandatory but encouraged)