diff options
| author | Alan Conway <aconway@apache.org> | 2013-05-15 21:53:55 +0000 |
|---|---|---|
| committer | Alan Conway <aconway@apache.org> | 2013-05-15 21:53:55 +0000 |
| commit | 93f86c53a7d9e2d16e50bd578c2df10763ab8f86 (patch) | |
| tree | 522a042d6cb8bb969e5892819a4a5528ce322cec /cpp/src/tests | |
| parent | 39eb881916826f9e70ef6c1fe32f805d7c4838c3 (diff) | |
| download | qpid-python-93f86c53a7d9e2d16e50bd578c2df10763ab8f86.tar.gz | |
QPID-4850: HA Sporadic stall in ha_tests.test_failover_send_receive
Fixed a deadlock in the test: test was killing a backup broker then waiting to
pass a checkpoint: however if the backup was expected by the primary, the primary
holds queues till it connects, so we deadlock.
git-svn-id: https://svn.apache.org/repos/asf/qpid/trunk/qpid@1483107 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'cpp/src/tests')
| -rwxr-xr-x | cpp/src/tests/ha_tests.py | 8 |
1 files changed, 1 insertions, 7 deletions
diff --git a/cpp/src/tests/ha_tests.py b/cpp/src/tests/ha_tests.py index 3f870ba5b8..8b4ff80dd0 100755 --- a/cpp/src/tests/ha_tests.py +++ b/cpp/src/tests/ha_tests.py @@ -972,19 +972,13 @@ class LongTests(HaBrokerTest): brokers.bounce(victim) # Next one is promoted primary = next else: - brokers.kill(victim, promote_next=False, final=False) - dead = victim + brokers.bounce(victim, promote_next=False) # Make sure we are not stalled map(wait_passed, receivers, checkpoint) # Run another checkpoint to ensure things work in this configuration checkpoint = [ r.received+100 for r in receivers ] map(wait_passed, receivers, checkpoint) - - if dead is not None: - brokers.restart(dead) # Restart backup - brokers[dead].ready() - dead = None i += 1 except: traceback.print_exc() |