openstack/charm-neutron-gateway
Code Issues Proposed changes

Merge "CH Sync for rabbit relation bug fix"

Browse Source
This commit is contained in:
Zuul 2020-07-03 06:50:14 +00:00 committed by Gerrit Code Review
commit 5840fe1e24
1 changed files with 19 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
hooks/charmhelpers/contrib/openstack/context.py
View File

@ -721,6 +721,12 @@ class AMQPContext(OSContextGenerator):
rabbitmq_hosts = [] rabbitmq_hosts = []
for unit in related_units(rid): for unit in related_units(rid):
host = relation_get('private-address', rid=rid, unit=unit) host = relation_get('private-address', rid=rid, unit=unit)
if not relation_get('password', rid=rid, unit=unit):
log(
("Skipping {} password not sent which indicates "
"unit is not ready.".format(host)),
level=DEBUG)
continue
host = format_ipv6_addr(host) or host host = format_ipv6_addr(host) or host
rabbitmq_hosts.append(host) rabbitmq_hosts.append(host)
@ -2714,6 +2720,19 @@ class BridgePortInterfaceMap(object):
self._ifname_mac_map[ifname] = [mac] self._ifname_mac_map[ifname] = [mac]
self._mac_ifname_map[mac] = ifname self._mac_ifname_map[mac] = ifname
# check if interface is part of a linux bond
_bond_name = get_bond_master(ifname)
if _bond_name and _bond_name != ifname:
log('Add linux bond "{}" to map for physical interface "{}" '
'with mac "{}".'.format(_bond_name, ifname, mac),
level=DEBUG)
# for bonds we want to be able to get a list of the mac
# addresses for the physical interfaces the bond is made up of.
if self._ifname_mac_map.get(_bond_name):
self._ifname_mac_map[_bond_name].append(mac)
else:
self._ifname_mac_map[_bond_name] = [mac]
# In light of the pre-deprecation notice in the docstring of this # In light of the pre-deprecation notice in the docstring of this
# class we will expose the ability to configure OVS bonds as a # class we will expose the ability to configure OVS bonds as a
# DPDK-only feature, but generally use the data structures internally. # DPDK-only feature, but generally use the data structures internally.