Merge lp:~james-page/charm-helpers/fixup-midonet-liberty-nonmem into lp:charm-helpers

Proposed by James Page
Status: Merged
Merged at revision: 532
Proposed branch: lp:~james-page/charm-helpers/fixup-midonet-liberty-nonmem
Merge into: lp:charm-helpers
Diff against target: 22 lines (+6/-8)
1 file modified
charmhelpers/contrib/openstack/neutron.py (+6/-8)
To merge this branch: bzr merge lp:~james-page/charm-helpers/fixup-midonet-liberty-nonmem
Reviewer Review Type Date Requested Status
Charles Butler (community) Approve
Review via email: mp+286524@code.launchpad.net

Description of the change

Ensure that correct networking package is used for liberty generally, not just for mem.

To post a comment you must log in.
Revision history for this message
Charles Butler (lazypower) wrote :

+1 LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'charmhelpers/contrib/openstack/neutron.py'
--- charmhelpers/contrib/openstack/neutron.py 2016-02-09 10:38:19 +0000
+++ charmhelpers/contrib/openstack/neutron.py 2016-02-18 15:22:12 +0000
@@ -237,14 +237,12 @@
237 plugins['midonet']['driver'] = (237 plugins['midonet']['driver'] = (
238 'neutron.plugins.midonet.plugin.MidonetPluginV2')238 'neutron.plugins.midonet.plugin.MidonetPluginV2')
239 if release >= 'liberty':239 if release >= 'liberty':
240 midonet_origin = config('midonet-origin')240 plugins['midonet']['driver'] = (
241 if midonet_origin is not None and midonet_origin[4:5] == '1':241 'midonet.neutron.plugin_v1.MidonetPluginV2')
242 plugins['midonet']['driver'] = (242 plugins['midonet']['server_packages'].remove(
243 'midonet.neutron.plugin_v1.MidonetPluginV2')243 'python-neutron-plugin-midonet')
244 plugins['midonet']['server_packages'].remove(244 plugins['midonet']['server_packages'].append(
245 'python-neutron-plugin-midonet')245 'python-networking-midonet')
246 plugins['midonet']['server_packages'].append(
247 'python-networking-midonet')
248 return plugins246 return plugins
249247
250248

Subscribers

People subscribed via source and target branches