[SSSD] [PATCH] MONITOR: use sigchld handler for monitoring SSSD services

Stephen Gallagher sgallagh at redhat.com
Tue Dec 20 14:04:38 UTC 2011


On Tue, 2011-12-20 at 11:05 +0100, Jakub Hrozek wrote:
> On Mon, Dec 19, 2011 at 03:22:31PM -0500, Stephen Gallagher wrote:
> > On Mon, 2011-12-19 at 11:36 -0500, Stephen Gallagher wrote:
> > > On Mon, 2011-12-19 at 17:29 +0100, Jakub Hrozek wrote:
> > > > On Wed, Dec 14, 2011 at 10:06:38PM -0500, Stephen Gallagher wrote:
> > > > > This patch depends on the SIGCHLD patches currently under review. This
> > > > > is the first consumer of those patches, and therefore can be used to
> > > > > test them.
> > > > > 
> > > > > This changes the behavior of the monitor's child-monitoring code to use
> > > > > the new sigchld handler. It also replaces some useless loop-checks that
> > > > > we were doing to verify that the process was still up using waitpid().
> > > > > It's less wasteful to simply rely on SIGCHLD.
> > > > > 
> > > > > I also revamped the restart logic so that now it will always restart
> > > > > when the SIGCHLD handler is hit. To that end, I changed it so that the
> > > > > ping timeout limit would kill the PID (thereby triggering the SIGCHLD
> > > > > handler to take over).
> > > > > 
> > > > 
> > > > Why was the logic to limit the number of service restarts removed? I
> > > > think it can be useful for cases where a domain is badly misconfigured
> > > > so that the monitor does not spin forever trying to restart a service.
> > > > 
> > > 
> > > Oops, that was an oversight. I forgot to add that back in. I'll get a
> > > patch for this shortly.
> > 
> > New patch attached re-adds support for service startup limitation. I
> > tested it by setting 'command = /bin/false' in the [pam] section of
> > sssd.conf.
> > 
> > 
> 
> Ack

Pushed to master and opened https://fedorahosted.org/sssd/ticket/1119 to
track making the SIGKILL timer configurable.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20111220/d96effb2/attachment.sig>


More information about the sssd-devel mailing list