From mboxrd@z Thu Jan  1 00:00:00 1970
From: pcaulfield@sourceware.org <pcaulfield@sourceware.org>
Date: 26 Jun 2006 08:42:07 -0000
Subject: [Cluster-devel] cluster/cman-kernel/src membership.c
Message-ID: <20060626084207.18384.qmail@sourceware.org>
List-Id: <cluster-devel.redhat.com>
To: cluster-devel.redhat.com
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit

CVSROOT:	/cvs/cluster
Module name:	cluster
Branch: 	RHEL4
Changes by:	pcaulfield at sourceware.org	2006-06-26 08:42:06

Modified files:
	cman-kernel/src: membership.c 

Log message:
	If we get a Master-HELLO and we are not the master for this transition then
	kick off a new one to resolve the ambiguity.
	
	I reckon this was probably the cause of the first trouble in bz#194491

Patches:
http://sourceware.org/cgi-bin/cvsweb.cgi/cluster/cman-kernel/src/membership.c.diff?cvsroot=cluster&only_with_tag=RHEL4&r1=1.44.2.23&r2=1.44.2.24

--- cluster/cman-kernel/src/Attic/membership.c	2006/05/09 09:26:31	1.44.2.23
+++ cluster/cman-kernel/src/Attic/membership.c	2006/06/26 08:42:06	1.44.2.24
@@ -2882,6 +2882,13 @@
 		}
 	}
 
+	/* If we get a master hello and we are not the master then start a CHECK transition, cos the
+	   real master must have gone away in a period of confusion */
+	if (node_state != MASTER && hellomsg->flags & HELLO_FLAG_MASTER) {
+		node = find_node_by_nodeid(saddr->scl_nodeid);
+		start_transition(TRANS_CHECK, node);
+	}
+
 	return 0;
 
 }