[SSSD] [PATCH] Monitor: read the correct SIGKILL timeout for providers, too

Pavel Březina pbrezina at redhat.com
Thu Nov 1 10:49:19 UTC 2012


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.




More information about the sssd-devel mailing list