From: Iustin Pop <iustin@google.com>
Date: Tue, 30 Jun 2009 16:10:16 +0000 (+0200)
Subject: Cleanup config data when draining nodes
X-Git-Tag: v2.0.2~8
X-Git-Url: https://code.grnet.gr/git/ganeti-local/commitdiff_plain/dec0d9da26c6929e9157ec700f1e6dc8ab5a6bed

Cleanup config data when draining nodes

Currently, when draining nodes we reset their master candidate flag, but
we don't instruct them to demote themselves. This leads to “ERROR: file
'/var/lib/ganeti/config.data' should not exist on non master candidates
(and the file is outdated)”.

This patch simply adds a call to node_demote_from_mc in this case.

Signed-off-by: Iustin Pop <iustin@google.com>
Reviewed-by: Michael Hanselmann <hansmi@google.com>
---

diff --git a/lib/cmdlib.py b/lib/cmdlib.py
index f556677..cb3bb31 100644
--- a/lib/cmdlib.py
+++ b/lib/cmdlib.py
@@ -2464,6 +2464,10 @@ class LUSetNodeParams(LogicalUnit):
           node.master_candidate = False
           changed_mc = True
           result.append(("master_candidate", "auto-demotion due to drain"))
+          rrc = self.rpc.call_node_demote_from_mc(node.name)
+          msg = rrc.RemoteFailMsg()
+          if msg:
+            self.LogWarning("Node failed to demote itself: %s" % msg)
         if node.offline:
           node.offline = False
           result.append(("offline", "clear offline status due to drain"))