x/vmware-nsx
Code Issues Proposed changes

Merge "Fix alias for ext-gw-mode extension in nicira plugin"

Browse Source
This commit is contained in:
Jenkins 2013-08-02 17:32:59 +00:00 committed by Gerrit Code Review
commit 256f9743a3
2 changed files with 26 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
neutron/plugins/nicira/NeutronPlugin.py
View File

@ -150,7 +150,7 @@ class NvpPluginV2(db_base_plugin_v2.NeutronDbPluginV2,
supported_extension_aliases = ["agent", supported_extension_aliases = ["agent",
"binding", "binding",
"dhcp_agent_scheduler", "dhcp_agent_scheduler",
"ext_gw_mode", "ext-gw-mode",
"extraroute", "extraroute",
"mac-learning", "mac-learning",
"network-gateway", "network-gateway",

17
neutron/tests/unit/test_extension_ext_gw_mode.py
View File

@ -326,7 +326,8 @@ class ExtGwModeTestCase(test_db_plugin.NeutronDbPluginV2TestCase,
expected_code=exc.HTTPOk.code, expected_code=exc.HTTPOk.code,
neutron_context=None): neutron_context=None):
ext_gw_info = {'network_id': network_id} ext_gw_info = {'network_id': network_id}
if snat_enabled in (True, False): # Need to set enable_snat also if snat_enabled == False
if snat_enabled is not None:
ext_gw_info['enable_snat'] = snat_enabled ext_gw_info['enable_snat'] = snat_enabled
return self._update('routers', router_id, return self._update('routers', router_id,
{'router': {'external_gateway_info': {'router': {'external_gateway_info':
@ -378,19 +379,25 @@ class ExtGwModeTestCase(test_db_plugin.NeutronDbPluginV2TestCase,
self._test_router_create_show_ext_gwinfo(False, False) self._test_router_create_show_ext_gwinfo(False, False)
def _test_router_update_ext_gwinfo(self, snat_input_value, def _test_router_update_ext_gwinfo(self, snat_input_value,
snat_expected_value): snat_expected_value=False,
expected_http_code=exc.HTTPOk.code):
with self.router() as r: with self.router() as r:
with self.subnet() as s: with self.subnet() as s:
try:
ext_net_id = s['subnet']['network_id'] ext_net_id = s['subnet']['network_id']
self._set_net_external(ext_net_id) self._set_net_external(ext_net_id)
self._set_router_external_gateway( self._set_router_external_gateway(
r['router']['id'], ext_net_id, r['router']['id'], ext_net_id,
snat_enabled=snat_input_value) snat_enabled=snat_input_value,
expected_code=expected_http_code)
if expected_http_code != exc.HTTPOk.code:
return
body = self._show('routers', r['router']['id']) body = self._show('routers', r['router']['id'])
res_gw_info = body['router']['external_gateway_info'] res_gw_info = body['router']['external_gateway_info']
self.assertEqual(res_gw_info['network_id'], ext_net_id) self.assertEqual(res_gw_info['network_id'], ext_net_id)
self.assertEqual(res_gw_info['enable_snat'], self.assertEqual(res_gw_info['enable_snat'],
snat_expected_value) snat_expected_value)
finally:
self._remove_external_gateway_from_router( self._remove_external_gateway_from_router(
r['router']['id'], ext_net_id) r['router']['id'], ext_net_id)
@ -402,3 +409,7 @@ class ExtGwModeTestCase(test_db_plugin.NeutronDbPluginV2TestCase,
def test_router_update_ext_gwinfo_with_snat_disabled(self): def test_router_update_ext_gwinfo_with_snat_disabled(self):
self._test_router_update_ext_gwinfo(False, False) self._test_router_update_ext_gwinfo(False, False)
def test_router_update_ext_gwinfo_with_invalid_snat_setting(self):
self._test_router_update_ext_gwinfo(
'xxx', None, expected_http_code=exc.HTTPBadRequest.code)