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

Recursive move with test #28457

Closed
wants to merge 2 commits into from
Closed

Conversation

keszybz
Copy link
Member

@keszybz keszybz commented Jul 19, 2023

Combine #28456 with #28454.

mrc0mmand and others added 2 commits July 19, 2023 15:17
Since 7c764d4 we bind mount certain directories during switch root
instead of moving the mount directly, and for /run we do this without
MS_REC. This, unfortunately, leaves all mounts under /run behind
in the old root, which breaks certain use cases.

See: systemd#28452
In 7c764d4, we changed behavior
from recursively moving all mounts in /run from the initrd to the
rootfs on switch-root to only moving /run itself from the initrd to
the rootfs on switch-root. This breaks applications that rely on
mounts in /run to survive the switch-root operation.

Let's switch back to recursively moving all mounts from /run to the
rootfs to keep backwards compat intact.
@github-actions github-actions bot added util-lib tests please-review PR is ready for (re-)review by a maintainer labels Jul 19, 2023
@github-actions
Copy link

An -rc1 tag has been created and a release is being prepared, so please note that PRs introducing new features and APIs will be held back until the new version has been released.

@bluca
Copy link
Member

bluca commented Jul 24, 2023

superseded by #28497

@bluca bluca closed this Jul 24, 2023
@github-actions github-actions bot removed the please-review PR is ready for (re-)review by a maintainer label Jul 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

4 participants