Properly reconnect subscribing clients when QPID broker restarts
When the QPID broker is restarted (or fails over), subscribed clients
will attempt to re-establish their connections. In the case of fanout
subscriptions, this reconnection functionality is broken. For version
1 topologies, the clients attempt to reconnect twice to the same
exclusive address - which is illegal. In the case of version 2
topologies, the address parsing is broken and an illegal address is
created on reconnect. This fix avoids the problem by removing the
special-case reconnect code that manages UUID addresses; it is
unnecessary as the QPID broker will generate unique queue names
automatically when the clients reconnect.
Closes-bug: #1251757
Co-authored-by: Mark McLoughlin <email address hidden>
Change-Id: I6051fb503663bb8c7c5468db6bcde10f6cf1b318
(cherry picked from commit e227c0ed7e0ed1f9b8d029336f8aeb60e38c23df)
Reviewed: https:/ /review. openstack. org/60280 github. com/openstack/ oslo-incubator/ commit/ 55678c7ab7fbc1b 2beb428d046bb41 62f23b0a53
Committed: http://
Submitter: Jenkins
Branch: stable/havana
commit 55678c7ab7fbc1b 2beb428d046bb41 62f23b0a53
Author: Kenneth Giusti <email address hidden>
Date: Tue Dec 3 10:39:49 2013 +0000
Properly reconnect subscribing clients when QPID broker restarts
When the QPID broker is restarted (or fails over), subscribed clients
will attempt to re-establish their connections. In the case of fanout
subscriptions, this reconnection functionality is broken. For version
1 topologies, the clients attempt to reconnect twice to the same
exclusive address - which is illegal. In the case of version 2
topologies, the address parsing is broken and an illegal address is
created on reconnect. This fix avoids the problem by removing the
special-case reconnect code that manages UUID addresses; it is
unnecessary as the QPID broker will generate unique queue names
automatically when the clients reconnect.
Closes-bug: #1251757 8c7c5468db6bcde 10f6cf1b318 9b8d029336f8aeb 60e38c23df)
Co-authored-by: Mark McLoughlin <email address hidden>
Change-Id: I6051fb503663bb
(cherry picked from commit e227c0ed7e0ed1f