openstack/neutron
Code Issues Proposed changes

Remove deprecated TODO notes in L2 agent extension manager

Browse Source 
I asked Ajo what specifically these meant, and he thought their meaning was
lost in the mists of history.  He recommended that I put up a change to
remove them, and see if that elicits commentary that would reveal their
purpose.

Change-Id: I0206e42064287a9c1b5d6f3f265e53baea7d4043
This commit is contained in:
Nate Johnston
2016-06-16 15:06:30 +00:00
parent ff19748e30
commit 37ba46d81c
1 changed files with 0 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
neutron/agent/l2/extensions/manager.py
View File

@@ -66,7 +66,6 @@ class AgentExtensionsManager(stevedore.named.NamedExtensionManager):
for extension in self:
try:
extension.obj.handle_port(context, data)
# TODO(QoS) add agent extensions exception and catch them here
except AttributeError:
LOG.exception(
_LE("Agent Extension '%(name)s' failed "
@@ -79,8 +78,6 @@ class AgentExtensionsManager(stevedore.named.NamedExtensionManager):
for extension in self:
try:
extension.obj.delete_port(context, data)
# TODO(QoS) add agent extensions exception and catch them here
# instead of AttributeError
except AttributeError:
LOG.exception(
_LE("Agent Extension '%(name)s' failed "