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

systemd: remove options unsupported in user daemon #50

Merged

Conversation

terinjokes
Copy link
Contributor

Many unit options are documented to not work in the user daemons,
usually with the following note in the systemd.exec(5) man page:

This option is only available for system services and is not supported
for services running in per-user instances of the service manager.

In some circumstances these settings work where user namespaces are
supported (by setting the PrivateUser option). However users
namespaces are disabled on most distributions.

This changeset pares the options to those supported in user daemons to
allow for ease of use for users.

Fixes: #49


CC: @flokli @LeSuisse @mdlayher as previous contributors to the systemd unit.

Many unit options are documented to not work in the user daemons,
usually with the following note in the systemd.exec(5) man page:

> This option is only available for system services and is not supported
> for services running in per-user instances of the service manager.

In some circumstances these settings work where user namespaces are
supported (by setting the `PrivateUser` option). However users
namespaces are disabled on most distributions.

This changeset pares the options to those supported in user daemons to
allow for ease of use for users.

Fixes: FiloSottile#49
@LeSuisse
Copy link
Contributor

Hi!

I would say user namespaces are enabled on most modern distribution these days but since it has already been a source of trouble (#41) it might be easier for everyone if those options are removed. Downstream packagers can still add them back.

@terinjokes
Copy link
Contributor Author

@LeSuisse There's also some discussion on the linked ticket (#49).

@terinjokes
Copy link
Contributor Author

If we decide to move forward with this, I'm happy to reword the commit message to better reflect the problem (older systemd versions).

@FiloSottile
Copy link
Owner

Sad, but it sounds like the systemd ecosystem was not built to enable upstreams to sandbox our applications effectively. Merging, thank you.

@FiloSottile FiloSottile merged commit 5000c32 into FiloSottile:master Sep 5, 2020
@flokli
Copy link
Contributor

flokli commented Sep 6, 2020

Sad, but it sounds like the systemd ecosystem was not built to enable upstreams to sandbox our applications effectively. Merging, thank you.

Hm, I wouldn't think this is per se true - it just seems older versions contain a bug, causing failures when some sandboxing options are requested, but user namespaces disabled.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

contributed systemd unit doesn't work in user session
4 participants