Should I go ahead or commit that patch or do you still have concerns about it?
On May 11, 2011, at 11:23 AM, S.Çağlar Onur wrote: > Hi Seth, > > On May 9, 2011, at 4:30 PM, seth vidal wrote: > >> On Wed, 2011-04-27 at 15:39 -0400, S.Çağlar Onur wrote: >>> Hi, >>> >>> Attached patch adds a new function called get_all_up_hosts which returns >>> host list - downed host list so that scripts (func-yum and func-ps-compare) >>> and modules (ping and grep) can relay on it. >>> >> >> In general I think this patch is good - but do we want to put it >> somewhere more generic so all the scripts can get to it? > > IMHO we should. Downed host list is a feature provided by func and external > scripts/internal modules shouldn't handle/implement that feature themselves. > >> -sv > Cheers, -- S.Çağlar Onur <[email protected]> _______________________________________________ Func-list mailing list [email protected] https://www.redhat.com/mailman/listinfo/func-list
