From 7777cf9b68d74dd3d90a9646e5afcfcb1a69dfe1 Mon Sep 17 00:00:00 2001
From: Alan Conway <aconway@apache.org>
Date: Mon, 25 Feb 2013 19:43:29 +0000
Subject: Bug 891689 - New HA regularly shutting down active node

qpid-primary script was incorrect and failing on status calls,
causing the broker to be restarted by rgmanager.

git-svn-id: https://svn.apache.org/repos/asf/qpid/trunk/qpid@1449832 13f79535-47bb-0310-9956-ffa450edef68
---
 cpp/etc/qpidd-primary.in  | 2 +-
 cpp/src/qpid/ha/types.cpp | 5 +++++
 2 files changed, 6 insertions(+), 1 deletion(-)

(limited to 'cpp')

diff --git a/cpp/etc/qpidd-primary.in b/cpp/etc/qpidd-primary.in
index 39700bead3..da0c0c2771 100755
--- a/cpp/etc/qpidd-primary.in
+++ b/cpp/etc/qpidd-primary.in
@@ -57,7 +57,7 @@ if [[ !(-x $QPID_HA) ]]; then
 fi
 
 status() {
-    if $QPID_HA -b localhost:$QPID_PORT status --expect=primary ; then
+    if $QPID_HA -b localhost:$QPID_PORT status --is-primary ; then
 	echo "qpidd is primary"
     else
 	echo "qpidd is not primary"
diff --git a/cpp/src/qpid/ha/types.cpp b/cpp/src/qpid/ha/types.cpp
index 4010ec03a8..bb4bf83574 100644
--- a/cpp/src/qpid/ha/types.cpp
+++ b/cpp/src/qpid/ha/types.cpp
@@ -56,6 +56,11 @@ template <> const char* Enum<ReplicateLevel>::NAMES[] = { "none", "configuration
 template <> const size_t Enum<ReplicateLevel>::N = 3;
 
 template <> const char* Enum<BrokerStatus>::NAME = "HA broker status";
+
+// NOTE: Changing status names will  have an impact on qpid-ha and
+// the qpidd-primary init script.
+// Don't change them unless you are going to  update all dependent code.
+//
 template <> const char* Enum<BrokerStatus>::NAMES[] = {
     "joining", "catchup", "ready", "recovering", "active", "standalone"
 };
-- 
cgit v1.2.1