[PATCH] CTDB takeover clean-ups, fixes, optimisations

[Amitay Isaacs]

On Thu, Feb 23, 2017 at 4:34 PM, Martin Schwenke <martin at meltin.net> wrote:

> Attached are some improvements to the CTDB takeover code.
>
> This also moves lib/util/bitmap.c to samba-util-core so it is available
> to CTDB.
>
> Please review and maybe push...
>
>
* ctdb-takeover: Generalise error handling for GET_PUBLIC_IPS

Following hunk is not required.  You don't need to test the framework.

+        if (! found_errors) {
+            D_ERR("control GET_PUBLIC_IPS internal error, "
                   "ret=%d\n", ret);
+            err_any = EIO;
         }

* ctdb-takeover: Fetch public IP addresses from all connected nodes

Change the commit message to "Fetch known public IP ...".

The side-effect of this patch is that an already banned node can get banned
again now that you have added banning on get_public_ips failure.  I guess
it will get re-banned even if it fails to reply to redundant releases.

Amitay.