Merge lp:~junaidali/charms/trusty/plumgrid-gateway/mgmt_val into lp:~plumgrid-team/charms/trusty/plumgrid-gateway/trunk

Proposed by Junaid Ali
Status: Merged
Merged at revision: 31
Proposed branch: lp:~junaidali/charms/trusty/plumgrid-gateway/mgmt_val
Merge into: lp:~plumgrid-team/charms/trusty/plumgrid-gateway/trunk
Diff against target: 26 lines (+3/-2)
2 files modified
config.yaml (+0/-1)
hooks/pg_gw_utils.py (+3/-1)
To merge this branch: bzr merge lp:~junaidali/charms/trusty/plumgrid-gateway/mgmt_val
Reviewer Review Type Date Requested Status
Bilal Baqar Approve
Review via email: mp+301315@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Bilal Baqar (bbaqar) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'config.yaml'
--- config.yaml 2016-06-19 18:54:39 +0000
+++ config.yaml 2016-07-27 19:08:53 +0000
@@ -11,7 +11,6 @@
11 description: Public SSH key of PLUMgrid LCM which is running PG-Tools11 description: Public SSH key of PLUMgrid LCM which is running PG-Tools
12 mgmt-interface:12 mgmt-interface:
13 type: string13 type: string
14 default: 'juju-br0'
15 description: The interface connected to PLUMgrid Managment network.14 description: The interface connected to PLUMgrid Managment network.
16 fabric-interfaces:15 fabric-interfaces:
17 default: 'MANAGEMENT'16 default: 'MANAGEMENT'
1817
=== modified file 'hooks/pg_gw_utils.py'
--- hooks/pg_gw_utils.py 2016-06-19 18:54:39 +0000
+++ hooks/pg_gw_utils.py 2016-07-27 19:08:53 +0000
@@ -222,7 +222,9 @@
222 Returns the managment interface.222 Returns the managment interface.
223 '''223 '''
224 mgmt_interface = config('mgmt-interface')224 mgmt_interface = config('mgmt-interface')
225 if interface_exists(mgmt_interface):225 if not mgmt_interface:
226 return get_iface_from_addr(unit_get('private-address'))
227 elif interface_exists(mgmt_interface):
226 return mgmt_interface228 return mgmt_interface
227 else:229 else:
228 log('Provided managment interface %s does not exist'230 log('Provided managment interface %s does not exist'

Subscribers

People subscribed via source and target branches