-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
rootless: fix fast join userns path #10611
rootless: fix fast join userns path #10611
Conversation
commit ab88632 changed the path for the pause.pid file but didn't update the same path in the C code. This prevented Podman to take the fast path when the userns is already created and to join it without re-execing itself. Fix the path in the C code as well so we can join the rootless user+mount namespace without having to re-exec Podman. [NO TESTS NEEDED] Signed-off-by: Giuseppe Scrivano <[email protected]>
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: giuseppe The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
LGTM |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thank you!
/lgtm |
A quick test locally with: for i in {1..20}
do
$PODMAN run -d alpine ls
done Ran three times each: By no means a meaningful benchmark but we may consider backporting that to 3.0 and 3.1 branches. |
tests got stuck. /retest |
@cevich do I need to re-push? |
They look okay to me (now). Perhaps github was showing broken re-run status but Cirrus was still executing? |
I've closed the PR and opened it again :) That seemed to unblock the CI. All green now! @vrothberg PTAL |
@vrothberg unping, sorry, I've not seen there are already two LGTM /hold cancel |
commit ab88632 changed the path for
the pause.pid file but didn't update the same path in the C code.
This prevented Podman to take the fast path when the userns is already
created and to join it without re-execing itself.
Fix the path in the C code as well so we can join the rootless
user+mount namespace without having to re-exec Podman.
[NO TESTS NEEDED]
Signed-off-by: Giuseppe Scrivano [email protected]