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
Make user io.podman.service unit WantedBy=default.target #4231
Make user io.podman.service unit WantedBy=default.target #4231
Conversation
/approve Commit is missing a signoff - can you add one and re-push? |
LGTM! |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: cgwalters, jlebon, mheon 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 once signed and repushed. Thanks @cgwalters |
@cgwalters Sign the PR and we will merge. |
`multi-user.target` doesn't exist in the systemd *user* instance. We can't hook up the startup of a user unit to a system target. Doing so causes systemd to error out in Fedora CoreOS builds during presets. Make it depend on `default.target` instead. (Having the same unit in both system and user sessions has some tricky bits like this) Signed-off-by: Colin Walters <walters@verbum.org>
f9ed9cf
to
ca29a5b
Compare
Done |
/lgtm |
multi-user.target
doesn't exist in the systemd user instance.We can't hook up the startup of a user unit to a system target.
Doing so causes systemd to error out in Fedora CoreOS builds
during presets.
Make it depend on
default.target
instead.(Having the same unit in both system and user sessions has some
tricky bits like this)