Merge lp:~smoser/maas-images/trunk.fix-apply-networking into lp:maas-images

Proposed by Scott Moser
Status: Merged
Merged at revision: 379
Proposed branch: lp:~smoser/maas-images/trunk.fix-apply-networking
Merge into: lp:maas-images
Diff against target: 25 lines (+2/-2)
2 files modified
curtin/centos6/curtin-hooks.py (+1/-1)
curtin/centos7/curtin-hooks.py (+1/-1)
To merge this branch: bzr merge lp:~smoser/maas-images/trunk.fix-apply-networking
Reviewer Review Type Date Requested Status
Lee Trager (community) Approve
Andres Rodriguez (community) Approve
Review via email: mp+328467@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Andres Rodriguez (andreserl) wrote :

lgtm!

review: Approve
Revision history for this message
Lee Trager (ltrager) wrote :

Tested with generated stream at http://162.213.35.187/centos_networking/

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'curtin/centos6/curtin-hooks.py'
--- curtin/centos6/curtin-hooks.py 2017-08-01 15:58:24 +0000
+++ curtin/centos6/curtin-hooks.py 2017-08-02 19:22:30 +0000
@@ -330,7 +330,7 @@
330330
331def apply_networking(cfg, target, bootmac):331def apply_networking(cfg, target, bootmac):
332 if 'network' in cfg and centos_apply_network_config:332 if 'network' in cfg and centos_apply_network_config:
333 centos_apply_network_config(cfg, target)333 centos_apply_network_config(cfg['network'], target)
334 return334 return
335335
336 if 'network' in cfg:336 if 'network' in cfg:
337337
=== modified file 'curtin/centos7/curtin-hooks.py'
--- curtin/centos7/curtin-hooks.py 2017-08-01 15:58:24 +0000
+++ curtin/centos7/curtin-hooks.py 2017-08-02 19:22:30 +0000
@@ -319,7 +319,7 @@
319319
320def apply_networking(cfg, target, bootmac):320def apply_networking(cfg, target, bootmac):
321 if 'network' in cfg and centos_apply_network_config:321 if 'network' in cfg and centos_apply_network_config:
322 centos_apply_network_config(cfg, target)322 centos_apply_network_config(cfg['network'], target)
323 return323 return
324324
325 if 'network' in cfg:325 if 'network' in cfg:

Subscribers

People subscribed via source and target branches