diff options
| author | Alan Conway <aconway@apache.org> | 2013-02-25 20:08:05 +0000 |
|---|---|---|
| committer | Alan Conway <aconway@apache.org> | 2013-02-25 20:08:05 +0000 |
| commit | 3c09b0a8be811f899a57d62f26ab6bde3631b87f (patch) | |
| tree | 23d8fe85d2b8ca331dd35b23f19f30a30da2f821 | |
| parent | 7777cf9b68d74dd3d90a9646e5afcfcb1a69dfe1 (diff) | |
| download | qpid-python-3c09b0a8be811f899a57d62f26ab6bde3631b87f.tar.gz | |
Revert "Bug 891689 - New HA regularly shutting down active node"
This reverts commit r1449832, the commit comment did not include
a QPID JIRA number. A corrected version of the commit follows...
git-svn-id: https://svn.apache.org/repos/asf/qpid/trunk/qpid@1449869 13f79535-47bb-0310-9956-ffa450edef68
| -rwxr-xr-x | cpp/etc/qpidd-primary.in | 2 | ||||
| -rw-r--r-- | cpp/src/qpid/ha/types.cpp | 5 | ||||
| -rwxr-xr-x | tools/src/py/qpid-ha | 7 |
3 files changed, 2 insertions, 12 deletions
diff --git a/cpp/etc/qpidd-primary.in b/cpp/etc/qpidd-primary.in index da0c0c2771..39700bead3 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 --is-primary ; then + if $QPID_HA -b localhost:$QPID_PORT status --expect=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 bb4bf83574..4010ec03a8 100644 --- a/cpp/src/qpid/ha/types.cpp +++ b/cpp/src/qpid/ha/types.cpp @@ -56,11 +56,6 @@ 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" }; diff --git a/tools/src/py/qpid-ha b/tools/src/py/qpid-ha index 3d56f24fb8..4414623855 100755 --- a/tools/src/py/qpid-ha +++ b/tools/src/py/qpid-ha @@ -88,13 +88,8 @@ class StatusCmd(Command): Command.__init__(self, "status", "Print HA status") self.op.add_option( "--expect", type="string", metavar="<status>", - help="Don't print status. Return 0 if it matches <status>, 1 otherwise") - self.op.add_option( - "--is-primary", action="store_true", default=False, - help="Don't print status. Return 0 if the broker is primary, 1 otherwise") + help="Don't print status but return 0 if it matches <status>, 1 otherwise") def do_execute(self, qmf_broker, ha_broker, opts, args): - if opts.is_primary: - if not ha_broker.status in ["active", "recovering"]: raise ExitStatus(1) if opts.expect: if opts.expect != ha_broker.status: raise ExitStatus(1) else: |