ctdb-recoverd: Remove unnecessary assignments of need_takeover_run
authorMartin Schwenke <martin@meltin.net>
Wed, 28 Oct 2015 08:52:37 +0000 (19:52 +1100)
committerAmitay Isaacs <amitay@samba.org>
Thu, 12 Nov 2015 05:24:15 +0000 (06:24 +0100)
do_takeover_run() unsets this if it succeeds.

Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
ctdb/server/ctdb_recoverd.c

index 1ef7560bc4cd6ae789bd4037b563d64c109c62e6..a772a15765177849c8b435535dc67480a2fc7a03 100644 (file)
@@ -4025,8 +4025,6 @@ static void main_loop(struct ctdb_context *ctdb, struct ctdb_recoverd *rec,
        if (rec->need_takeover_run) {
                uint32_t culprit = (uint32_t)-1;
 
-               rec->need_takeover_run = false;
-
                /* update the list of public ips that a node can handle for
                   all connected nodes
                */
@@ -4034,7 +4032,6 @@ static void main_loop(struct ctdb_context *ctdb, struct ctdb_recoverd *rec,
                if (ret != 0) {
                        DEBUG(DEBUG_ERR,("Failed to read public ips from remote node %d\n",
                                         culprit));
-                       rec->need_takeover_run = true;
                        return;
                }