Merge lp://qastaging/~niedbalski/charm-helpers/lp1671861 into lp://qastaging/charm-helpers
Proposed by
Jorge Niedbalski
Status: | Merged | ||||
---|---|---|---|---|---|
Merged at revision: | 713 | ||||
Proposed branch: | lp://qastaging/~niedbalski/charm-helpers/lp1671861 | ||||
Merge into: | lp://qastaging/charm-helpers | ||||
Diff against target: |
106 lines (+47/-16) 2 files modified
charmhelpers/contrib/network/ip.py (+32/-16) tests/contrib/network/test_ip.py (+15/-0) |
||||
To merge this branch: | bzr merge lp://qastaging/~niedbalski/charm-helpers/lp1671861 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Felipe Reyes (community) | Approve | ||
Review via email: mp+319595@code.qastaging.launchpad.net |
Description of the change
Fix required for solving LP: #1671861.
- Extends the get_address_
to use a prefix when the netmask comes in the 'ffff:ffff:
- Link-local and loopback addresses are skipped
- Added tests to validate the change.
To post a comment you must log in.
the tests are OK, and the code looks code to me.
I tested the code against my local ipv6 address and does the right thing detecting them.