-
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
healthcheck: support rootless mode #3529
healthcheck: support rootless mode #3529
Conversation
[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 |
now that dbus authentication works fine from a user namespace (systemd 241 works fine), we can enable rootless healthchecks. It uses "systemd-run --user" for creating the healthcheck timer and communicates with the user instance of systemd listening at $XDG_RUNTIME_DIR/systemd/private. Closes: containers#3523 Signed-off-by: Giuseppe Scrivano <[email protected]>
301f892
to
c6c637d
Compare
Looks like this fixes this issue:
I'm unsure: should I report this here or on the issue? Is there a comment command I have to write to "acknowledge" the fix? |
@stefanb2 here it is enough. Thanks to have tried it out! |
... and the service & timer units are removed after the container is stopped. |
/lgtm |
LGTM, nice work @giuseppe |
/hold cancel |
now that dbus authentication works fine from a user namespace (systemd
241 works fine), we can enable rootless healthchecks.
It uses "systemd-run --user" for creating the healthcheck timer and
communicates with the user instance of systemd listening at
$XDG_RUNTIME_DIR/systemd/private.
Closes: #3523
Signed-off-by: Giuseppe Scrivano [email protected]