On 10/29/2012 05:02 PM, Jakub Hrozek wrote:
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.
Ack.