Qpid, RabbitMQ: advance thru the list of brokers on reconnect
When using multiple brokers, we don't want to immediately retry failed
connection for the same failed broker. This was not the case in
existing implementation though, where we've always attempted to
reconnect starting from the first broker in the list of candidates. So
if the first broker failed, we initiated reconnect to the same failed
broker.
This change makes reconnect() implementation to select the next broker
in the list. This also means that non-failure reconnect attempts will
also switch the current broker. All in all, users should not rely on
any particular order to use brokers from the list, so this should not
constitute an issue.
This sync includes the following fixes for HA deployments:
* Ia148baa6e1ec632789ac3621c85173c2c16f3918
* I67923cb024bbd143edc8edccf35b9b400df31eb3
Reviewed: https:/ /review. openstack. org/83460 /git.openstack. org/cgit/ openstack/ neutron/ commit/ ?id=eb4c0458136 139561e6286fce0 ad81d542dd0d9d
Committed: https:/
Submitter: Jenkins
Branch: stable/havana
commit eb4c04581361395 61e6286fce0ad81 d542dd0d9d
Author: Ihar Hrachyshka <email address hidden>
Date: Thu Mar 27 17:19:32 2014 +0100
Qpid, RabbitMQ: advance thru the list of brokers on reconnect
When using multiple brokers, we don't want to immediately retry failed
connection for the same failed broker. This was not the case in
existing implementation though, where we've always attempted to
reconnect starting from the first broker in the list of candidates. So
if the first broker failed, we initiated reconnect to the same failed
broker.
This change makes reconnect() implementation to select the next broker
in the list. This also means that non-failure reconnect attempts will
also switch the current broker. All in all, users should not rely on
any particular order to use brokers from the list, so this should not
constitute an issue.
This sync includes the following fixes for HA deployments: 2789ac3621c8517 3c2c16f3918 43edc8edccf35b9 b400df31eb3
* Ia148baa6e1ec63
* I67923cb024bbd1
Change-Id: I77be95dd2db454 6297d5970320554 59a12b52cfa
Closes-Bug: 1261631