Bug 1017414

Summary: OVS flow rules should strip vlan for tunnel data
Product: Red Hat OpenStack Reporter: Chris Pelland <cpelland>
Component: openstack-quantumAssignee: Bob Kukura <rkukura>
Status: CLOSED WONTFIX QA Contact: Ofer Blaut <oblaut>
Severity: urgent Docs Contact:
Priority: urgent    
Version: 3.0CC: acathrow, chrisw, dmaley, hateya, lhh, lpeer, oblaut, pep, rkukura, sclewis, yeylon
Target Milestone: asyncKeywords: ZStream
Target Release: 3.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: 1007158 Environment:
Last Closed: 2013-12-02 19:10:12 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 1007158    
Bug Blocks:    
Attachments:
Description Flags
Fix from upstream
none
Replacement patch none

Comment 1 Lon Hohberger 2013-10-09 20:49:09 UTC 
Created attachment 810180 [details]
Fix from upstream
Comment 6 Lon Hohberger 2013-10-10 19:49:29 UTC 
Created attachment 810725 [details]
Replacement patch
Comment 8 Bob Kukura 2013-11-19 14:48:32 UTC 
I'd like to CLOSE/WONTFIX this bug for RHOS 3.0. The main motivation for stripping the VLAN tags in tunnel payloads was to work around the performance issue whose root cause was bug 997632, which has since been fixed. Therefore, fixing this bug will not result in any further significant performance improvement.

Bug 1007158 addresses this same issue for RHOS 4.0, and is fixed. The openvswitch-agent's OVS flow rules have been reworked upstream in havana so that the VLAN tags are stripped. This involves substantial code and behavioural changes that are not practical to back-port to RHOS 3.0. Simpler possbile fixes for RHOS 3.0 have been tried, but result in all traffic being flooded to all tunnel endpoints, which would seriously limit scalability and performance.

Are there any objections to closing this?
Comment 9 Bob Kukura 2013-12-02 19:10:12 UTC 
Closing with WONTFIX as proposed above.