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

Jakub Hrozek jhrozek at redhat.com
Thu Nov 1 14:42:25 UTC 2012


On Thu, Nov 01, 2012 at 11:49:19AM +0100, Pavel Březina wrote:
> 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.

Pushed to master and sssd-1-9



More information about the sssd-devel mailing list