2008/10/1 Iustin Pop <[EMAIL PROTECTED]>:
> The watcher didn't handle the down nodes, fix this by ignoring (in
> secondary node reboot checks) any node that doesn't return a boot id.
> ---
>  daemons/ganeti-watcher |    5 +++++
>  1 files changed, 5 insertions(+), 0 deletions(-)
>
> diff --git a/daemons/ganeti-watcher b/daemons/ganeti-watcher
> index 438cf00..d7250e4 100755
> --- a/daemons/ganeti-watcher
> +++ b/daemons/ganeti-watcher
> @@ -306,6 +306,11 @@ class Watcher(object):
>     check_nodes = []
>     for name, new_id in self.bootids.iteritems():
>       old = notepad.GetNodeBootID(name)
> +      if new_id is None:
> +        # Bad node, not returning a boot id
> +        logging.debug("Node %s, missing boot id, skipping secondary checks",
> +                      name)

Remove the first comma ("Node %s missing boot id, skipping …"), then LGTM

Reply via email to