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

Add "userns" to `docker info` security options output #27840

Merged
merged 1 commit into from Oct 29, 2016

Conversation

Projects
None yet
5 participants
@estesp
Contributor

estesp commented Oct 27, 2016

Fixes: #27740

If user namespaces is enabled on the daemon, reveal that via docker info
by adding "userns" to the list of security options reported by the
info endpoint.

Signed-off-by: Phil Estes estesp@linux.vnet.ibm.com

@vdemeester

LGTM 👼

@thaJeztah

just some nits, also, can you;

Show outdated Hide outdated docs/reference/commandline/dockerd.md
Add "userns" to `docker info` security options output
If user namespaces is enabled on the daemon, reveal that via docker info
by adding "userns" to the list of security options reported by the
info endpoint.

Signed-off-by: Phil Estes <estesp@linux.vnet.ibm.com>
@estesp

This comment has been minimized.

Show comment
Hide comment
@estesp

estesp Oct 28, 2016

Contributor

Thanks @thaJeztah! Latest push has the extra updates and reworked wording in docs

Contributor

estesp commented Oct 28, 2016

Thanks @thaJeztah! Latest push has the extra updates and reworked wording in docs

@thaJeztah

perfect! LGTM thanks!

@vdemeester vdemeester merged commit 1fb9c4e into moby:master Oct 29, 2016

4 checks passed

docker/dco-signed All commits signed
Details
experimental Jenkins build Docker-PRs-experimental 25678 has succeeded
Details
janky Jenkins build Docker-PRs 34273 has succeeded
Details
windowsRS1 Jenkins build Docker-PRs-WoW-RS1 5163 has succeeded
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment