Don't remove master_candidate flag from merged nodes
authorStephen Shirley <diamond@google.com>
Tue, 15 Feb 2011 14:29:03 +0000 (15:29 +0100)
committerStephen Shirley <diamond@google.com>
Thu, 17 Feb 2011 10:09:24 +0000 (11:09 +0100)
Prevents lots of spurious warnings like:
2011-02-10 17:00:22,776: CRITICAL Configuration data is not consistent:
Not enough master candidates: actual 3, target 4

Signed-off-by: Stephen Shirley <diamond@google.com>
Reviewed-by: Iustin Pop <iustin@google.com>

tools/cluster-merge

index 52ecfc2..d2e8c1f 100755 (executable)
@@ -273,7 +273,6 @@ class Merger(object):
 
       for node in other_config.GetNodeList():
         node_info = other_config.GetNodeInfo(node)
-        node_info.master_candidate = False
         my_config.AddNode(node_info, _CLUSTERMERGE_ECID + str(fake_ec_id))
         fake_ec_id += 1