x/vmware-nsx
Code Issues Proposed changes

Remove superfluous except/re-raise

Browse Source 
This exception code catches all exceptions, and then always raises them
again.

More interestingly, it uses excutils.save_and_reraise_exception
incorrectly (outside a `with` statement), which triggers the pylint test
designed to catch exactly this.

Change-Id: Iab9d69944cbae5257682ac65ff2b8fba3ef7017e
This commit is contained in:
Angus Lees 2014-10-20 11:19:02 +11:00
parent 0c1223862c
commit 1f138e25d3
1 changed files with 7 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
neutron/services/l3_router/brocade/l3_router_plugin.py
View File

@ -101,15 +101,13 @@ class BrocadeSVIPlugin(router.L3RouterPlugin):
""" delete a vrf on NOS device."""
router = super(BrocadeSVIPlugin, self).get_router(context, router_id)
super(BrocadeSVIPlugin, self).delete_router(context, router_id)
try:
switch = self._switch
self._driver.delete_router(switch['address'],
switch['username'],
switch['password'],
switch['rbridge_id'],
str(router['id']))
except Exception:
excutils.save_and_reraise_exception()
def add_router_interface(self, context, router_id, interface_info):
"""creates svi on NOS device and assigns ip addres to SVI."""