Skip to content

contrib/check-config.sh: remove special case for userns on CentOS/RHEL 7#48212

Merged
tianon merged 1 commit intomoby:masterfrom
thaJeztah:burning_them_with_fire
Jul 23, 2024
Merged

contrib/check-config.sh: remove special case for userns on CentOS/RHEL 7#48212
tianon merged 1 commit intomoby:masterfrom
thaJeztah:burning_them_with_fire

Conversation

@thaJeztah
Copy link
Copy Markdown
Member


contrib/check-config.sh: remove special case for userns on CentOS/RHEL 7

This check was added in 2355151 to handle non-standard behavior in CentOS/RHEL 7;

Add check for RHEL7/CentOS7 experimental userns disabled

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.

Now that CentOS/RHEL 7 reached EOL, we can remove this special case.

- A picture of a cute animal (not mandatory but encouraged)

il_1588xN 6041058883_oquq

This check was added in 2355151 to handle
non-standard behavior in CentOS/RHEL 7;

> Add check for RHEL7/CentOS7 experimental userns disabled
>
> 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.

Now that CentOS/RHEL 7 reached EOL, we can remove this special case.

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah
Copy link
Copy Markdown
Member Author

This one's for you @tianon 🔥 🔥 🔥 🔥 #20643 (comment)

Screenshot 2024-07-22 at 23 49 50

@thaJeztah thaJeztah self-assigned this Jul 22, 2024
Copy link
Copy Markdown
Member

@tianon tianon left a comment

Choose a reason for hiding this comment

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

Heck yes

@tianon tianon merged commit 70eca41 into moby:master Jul 23, 2024
@thaJeztah thaJeztah deleted the burning_them_with_fire branch July 23, 2024 22:32
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.

3 participants