yahoo-eng-team team mailing list archive

[Slawek Kaplonski <1743647@xxxxxxxxxxxxxxxxxx>]

Public bug reported:

L2 extension manager which calls L2 extension drivers (like QoS) should handle somehow exceptions from those drivers. For example ext_manager.delete_port(), like in: https://github.com/openstack/neutron/blob/master/neutron/plugins/ml2/drivers/openvswitch/agent/ovs_neutron_agent.py#L1603 should handle cases where e.g. port already not exists and driver wants to do something with it.
It was discussed in comments to patch https://review.openstack.org/#/c/533318/ and on Neutron CI meeting: http://eavesdrop.openstack.org/meetings/neutron_ci/2018/neutron_ci.2018-01-16-16.01.log.html#l-150

** Affects: neutron
     Importance: Low
         Status: Confirmed

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1743647

Title:
  L2 extension manager don't handle drivers errors

Status in neutron:
  Confirmed

Bug description:
  L2 extension manager which calls L2 extension drivers (like QoS) should handle somehow exceptions from those drivers. For example ext_manager.delete_port(), like in: https://github.com/openstack/neutron/blob/master/neutron/plugins/ml2/drivers/openvswitch/agent/ovs_neutron_agent.py#L1603 should handle cases where e.g. port already not exists and driver wants to do something with it.
  It was discussed in comments to patch https://review.openstack.org/#/c/533318/ and on Neutron CI meeting: http://eavesdrop.openstack.org/meetings/neutron_ci/2018/neutron_ci.2018-01-16-16.01.log.html#l-150

To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/1743647/+subscriptions