From b18a22b820822e6d089ca64af90b53e89edda9a6 Mon Sep 17 00:00:00 2001 From: Ronnie Sahlberg Date: Tue, 29 Nov 2011 14:38:02 +1100 Subject: This breaks the build since the recovery loop is different in master compared to old 1.0 branches This must have been mistakenly applied to master when you intended to push for a different branch i guess. Revert "recoverd: try to become the recovery master if we have the capability, but the current master doesn't" This reverts commit a97d417aba85e901540147a4dff4794249442939. (This used to be ctdb commit c19cb751077b78cf4b6e28a1e3746d4ffedbfd68) --- ctdb/server/ctdb_recoverd.c | 23 +---------------------- 1 file changed, 1 insertion(+), 22 deletions(-) diff --git a/ctdb/server/ctdb_recoverd.c b/ctdb/server/ctdb_recoverd.c index 36abfd4d21..fa4b6ba882 100644 --- a/ctdb/server/ctdb_recoverd.c +++ b/ctdb/server/ctdb_recoverd.c @@ -3010,13 +3010,6 @@ static void main_loop(struct ctdb_context *ctdb, struct ctdb_recoverd *rec, } nodemap = rec->nodemap; - /* update the capabilities for all nodes */ - ret = update_capabilities(ctdb, nodemap); - if (ret != 0) { - DEBUG(DEBUG_ERR, (__location__ " Unable to update node capabilities.\n")); - goto again; - } - /* check which node is the recovery master */ ret = ctdb_ctrl_getrecmaster(ctdb, mem_ctx, CONTROL_TIMEOUT(), pnn, &rec->recmaster); if (ret != 0) { @@ -3039,6 +3032,7 @@ static void main_loop(struct ctdb_context *ctdb, struct ctdb_recoverd *rec, return; } + /* if the local daemon is STOPPED, we verify that the databases are also frozen and thet the recmode is set to active */ @@ -3073,21 +3067,6 @@ static void main_loop(struct ctdb_context *ctdb, struct ctdb_recoverd *rec, return; } - /* - * if the current recmaster do not have CTDB_CAP_RECMASTER, - * but we have force an election and try to become the new - * recmaster - */ - if ((rec->ctdb->nodes[rec->recmaster]->capabilities & CTDB_CAP_RECMASTER) == 0 && - (rec->ctdb->capabilities & CTDB_CAP_RECMASTER) && - !(nodemap->nodes[pnn].flags & NODE_FLAGS_INACTIVE)) { - DEBUG(DEBUG_ERR, (__location__ " Current recmaster node %u does not have CAP_RECMASTER," - " but we (node %u) have - force an election\n", - rec->recmaster, pnn)); - force_election(rec, pnn, nodemap); - goto again; - } - /* check that we (recovery daemon) and the local ctdb daemon agrees on whether we are banned or not */ -- cgit