ctdb-recoverd: Drop unnecessary and broken code
authorMartin Schwenke <martin@meltin.net>
Tue, 16 Jan 2018 04:15:51 +0000 (15:15 +1100)
committerKarolin Seeger <kseeger@samba.org>
Tue, 27 Oct 2020 08:22:01 +0000 (08:22 +0000)
commit710b287bdc99eb40b31eed6f952d9cd83f7e64a3
tree19484912825e4e1f91024de3796d9270a0b1f441
parent895bc9a7c2aa6fab2029618d1267d97f705c4b1b
ctdb-recoverd: Drop unnecessary and broken code

update_flags() has already updated the recovery master's canonical
node map, based on the flags from each remote node, and pushed out
these flags to all nodes.

If i == j then the node map has already been updated from this remote
node's flags, so simply drop this case.

Although update_flags() has updated flags for all nodes, it did not
update each node map in remote_nodemaps[] to reflect this.  This means
that remote_nodemaps[] may contain inconsistent flags for some nodes
so it should not be used to check consistency when i != j.

Further, a meaningful difference in flags can only really occur if
update_flags() failed.  In that case this code is never reached.

These observations combine to imply that this whole loop should be
dropped.

This leaves potential sub-second inconsistencies due to out-of-band
healthy/unhealthy flag changes pushed via CTDB_SRVID_PUSH_NODE_FLAGS.
These updates could be dropped (takeover run asks each node for
available IPs rather than making centralised decisions based on node
flags) but for now they will be fixed in the next iteration of
main_loop().

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 4b01f54041dee469971f244e64064eed46de2ed5)
ctdb/server/ctdb_recoverd.c