Skip to content

Conversation

@kolyshkin
Copy link
Contributor

Backport of #4724 to release-1.3 branch. Original description follows.


This PR adds support for applying mount propagation settings (MS_SHARED or MS_UNBINDABLE) to the container root based on the value of config.RootPropagation.
We apply mount propagation after executing pivot_root and rootfsParentMountPrivate

Fixes #1755

Related:
#1815
youki-dev/youki#3141

Signed-off-by: Yusuke Sakurai [email protected]

@kolyshkin kolyshkin added the backport/1.3-pr A backport PR to release-1.3 label Jun 19, 2025
@kolyshkin kolyshkin changed the base branch from main to release-1.3 June 19, 2025 17:46
@kolyshkin kolyshkin added this to the 1.3.1 milestone Jun 19, 2025
@kolyshkin kolyshkin requested review from AkihiroSuda, lifubang and rata and removed request for rata June 19, 2025 17:47
Copy link
Member

@rata rata left a comment

Choose a reason for hiding this comment

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

LGTM

Do we want this in 1.2?

@rata
Copy link
Member

rata commented Jun 20, 2025

Hmm, all-done is not marked as ready nor I can re-trigger it. Do you want to push again? Clearly tests didn't run

Signed-off-by: Yusuke Sakurai <[email protected]>
(cherry picked from commit 04be81b)
Signed-off-by: Kir Kolyshkin <[email protected]>
@lifubang
Copy link
Member

Do we want this in 1.2?

In fact, this raises a question: Do we need to backport any bug fixes to the release-1.2 branch?

@lifubang lifubang merged commit e863512 into opencontainers:release-1.3 Jun 20, 2025
31 checks passed
@rata
Copy link
Member

rata commented Jun 20, 2025

@lifubang yes, we said we will be quite liberals with backporting to n-1. I think we should :)

@rata
Copy link
Member

rata commented Jun 20, 2025

1.2 PR: #4791

@kolyshkin
Copy link
Contributor Author

@lifubang yes, we said we will be quite liberals with backporting to n-1. I think we should :)

1.2 PR: #4791

I wasn't aware we'll do any more 1.2.x releases.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport/1.3-pr A backport PR to release-1.3

Projects

None yet

Development

Successfully merging this pull request may close these issues.

rootfsPropagation=shared does not work

5 participants