Merge lp:~thedac/charm-helpers/nsx-updates into lp:charm-helpers

Proposed by David Ames
Status: Merged
Merged at revision: 619
Proposed branch: lp:~thedac/charm-helpers/nsx-updates
Merge into: lp:charm-helpers
Diff against target: 59 lines (+17/-0)
2 files modified
charmhelpers/contrib/openstack/neutron.py (+4/-0)
tests/contrib/openstack/test_neutron_utils.py (+13/-0)
To merge this branch: bzr merge lp:~thedac/charm-helpers/nsx-updates
Reviewer Review Type Date Requested Status
Ryan Beisner (community) Approve
Billy Olsen Approve
Review via email: mp+302863@code.launchpad.net

Commit message

Add support for Mitaka NSX

Update Neutron plugins for NSX on Mitaka

Description of the change

Update Neutron plugins for NSX on Mitaka

To post a comment you must log in.
Revision history for this message
Billy Olsen (billy-olsen) wrote :

This looks good to me. +1.

review: Approve
Revision history for this message
Ryan Beisner (1chb1n) :
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-07-06 14:41:05 +0000
+++ charmhelpers/contrib/openstack/neutron.py 2016-08-12 22:59:45 +0000
@@ -245,6 +245,10 @@
245 'networking_plumgrid.neutron.plugins.plugin.NeutronPluginPLUMgridV2')245 'networking_plumgrid.neutron.plugins.plugin.NeutronPluginPLUMgridV2')
246 plugins['plumgrid']['server_packages'].remove(246 plugins['plumgrid']['server_packages'].remove(
247 'neutron-plugin-plumgrid')247 'neutron-plugin-plumgrid')
248 if release >= 'mitaka':
249 plugins['nsx']['server_packages'].remove('neutron-plugin-vmware')
250 plugins['nsx']['server_packages'].append('python-vmware-nsx')
251 plugins['nsx']['config'] = '/etc/neutron/nsx.ini'
248 return plugins252 return plugins
249253
250254
251255
=== modified file 'tests/contrib/openstack/test_neutron_utils.py'
--- tests/contrib/openstack/test_neutron_utils.py 2016-02-10 14:07:13 +0000
+++ tests/contrib/openstack/test_neutron_utils.py 2016-08-12 22:59:45 +0000
@@ -69,6 +69,8 @@
69 '/etc/neutron/plugins/ml2/ml2_conf.ini')69 '/etc/neutron/plugins/ml2/ml2_conf.ini')
70 self.assertEquals(plugins['nvp']['config'],70 self.assertEquals(plugins['nvp']['config'],
71 '/etc/neutron/plugins/vmware/nsx.ini')71 '/etc/neutron/plugins/vmware/nsx.ini')
72 self.assertTrue('neutron-plugin-vmware' in
73 plugins['nvp']['server_packages'])
72 self.assertEquals(plugins['n1kv']['config'],74 self.assertEquals(plugins['n1kv']['config'],
73 '/etc/neutron/plugins/cisco/cisco_plugins.ini')75 '/etc/neutron/plugins/cisco/cisco_plugins.ini')
74 self.assertEquals(plugins['Calico']['config'],76 self.assertEquals(plugins['Calico']['config'],
@@ -77,6 +79,7 @@
77 '/etc/neutron/plugins/plumgrid/plumgrid.ini')79 '/etc/neutron/plugins/plumgrid/plumgrid.ini')
78 self.assertEquals(plugins['midonet']['config'],80 self.assertEquals(plugins['midonet']['config'],
79 '/etc/neutron/plugins/midonet/midonet.ini')81 '/etc/neutron/plugins/midonet/midonet.ini')
82
80 self.assertEquals(plugins['nvp']['services'], [])83 self.assertEquals(plugins['nvp']['services'], [])
81 self.assertEquals(plugins['nsx'], plugins['nvp'])84 self.assertEquals(plugins['nsx'], plugins['nvp'])
8285
@@ -84,6 +87,8 @@
84 plugins = neutron.neutron_plugins()87 plugins = neutron.neutron_plugins()
85 self.assertEquals(plugins['midonet']['driver'],88 self.assertEquals(plugins['midonet']['driver'],
86 'neutron.plugins.midonet.plugin.MidonetPluginV2')89 'neutron.plugins.midonet.plugin.MidonetPluginV2')
90 self.assertEquals(plugins['nsx']['config'],
91 '/etc/neutron/plugins/vmware/nsx.ini')
8792
88 self.os_release.return_value = 'liberty'93 self.os_release.return_value = 'liberty'
89 self.config.return_value = 'mem-1.9'94 self.config.return_value = 'mem-1.9'
@@ -93,6 +98,14 @@
93 self.assertTrue('python-networking-midonet' in98 self.assertTrue('python-networking-midonet' in
94 plugins['midonet']['server_packages'])99 plugins['midonet']['server_packages'])
95100
101 self.os_release.return_value = 'mitaka'
102 self.config.return_value = 'mem-1.9'
103 plugins = neutron.neutron_plugins()
104 self.assertEquals(plugins['nsx']['config'],
105 '/etc/neutron/nsx.ini')
106 self.assertTrue('python-vmware-nsx' in
107 plugins['nsx']['server_packages'])
108
96 @patch.object(neutron, 'network_manager')109 @patch.object(neutron, 'network_manager')
97 def test_neutron_plugin_attribute_quantum(self, _network_manager):110 def test_neutron_plugin_attribute_quantum(self, _network_manager):
98 self.config.return_value = 'foo'111 self.config.return_value = 'foo'

Subscribers

People subscribed via source and target branches