ctdb-recoverd: Drop unnecessary code

This has already been done in update_flags().

BUG: https://bugzilla.samba.org/show_bug.cgi?id=14513
Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>

diff --git a/ctdb/server/ctdb_recoverd.c b/ctdb/server/ctdb_recoverd.c
index aeb23276fe7d49fcb5e32a7e2ca3eec7d82dfe71..b69462094ee202f300753a5654944b6e72573a8d 100644 (file)
--- a/ctdb/server/ctdb_recoverd.c
+++ b/ctdb/server/ctdb_recoverd.c
@@ -2669,20 +2669,6 @@ static void main_loop(struct ctdb_context *ctdb, struct ctdb_recoverd *rec,
                }
        }
 
-       /*
-        * Update node flags obtained from each active node. This ensure we have
-        * up-to-date information for all the nodes.
-        */
-       for (j=0; j<nodemap->num; j++) {
-               if (nodemap->nodes[j].pnn == ctdb->pnn) {
-                       continue;
-               }
-               if (nodemap->nodes[j].flags & NODE_FLAGS_INACTIVE) {
-                       continue;
-               }
-               nodemap->nodes[j].flags = remote_nodemaps[j]->nodes[j].flags;
-       }
-
        for (j=0; j<nodemap->num; j++) {
                if (nodemap->nodes[j].pnn == ctdb->pnn) {
                        continue;