Comment 44 for bug 1261631

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to heat (stable/havana)

Reviewed: https://review.openstack.org/107789
Committed: https://git.openstack.org/cgit/openstack/heat/commit/?id=2a5b4a78f3adb2b180b157532247ae0a87159569
Submitter: Jenkins
Branch: stable/havana

commit 2a5b4a78f3adb2b180b157532247ae0a87159569
Author: Steven Hardy <email address hidden>
Date: Thu Jul 17 19:24:03 2014 +0100

    Qpid: advance thru the list of brokers on reconnect

    Sync 877f1cab328fe8a9f8214687ea7e1287862de87d from oslo-incubator's
    stable/havana branch.

    In Qpid implementation, when using multiple qpid_hosts, 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.

    Change-Id: I5a630e76880c0d4219c4a8a2b3354e41899d9695
    Partial-Bug: 1261631