On Tue, Dec 1, 2009 at 1:47 PM, Michael Hanselmann <[email protected]> wrote: > 2009/11/27 Guido Trotter <[email protected]>: >> On Fri, Nov 27, 2009 at 5:16 PM, Michael Hanselmann <[email protected]> >> wrote: >>> Passing it in as a parameter seems more logical. >> >> Why? Also then we might want to put it just in masterd, rather, since >> it's currently the only multithreaded daemon (and it's a single >> assignment anyway). > > The parameter is also passed to utils.SetupLogging. Disabling forks is > another open issue (and not too easy, ganeti-masterd uses os.fork in > its check function). I suggest committing this patch as it is and then > I can work on disabling os.fork and friends. >
Ok, LGTM then :) Guido
