diff options
| author | Amitay Isaacs <amitay@gmail.com> | 2015-10-06 17:31:41 +1100 |
|---|---|---|
| committer | Amitay Isaacs <amitay@samba.org> | 2015-10-07 17:55:05 +0200 |
| commit | 01017482876defa74137295fa08ed7f27fc12f61 (patch) | |
| tree | 34016dbf3d3874a5431bf2385a66bd9063988dc7 /ctdb/server | |
| parent | 3cf93d91361191a7accdf0ddf5e811212e0a5ca2 (diff) | |
| download | samba-01017482876defa74137295fa08ed7f27fc12f61.tar.gz | |
ctdb-recoverd: Always check for recmaster before doing recovery
Recovery daemon checks if it is the recovery master before performing
certain checks. During those checks it's possible that re-election can
change the recmaster. In such a case, the recovery daemon should never
do a database recovery.
This is not complete fix since the recovery master can still change
while the recovery is going on. The correct fix is to abort recovery
if the recovery master changes.
Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Martin Schwenke <martin@meltin.net>
Autobuild-User(master): Amitay Isaacs <amitay@samba.org>
Autobuild-Date(master): Wed Oct 7 17:55:05 CEST 2015 on sn-devel-104
Diffstat (limited to 'ctdb/server')
| -rw-r--r-- | ctdb/server/ctdb_recoverd.c | 18 |
1 files changed, 18 insertions, 0 deletions
diff --git a/ctdb/server/ctdb_recoverd.c b/ctdb/server/ctdb_recoverd.c index 3c7f7f449cf..116531800cb 100644 --- a/ctdb/server/ctdb_recoverd.c +++ b/ctdb/server/ctdb_recoverd.c @@ -2103,6 +2103,24 @@ static int do_recovery(struct ctdb_recoverd *rec, DEBUG(DEBUG_NOTICE, (__location__ " Starting do_recovery\n")); + /* Check if the current node is still the recmaster. It's possible that + * re-election has changed the recmaster, but we have not yet updated + * that information. + */ + ret = ctdb_ctrl_getrecmaster(ctdb, mem_ctx, CONTROL_TIMEOUT(), + pnn, &ctdb->recovery_master); + if (ret != 0) { + DEBUG(DEBUG_ERR, (__location__ " Unable to get recmaster\n")); + return -1; + } + + if (pnn != ctdb->recovery_master) { + DEBUG(DEBUG_NOTICE, + ("Recovery master changed to %u, aborting recovery\n", + ctdb->recovery_master)); + return -1; + } + /* if recovery fails, force it again */ rec->need_recovery = true; |