Merge lp:~peter-sabaini/charm-helpers/charm-helpers-get_address_in_network into lp:charm-helpers

Proposed by Peter Sabaini
Status: Merged
Merged at revision: 731
Proposed branch: lp:~peter-sabaini/charm-helpers/charm-helpers-get_address_in_network
Merge into: lp:charm-helpers
Diff against target: 20 lines (+5/-5)
1 file modified
charmhelpers/contrib/network/ip.py (+5/-5)
To merge this branch: bzr merge lp:~peter-sabaini/charm-helpers/charm-helpers-get_address_in_network
Reviewer Review Type Date Requested Status
Jorge Niedbalski (community) Approve
Review via email: mp+321563@code.launchpad.net

Description of the change

 When searching for network, check all ipv4 addr for an iface

To post a comment you must log in.
Revision history for this message
Jorge Niedbalski (niedbalski) wrote :

LGTM, I added an additional unit test to cover your change.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'charmhelpers/contrib/network/ip.py'
2--- charmhelpers/contrib/network/ip.py 2017-03-24 16:50:31 +0000
3+++ charmhelpers/contrib/network/ip.py 2017-03-31 13:26:40 +0000
4@@ -111,11 +111,11 @@
5 for iface in netifaces.interfaces():
6 addresses = netifaces.ifaddresses(iface)
7 if network.version == 4 and netifaces.AF_INET in addresses:
8- addr = addresses[netifaces.AF_INET][0]['addr']
9- netmask = addresses[netifaces.AF_INET][0]['netmask']
10- cidr = netaddr.IPNetwork("%s/%s" % (addr, netmask))
11- if cidr in network:
12- return str(cidr.ip)
13+ for addr in addresses[netifaces.AF_INET]:
14+ cidr = netaddr.IPNetwork("%s/%s" % (addr['addr'],
15+ addr['netmask']))
16+ if cidr in network:
17+ return str(cidr.ip)
18
19 if network.version == 6 and netifaces.AF_INET6 in addresses:
20 for addr in addresses[netifaces.AF_INET6]:

Subscribers

People subscribed via source and target branches