Re: [PATCH] BUG/MEDIUM: systemd: set KillMode to 'mixed'

2014-10-09 Thread Willy Tarreau
Hi Apollon, On Wed, Oct 08, 2014 at 03:14:41PM +0300, Apollon Oikonomopoulos wrote: By default systemd will send SIGTERM to all processes in the service's control group. In our case, this includes the wrapper, the master process and all worker processes. Since commit c54bdd2a the wrapper

Re: [PATCH] BUG/MEDIUM: systemd: set KillMode to 'mixed'

2014-10-09 Thread Jason J. W. Williams
I'm really amazed by the amount of breakage these new service managers are causing to a simple process management that has been working well for over 40 years of UNIX existence now, and the difficulty we have to work around this whole mess! If there was a poster child for knowing better than

Re: [PATCH] BUG/MEDIUM: systemd: set KillMode to 'mixed'

2014-10-09 Thread Apollon Oikonomopoulos
Hi Willy, On 11:26 Thu 09 Oct , Willy Tarreau wrote: Hi Apollon, On Wed, Oct 08, 2014 at 03:14:41PM +0300, Apollon Oikonomopoulos wrote: By default systemd will send SIGTERM to all processes in the service's control group. In our case, this includes the wrapper, the master process

Re: [PATCH] BUG/MEDIUM: systemd: set KillMode to 'mixed'

2014-10-09 Thread Willy Tarreau
On Thu, Oct 09, 2014 at 12:35:10PM +0300, Apollon Oikonomopoulos wrote: Hi Willy, On 11:26 Thu 09 Oct , Willy Tarreau wrote: Hi Apollon, On Wed, Oct 08, 2014 at 03:14:41PM +0300, Apollon Oikonomopoulos wrote: By default systemd will send SIGTERM to all processes in the service's

Re: [PATCH] BUG/MEDIUM: systemd: set KillMode to 'mixed'

2014-10-09 Thread Apollon Oikonomopoulos
On 11:44 Thu 09 Oct , Willy Tarreau wrote: On Thu, Oct 09, 2014 at 12:35:10PM +0300, Apollon Oikonomopoulos wrote: Hi Willy, On 11:26 Thu 09 Oct , Willy Tarreau wrote: Hi Apollon, On Wed, Oct 08, 2014 at 03:14:41PM +0300, Apollon Oikonomopoulos wrote: By default

Re: [PATCH] BUG/MEDIUM: systemd: set KillMode to 'mixed'

2014-10-09 Thread Willy Tarreau
On Thu, Oct 09, 2014 at 12:55:25PM +0300, Apollon Oikonomopoulos wrote: On 11:44 Thu 09 Oct , Willy Tarreau wrote: OK, so I'm merging your patch if you think it's the best solution. Well, I think it's the most sane thing to do and is behaviour-compatible with the current wrapper

Re: [PATCH] BUG/MEDIUM: systemd: set KillMode to 'mixed'

2014-10-09 Thread Apollon Oikonomopoulos
On 12:07 Thu 09 Oct , Willy Tarreau wrote: Anyway we're not there to discuss the benefits or defaults of systemd, some major distros have adopted it and now we have to work around its breakages so that users can continue to use their systems as if it was still a regular, manageable UNIX

[PATCH] BUG/MEDIUM: systemd: set KillMode to 'mixed'

2014-10-08 Thread Apollon Oikonomopoulos
By default systemd will send SIGTERM to all processes in the service's control group. In our case, this includes the wrapper, the master process and all worker processes. Since commit c54bdd2a the wrapper actually catches SIGTERM and survives to see the master process getting killed by systemd