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>
(cherry picked from commit 3ab52b528673e08caa66f00e963528c591a84fe1)

diff --git a/ctdb/server/ctdb_recoverd.c b/ctdb/server/ctdb_recoverd.c
index f825427e7a39a8bf620666ac0a774a02bef5f71b..d42333b860dc5f52d12d551e2afed6210551f9dd 100644 (file)
--- a/ctdb/server/ctdb_recoverd.c
+++ b/ctdb/server/ctdb_recoverd.c
@@ -2667,20 +2667,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;