Skip to content

daemon: remove setMayDetachMounts (set may_detach_mounts=1 on startup)#48210

Merged
thaJeztah merged 1 commit intomoby:masterfrom
thaJeztah:remove_setMayDetachMounts
Jul 29, 2024
Merged

daemon: remove setMayDetachMounts (set may_detach_mounts=1 on startup)#48210
thaJeztah merged 1 commit intomoby:masterfrom
thaJeztah:remove_setMayDetachMounts

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

This function was added in 83c2152 to automatically set /proc/sys/fs/may_detach_mounts=1 on startup.

This is a kernel config available in RHEL7.4 based kernels that enables mountpoint removal where the mountpoint exists in other namespaces. This setting is the default, and non-configurable, on upstream kernels since 3.15.

As this option was only supported in RHEL 7.x systems, which reached EOL, we can remove this code, as it's not doing anything on current kernels.

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

This function was added in 83c2152 to
automatically set `/proc/sys/fs/may_detach_mounts=1` on startup.

This is a kernel config available in RHEL7.4 based kernels that enables
mountpoint removal where the mountpoint exists in other namespaces. This
setting is the default, and non-configurable, on upstream kernels since
3.15.

As this option was only supported in RHEL 7.x systems, which reached EOL,
we can remove this code, as it's not doing anything on current kernels.

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah thaJeztah added status/2-code-review area/daemon Core Engine kind/refactor PR's that refactor, or clean-up code labels Jul 22, 2024
@thaJeztah thaJeztah added this to the 28.0.0 milestone Jul 22, 2024
@thaJeztah thaJeztah self-assigned this Jul 22, 2024
@thaJeztah thaJeztah requested review from cpuguy83 and vvoland July 25, 2024 05:26
@thaJeztah thaJeztah merged commit c1b7972 into moby:master Jul 29, 2024
@thaJeztah thaJeztah deleted the remove_setMayDetachMounts branch July 29, 2024 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/daemon Core Engine kind/refactor PR's that refactor, or clean-up code status/2-code-review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants