When ovs-agent is restarted flows creatd by other than ovs-agent are deleted.

Bug #1525775 reported by SUZUKI, Kazuhiro
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Won't Fix
Undecided
SUZUKI, Kazuhiro
tap-as-a-service
Fix Released
Critical
SUZUKI, Kazuhiro

Bug Description

When ovs-agent is restarted, the cleanup logic drops flow entries unless they are stamped by agent_uuid (recorded as a cookie).

Referene: https://git.openstack.org/cgit/openstack/neutron/commit/?id=73673beacd75a2d9f51f15b284f1b458d32e992e

Not only old flows, but also flows created by other than ovs-agent (flows without a stamp) are deleted.

Version: Liberty

Tags: ovs
Changed in neutron:
assignee: nobody → SUZUKI, Kazuhiro (kaz-k)
status: New → In Progress
Henry Gessau (gessau)
tags: added: ovs
Revision history for this message
Eugene Nikanorov (enikanorov) wrote :

That is current as-designed behavior.
I'd be interested in knowing proposed solution for this issue.

Revision history for this message
SUZUKI, Kazuhiro (kaz-kaz) wrote :

I've posted a patch to solve this issue.
https://review.openstack.org/#/c/257285/

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on tap-as-a-service (master)

Change abandoned by Kazuhiro Suzuki (<email address hidden>) on branch: master
Review: https://review.openstack.org/257286
Reason: L2 agent extension api is available.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on neutron (master)

Change abandoned by Armando Migliaccio (<email address hidden>) on branch: master
Review: https://review.openstack.org/257285
Reason: This review is > 4 weeks without comment and currently blocked by a core reviewer with a -2. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and contacting the reviewer with the -2 on this review to ensure you address their concerns.

Revision history for this message
Armando Migliaccio (armando-migliaccio) wrote :
Changed in neutron:
status: In Progress → Won't Fix
Revision history for this message
Miguel Angel Ajo (mangelajo) wrote :

Yes, tap as a service may need to be implemented as an OVS agent extension, and make use of the API @armax referenced.

lizhixin (lizhixin3016)
Changed in tap-as-a-service:
assignee: nobody → lizhixin (lizhixin3016)
assignee: lizhixin (lizhixin3016) → nobody
Revision history for this message
YAMAMOTO Takashi (yamamoto) wrote :
Changed in tap-as-a-service:
assignee: nobody → SUZUKI, Kazuhiro (kaz-kaz)
importance: Undecided → Critical
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.