recoverd: Ignore failed flag updates on inactive nodes
authorAmitay Isaacs <amitay@gmail.com>
Fri, 27 Sep 2013 08:02:39 +0000 (18:02 +1000)
committerAmitay Isaacs <amitay@gmail.com>
Tue, 22 Oct 2013 03:34:03 +0000 (14:34 +1100)
Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Pair-programmed-with: Martin Schwenke <martin@meltin.net>

server/ctdb_recoverd.c

index 86a4d85dc52ff173dab07d74525bbfb1db87f122..381e998fb6b143034ec66626ebc45f35de8a0b6b 100644 (file)
@@ -1857,8 +1857,12 @@ static int do_recovery(struct ctdb_recoverd *rec,
 
                ret = update_flags_on_all_nodes(ctdb, nodemap, i, nodemap->nodes[i].flags);
                if (ret != 0) {
-                       DEBUG(DEBUG_ERR, (__location__ " Unable to update flags on all nodes for node %d\n", i));
-                       return -1;
+                       if (nodemap->nodes[i].flags & NODE_FLAGS_INACTIVE) {
+                               DEBUG(DEBUG_WARNING, (__location__ "Unable to update flags on inactive node %d\n", i));
+                       } else {
+                               DEBUG(DEBUG_ERR, (__location__ " Unable to update flags on all nodes for node %d\n", i));
+                               return -1;
+                       }
                }
        }