openstack/charm-neutron-openvswitch
Code Issues Proposed changes

Hotfix charmhelpers sync for local_address() fix

Browse Source 
The network-get --primary-address juju-info fails on pre-2.8.?
versions of juju.  This results in a NoNetworkBinding error.
Fallback to unit_get() if that occurs for local_address().

Change-Id: I37d21fbaf951b1fe5bc03306b67012ac37b2e75b
This commit is contained in:
Alex Kavanagh 2021-01-20 12:25:18 +00:00 committed by Aurelien Lourot
parent 840e255325
commit 292876f387
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
hooks/charmhelpers/contrib/openstack/ip.py
View File

@ -130,6 +130,9 @@ def local_address(unit_get_fallback='public-address'):
from network-get. If this is running with an old version of Juju then from network-get. If this is running with an old version of Juju then
fallback to unit_get. fallback to unit_get.
Note on juju < 2.9 the binding to juju-info may not exist, so fall back to
the unit-get.
:param unit_get_fallback: Either 'public-address' or 'private-address'. :param unit_get_fallback: Either 'public-address' or 'private-address'.
Only used with old versions of Juju. Only used with old versions of Juju.
:type unit_get_fallback: str :type unit_get_fallback: str
@ -138,7 +141,7 @@ def local_address(unit_get_fallback='public-address'):
""" """
try: try:
return network_get_primary_address('juju-info') return network_get_primary_address('juju-info')
except NotImplementedError: except (NotImplementedError, NoNetworkBinding):
return unit_get(unit_get_fallback) return unit_get(unit_get_fallback)