x/vmware-nsx
Code Issues Proposed changes

Cisco N1kv plugin to send subtype on network profile creation

Browse Source 
Make sure that the cisco n1kv neutron plugin sends the sub_type too
when a overlay network profile is created

Change-Id: I37e63131039077fa80a28fe725c09f0307acc2ea
Closes-Bug: 1357125
This commit is contained in:
Marga Millet 2014-08-14 22:05:23 -07:00
parent 6f20bfdbdf
commit ad822190fa
2 changed files with 9 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
neutron/plugins/cisco/n1kv/n1kv_client.py
View File

@ -256,6 +256,8 @@ class Client(object):
'id': network_profile['id'], 'id': network_profile['id'],
'logicalNetwork': logical_network_name, 'logicalNetwork': logical_network_name,
'tenantId': tenant_id} 'tenantId': tenant_id}
if network_profile['segment_type'] == c_const.NETWORK_TYPE_OVERLAY:
body['subType'] = network_profile['sub_type']
return self._post( return self._post(
self.network_segment_pool_path % network_profile['id'], self.network_segment_pool_path % network_profile['id'],
body=body) body=body)

7
neutron/tests/unit/cisco/n1kv/test_n1kv_plugin.py
View File

@ -273,6 +273,13 @@ class TestN1kvNetworkProfiles(N1kvPluginTestCase):
res = net_p_req.get_response(self.ext_api) res = net_p_req.get_response(self.ext_api)
self.assertEqual(res.status_int, 201) self.assertEqual(res.status_int, 201)
def test_create_network_profile_overlay_missing_subtype(self):
data = self._prepare_net_profile_data(c_const.NETWORK_TYPE_OVERLAY)
data['network_profile'].pop('sub_type')
net_p_req = self.new_create_request('network_profiles', data)
res = net_p_req.get_response(self.ext_api)
self.assertEqual(res.status_int, 400)
def test_create_network_profile_trunk(self): def test_create_network_profile_trunk(self):
data = self._prepare_net_profile_data(c_const.NETWORK_TYPE_TRUNK) data = self._prepare_net_profile_data(c_const.NETWORK_TYPE_TRUNK)
net_p_req = self.new_create_request('network_profiles', data) net_p_req = self.new_create_request('network_profiles', data)