Skip to content

Update bash completion and deprecation for synchronous service updates - #610

Merged
vdemeester merged 2 commits into
docker:masterfrom
albers:completion-service--detach
Oct 7, 2017
Merged

Update bash completion and deprecation for synchronous service updates#610
vdemeester merged 2 commits into
docker:masterfrom
albers:completion-service--detach

Conversation

@albers

@albersalbers commented Oct 7, 2017

Copy link
Copy Markdown
Collaborator

The default value for docker service create|update|scale|rollback --detach|-d changed from true to false.
This updates bash completion to complete just --detach|-d.

This should go into 17.10.0 because the behavior changes in this release.

Also updates the deprecation of asynchronous service updates:

  • The default was not changed in 17.09 but will be in 17.10
  • service scale and service rollback are also affected.

The default value for
`docker service create|update|scale|rollback --detach|-d` changed from
`true` to `false`.
This updates bash completion to complete just `--detach|-d`.
Signed-off-by: Harald Albers <github@albersweb.de>
- The default was not changed in 17.09 but will be in 17.10
- `service scale` and `service rollback` are also affected.
Signed-off-by: Harald Albers <github@albersweb.de>

@thaJeztahthaJeztah left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, thank you!

@vdemeestervdemeester left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM 🐯

Sign up for freeto 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.

4 participants

@albers@vdemeester@thaJeztah@GordonTheTurtle