Skip to content

Add check for RHEL7/CentOS7 experimental userns disabled#20643

Merged
thaJeztah merged 1 commit intomoby:masterfrom
estesp:userns-chkconfig-redhat
Feb 24, 2016
Merged

Add check for RHEL7/CentOS7 experimental userns disabled#20643
thaJeztah merged 1 commit intomoby:masterfrom
estesp:userns-chkconfig-redhat

Conversation

@estesp
Copy link
Copy Markdown
Contributor

@estesp estesp commented Feb 24, 2016

Fixes: #20193

Add a check in check-config.sh to see if we are running on a RHEL7 or
CentOS7 system, which may report that CONFIG_USERNS is OK/enabled, but
user namespaces still won't work because of the experimental feature
flag added by Redhat.

This will add a warning if it is actually disabled and notes what has to
be added to the grub/boot command line to enable it.

Docker-DCO-1.1-Signed-off-by: Phil Estes [email protected] (github: estesp)

Add a check in `check-config.sh` to see if we are running on a RHEL7 or
CentOS7 system, which may report that CONFIG_USERNS is OK/enabled, but
user namespaces still won't work because of the experimental feature
flag added by Redhat.

This will add a warning if it is actually disabled and notes what has to
be added to the grub/boot command line to enable it.

Docker-DCO-1.1-Signed-off-by: Phil Estes <[email protected]> (github: estesp)
@estesp
Copy link
Copy Markdown
Contributor Author

estesp commented Feb 24, 2016

ping @thaJeztah

@tianon
Copy link
Copy Markdown
Member

tianon commented Feb 24, 2016

BURN THEM WITH FIRE

@tianon
Copy link
Copy Markdown
Member

tianon commented Feb 24, 2016

(Code here looks OK for what it has to be though... ie LGTM)

@tianon
Copy link
Copy Markdown
Member

tianon commented Feb 24, 2016

I've gotta ask though; have you tested it against busybox's sh implementation? 🙈

Err, #!/usr/bin/env bash, ignore me. 😇

@thaJeztah
Copy link
Copy Markdown
Member

Oh, awesome, thanks!

LGTM

thaJeztah added a commit that referenced this pull request Feb 24, 2016
Add check for RHEL7/CentOS7 experimental userns disabled
@thaJeztah thaJeztah merged commit 2cc54a2 into moby:master Feb 24, 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.

Docker does not start containers in CentOS when user namespaces activated

4 participants