Re: [systemd-devel] [PATCH] unit: Set KillMode=mixed for services which use sulogin

2015-05-13 Thread Lennart Poettering
On Wed, 13.05.15 16:18, Michael Biebl (mbi...@gmail.com) wrote: > 2015-05-13 15:45 GMT+02:00 Lennart Poettering : > > Well, I am pretty sure that in this case, it should be sulogin that > > propagates the shutdown request to the shell it spawned, but we should > > not do it otherwise. > > > > Note

Re: [systemd-devel] [PATCH] unit: Set KillMode=mixed for services which use sulogin

2015-05-13 Thread Michael Biebl
2015-05-13 15:45 GMT+02:00 Lennart Poettering : > Well, I am pretty sure that in this case, it should be sulogin that > propagates the shutdown request to the shell it spawned, but we should > not do it otherwise. > > Note that by default we don't even clean up processes of unprivileged > users on

Re: [systemd-devel] [PATCH] unit: Set KillMode=mixed for services which use sulogin

2015-05-13 Thread Lennart Poettering
On Wed, 13.05.15 15:32, Michael Biebl (mbi...@gmail.com) wrote: > 2015-05-13 15:19 GMT+02:00 Lennart Poettering : > > sulogin generally does not set up a PAM session, and we indeed should > > allow processes like screen staying around in such a context. Hence > > KillMode=process is actually the r

Re: [systemd-devel] [PATCH] unit: Set KillMode=mixed for services which use sulogin

2015-05-13 Thread Michael Biebl
2015-05-13 15:19 GMT+02:00 Lennart Poettering : > sulogin generally does not set up a PAM session, and we indeed should > allow processes like screen staying around in such a context. Hence > KillMode=process is actually the right choice for all these services, > indeed. Do you really think it mak

Re: [systemd-devel] [PATCH] unit: Set KillMode=mixed for services which use sulogin

2015-05-13 Thread Lennart Poettering
On Tue, 12.05.15 14:36, Michael Biebl (mbi...@gmail.com) wrote: > 2015-05-12 0:03 GMT+02:00 Lennart Poettering : > > On Tue, 12.05.15 00:01, Lennart Poettering (lenn...@poettering.net) wrote: > > > >> On Mon, 11.05.15 23:50, Michael Biebl (mbi...@gmail.com) wrote: > >> > >> > 2015-05-08 17:43 GMT+

Re: [systemd-devel] [PATCH] unit: Set KillMode=mixed for services which use sulogin

2015-05-12 Thread Michael Biebl
2015-05-12 0:03 GMT+02:00 Lennart Poettering : > On Tue, 12.05.15 00:01, Lennart Poettering (lenn...@poettering.net) wrote: > >> On Mon, 11.05.15 23:50, Michael Biebl (mbi...@gmail.com) wrote: >> >> > 2015-05-08 17:43 GMT+02:00 Michael Biebl : >> > > See >> > > http://lists.freedesktop.org/archive

Re: [systemd-devel] [PATCH] unit: Set KillMode=mixed for services which use sulogin

2015-05-11 Thread Lennart Poettering
On Tue, 12.05.15 00:01, Lennart Poettering (lenn...@poettering.net) wrote: > On Mon, 11.05.15 23:50, Michael Biebl (mbi...@gmail.com) wrote: > > > 2015-05-08 17:43 GMT+02:00 Michael Biebl : > > > See > > > http://lists.freedesktop.org/archives/systemd-devel/2015-May/031536.html > > > > Lennart,

Re: [systemd-devel] [PATCH] unit: Set KillMode=mixed for services which use sulogin

2015-05-11 Thread Lennart Poettering
On Mon, 11.05.15 23:50, Michael Biebl (mbi...@gmail.com) wrote: > 2015-05-08 17:43 GMT+02:00 Michael Biebl : > > See http://lists.freedesktop.org/archives/systemd-devel/2015-May/031536.html > > Lennart, are you ok if I commit this? Yes, please! Lennart -- Lennart Poettering, Red Hat _

Re: [systemd-devel] [PATCH] unit: Set KillMode=mixed for services which use sulogin

2015-05-11 Thread Michael Biebl
2015-05-08 17:43 GMT+02:00 Michael Biebl : > See http://lists.freedesktop.org/archives/systemd-devel/2015-May/031536.html Lennart, are you ok if I commit this? -- Why is it that all of the instruments seeking intelligent life in the universe are pointed away from Earth? _

Re: [systemd-devel] [PATCH] unit: Set KillMode=mixed for services which use sulogin

2015-05-08 Thread David Herrmann
Hi On Fri, May 8, 2015 at 7:59 PM, Michael Biebl wrote: > 2015-05-08 17:43 GMT+02:00 Michael Biebl : >> See http://lists.freedesktop.org/archives/systemd-devel/2015-May/031536.html >> >> We probably want to use KillMode=mixed, but let's start with the ones > > We probably want to use KillMode=mix

Re: [systemd-devel] [PATCH] unit: Set KillMode=mixed for services which use sulogin

2015-05-08 Thread Michael Biebl
2015-05-08 17:43 GMT+02:00 Michael Biebl : > See http://lists.freedesktop.org/archives/systemd-devel/2015-May/031536.html > > We probably want to use KillMode=mixed, but let's start with the ones We probably want to use KillMode=mixed *as well for other units which spawn of child processes*, but .

[systemd-devel] [PATCH] unit: Set KillMode=mixed for services which use sulogin

2015-05-08 Thread Michael Biebl
See http://lists.freedesktop.org/archives/systemd-devel/2015-May/031536.html We probably want to use KillMode=mixed, but let's start with the ones using sulogin. Michael -- Why is it that all of the instruments seeking intelligent life in the universe are pointed away from Earth? From d3118dc47