On Fri, Oct 26, 2012 at 02:06:33PM +0200, Pavel Březina wrote:
On 10/24/2012 10:18 PM, Jakub Hrozek wrote:
>We only read the configured SIGKILL time for services and actually used
>the value of 0, which actually translated into
>tevent_schedule_immediate. That meant that after we sent the SIGTERM we
>also sent a SIGKILL right away, which is bad...
Nack,
I think that the debug message on line monitor.c:897 should be
"Failed to get ping timeout for %s\n".
In the patch you allow kill_timeout = 0. In the original version it
is replaced with default value. Is this desired behaviour?
You are absolutely right, thank you. A new patch is attached.