Skip to content

Update docs in docker service create/update for flag --user#25330

Merged
vdemeester merged 1 commit intomoby:masterfrom
yongtang:docs-service-create-update-user
Aug 2, 2016
Merged

Update docs in docker service create/update for flag --user#25330
vdemeester merged 1 commit intomoby:masterfrom
yongtang:docs-service-create-update-user

Conversation

@yongtang
Copy link
Copy Markdown
Member

@yongtang yongtang commented Aug 2, 2016

In docker service create/update, flag --user actually supports uid:gid (same as docker run). However, this is not reflected in the help and documentation yet.

This fix updates docs in docker service create/update to change the description to Username or UID (format: <name|uid>[:<group|gid>]).

The help message output has also been updated.

This fix is related to #25304.

cc @thaJeztah

Signed-off-by: Yong Tang yong.tang.github@outlook.com

In `docker service create/update`, flag `--user` actually supports
`uid:gid` (same as `docker run`). However, this is not reflected
in the help and documentation yet.

This fix updates docs in `docker service create/update` to change
the description to `Username or UID (format: <name|uid>[:<group|gid>])`.

The help message output has also been updated.

This fix is related to 25304.

Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
@thaJeztah
Copy link
Copy Markdown
Member

cool, you beat me to it!

LGTM

@vdemeester
Copy link
Copy Markdown
Member

LGTM 👼

@vdemeester vdemeester merged commit 50c08f0 into moby:master Aug 2, 2016
@yongtang yongtang deleted the docs-service-create-update-user branch August 2, 2016 16:13
@SvenDowideit SvenDowideit modified the milestones: 1.13.0, 1.12.0 Aug 2, 2016
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.

5 participants