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 lenn...@poettering.net: 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

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 lenn...@poettering.net: 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

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 lenn...@poettering.net: 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

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 lenn...@poettering.net: 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

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 lenn...@poettering.net: 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

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 lenn...@poettering.net: 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 mbi...@gmail.com: See

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 mbi...@gmail.com: 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-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 mbi...@gmail.com: 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

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 mbi...@gmail.com: See http://lists.freedesktop.org/archives/systemd-devel/2015-May/031536.html

[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

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 mbi...@gmail.com wrote: 2015-05-08 17:43 GMT+02:00 Michael Biebl mbi...@gmail.com: 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

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 mbi...@gmail.com: 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