ctdb-recoverd: Rename update_local_flags() -> update_flags()
authorMartin Schwenke <martin@meltin.net>
Tue, 23 Jan 2018 23:21:37 +0000 (10:21 +1100)
committerAmitay Isaacs <amitay@samba.org>
Tue, 18 Aug 2020 05:02:25 +0000 (05:02 +0000)
This also updates remote flags so the name is misleading.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=14466
Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
ctdb/server/ctdb_recoverd.c

index 3c355e105651a5e46821f5136411fdcba9d2acca..f825427e7a39a8bf620666ac0a774a02bef5f71b 100644 (file)
@@ -541,9 +541,9 @@ static void ctdb_wait_election(struct ctdb_recoverd *rec)
  * flags changes to all nodes.  This is only run by the recovery
  * master.
  */
-static int update_local_flags(struct ctdb_recoverd *rec,
-                             struct ctdb_node_map_old *nodemap,
-                             struct ctdb_node_map_old **remote_nodemaps)
+static int update_flags(struct ctdb_recoverd *rec,
+                       struct ctdb_node_map_old *nodemap,
+                       struct ctdb_node_map_old **remote_nodemaps)
 {
        unsigned int j;
        struct ctdb_context *ctdb = rec->ctdb;
@@ -2554,10 +2554,10 @@ static void main_loop(struct ctdb_context *ctdb, struct ctdb_recoverd *rec,
                return;
        }
 
-       /* ensure our local copies of flags are right */
-       ret = update_local_flags(rec, nodemap, remote_nodemaps);
+       /* Ensure our local and remote flags are correct */
+       ret = update_flags(rec, nodemap, remote_nodemaps);
        if (ret != 0) {
-               DEBUG(DEBUG_ERR,("Unable to update local flags\n"));
+               D_ERR("Unable to update flags\n");
                return;
        }