yahoo-eng-team team mailing list archive

[Édouard Thuleau <edouard.thuleau@xxxxxxxxxxxxx>]

Public bug reported:

OVS lib propose a deferred apply methods to save system calls to 'ovs-
ofctl' binaries ('ovs-ofctl' can apply flow from file or stdin if file
is '-').

This method use a dict for 'add', mod' or 'del' flow actions that contain a concatenated string flows. This dict is purge after all flows are applied at the end of 'deferred_apply_off' method.
If another call is made on that dict during the 'deferred_apply_off', some flows could be deleted a the end when they have not been applied.

I can see that on ML2 plugin with l2-pop mechanism driver. If I delete
more than one port at a time, some flooding flow rules could be not
deleted on the br-tun bridge.

** Affects: neutron
     Importance: Undecided
     Assignee: Édouard Thuleau (ethuleau)
         Status: New


** Tags: l2-pop ovs

** Changed in: neutron
     Assignee: (unassigned) => Édouard Thuleau (ethuleau)

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

Title:
  OVS lib deferred apply cannot handle concurrency

Status in OpenStack Neutron (virtual network service):
  New

Bug description:
  OVS lib propose a deferred apply methods to save system calls to 'ovs-
  ofctl' binaries ('ovs-ofctl' can apply flow from file or stdin if file
  is '-').

  This method use a dict for 'add', mod' or 'del' flow actions that contain a concatenated string flows. This dict is purge after all flows are applied at the end of 'deferred_apply_off' method.
  If another call is made on that dict during the 'deferred_apply_off', some flows could be deleted a the end when they have not been applied.

  I can see that on ML2 plugin with l2-pop mechanism driver. If I delete
  more than one port at a time, some flooding flow rules could be not
  deleted on the br-tun bridge.

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