[PATCH] Use downed

Discussion:

[PATCH] Use downed_hosts

S.Çağlar Onur

2011-04-27 19:39:03 UTC

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.

seth vidal

2011-05-09 20:30:47 UTC

Permalink

Post by S.ÃaÄlar Onur
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?

-sv

S.Çağlar Onur

2011-05-11 15:23:29 UTC

Permalink

Hi Seth,

Post by seth vidal

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.

Post by seth vidal
-sv

Cheers,
--
S.Çağlar Onur <***@10ur.org>

S.Çağlar Onur

2011-05-20 19:03:26 UTC

Permalink

Should I go ahead or commit that patch or do you still have concerns about it?

Post by S.ÃaÄlar Onur
Hi Seth,

Post by seth vidal

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.

Post by seth vidal
-sv

Cheers,
--
S.Çağlar Onur <***@10ur.org>

seth vidal

2011-05-23 19:42:01 UTC

Permalink

Post by S.ÃaÄlar Onur
Should I go ahead or commit that patch or do you still have concerns about it?

My hope was just to make it a bit more generic - but go ahead and apply
it and we'll sift things out as we go.

-sv