On Wed, 2009-09-09 at 13:33 -0400, Stephen Gallagher wrote:
> Patch 0001: Ensure that daemonization happens after reading in the
> configuration, so critical configuration errors repord "FAILED" during
> init script startup.

Nack, monitor_process_init() performs a number of operations (like
forking children) that assumes we are operating in the final monitor
process.
Moving become_daemon() after it would make these children loose, and
might interfere with other things like signal handlers etc.

In general I don't think we should do this.

> Patch 0002: Ensure that the minId and maxId values are sensible (minId
> > maxId, neither < 0)

ack this one.

Simo.

-- 
Simo Sorce * Red Hat, Inc * New York

_______________________________________________
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://fedorahosted.org/mailman/listinfo/sssd-devel

Reply via email to