The branch, master has been updated
       via  5833e6b99d9afaf35dc8354df8676b9115418b23 (commit)
      from  cb14ee57dd0a589242da1ac2830bb7939df460a5 (commit)

http://gitweb.samba.org/?p=sahlberg/ctdb.git;a=shortlog;h=master


- Log -----------------------------------------------------------------
commit 5833e6b99d9afaf35dc8354df8676b9115418b23
Author: Ronnie Sahlberg <[EMAIL PROTECTED]>
Date:   Fri May 9 13:41:31 2008 +1000

    fix a bug where the public ip addresses of the cluster would not be 
redistributed across the cluster after a recovery was performed.
    
    Remove a bogus check inside the recovery daemon that ONLY redistributed 
public addresses IFF the local node had/served public addresses.
    This was a valid optimization long ago when we enforced that all nodes must 
use the same public addresses file   but is invalid today where we can have 
different public addresses configs on all nodes  and even have some nodes that 
do NOT use public addresses at all.

-----------------------------------------------------------------------

Summary of changes:
 server/ctdb_recoverd.c |   16 ++++++----------
 1 files changed, 6 insertions(+), 10 deletions(-)


Changeset truncated at 500 lines:

diff --git a/server/ctdb_recoverd.c b/server/ctdb_recoverd.c
index a40dfbb..95dd94f 100644
--- a/server/ctdb_recoverd.c
+++ b/server/ctdb_recoverd.c
@@ -1452,18 +1452,14 @@ static int do_recovery(struct ctdb_recoverd *rec,
        DEBUG(DEBUG_NOTICE, (__location__ " Recovery - updated flags\n"));
 
        /*
-         if enabled, tell nodes to takeover their public IPs
+         tell nodes to takeover their public IPs
         */
-       if (ctdb->vnn) {
-               rec->need_takeover_run = false;
-               ret = ctdb_takeover_run(ctdb, nodemap);
-               if (ret != 0) {
-                       DEBUG(DEBUG_ERR, (__location__ " Unable to setup public 
takeover addresses\n"));
-                       return -1;
-               }
-               DEBUG(DEBUG_INFO, (__location__ " Recovery - done takeover\n"));
+       rec->need_takeover_run = false;
+       ret = ctdb_takeover_run(ctdb, nodemap);
+       if (ret != 0) {
+               DEBUG(DEBUG_ERR, (__location__ " Unable to setup public 
takeover addresses\n"));
+               return -1;
        }
-
        DEBUG(DEBUG_NOTICE, (__location__ " Recovery - takeip finished\n"));
 
        /* execute the "recovered" event script on all nodes */


-- 
CTDB repository

Reply via email to