Skip to content
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

mount-setup: change the system mount propagation to shared by default only at bootup #128

Closed
wants to merge 1 commit into from

Conversation

yummypeng
Copy link

The commit b3ac5f8 has changed the system mount propagation to
shared by default, and according to the following patch:
opencontainers/runc#208
When starting the container, the pouch daemon will call runc to execute
make-private.

However, if the systemctl daemon-reexec is executed after the container
has been started, the system mount propagation will be changed to share
again by default, and the make-private operation above will have no chance
to execute.

(cherry picked from commit f74349d)
Signed-off-by: Yuanhong Peng yummypeng@linux.alibaba.com

… only at bootup

The commit b3ac5f8 has changed the system mount propagation to
shared by default, and according to the following patch:
opencontainers/runc#208
When starting the container, the pouch daemon will call runc to execute
make-private.

However, if the systemctl daemon-reexec is executed after the container
has been started, the system mount propagation will be changed to share
again by default, and the make-private operation above will have no chance
to execute.

(cherry picked from commit f74349d)
Signed-off-by: Yuanhong Peng <yummypeng@linux.alibaba.com>
@systemd-rhel-bot systemd-rhel-bot added pr/needs-ci Formerly needs-ci pr/needs-review Formerly needs-review tracker/missing Formerly needs-bz and removed pr/needs-ci Formerly needs-ci labels Dec 18, 2020
@jamacku
Copy link
Member

jamacku commented Oct 20, 2023

We don't plan to merge this PR. It needs a proper tracker (JIRA issue). Also, RHEL7 in Maintenance Support 2 Phase.

@jamacku jamacku closed this Oct 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr/needs-review Formerly needs-review tracker/missing Formerly needs-bz
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants