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 <caglar@xxxxxxxx> _______________________________________________ Func-list mailing list Func-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/func-list