OVS agent: avoid the use of OVSDB port tags

Bug #1756296 reported by Thomas Morin
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
New
Wishlist
Unassigned

Bug Description

Currently, the OVS agent relies on OVSDB port tags in br-int to mark the traffic arriving on a VM port with a local VLAN that is used to isolate the traffic once it exits br-int. The vlan (dot1q) tag is imposed on the packet when the NORMAL action is applied on the packet.

This approach is incompatible with the goal of having VLAN transparent ports (VM ports which can send tagged traffic that is forwarded as-is to other ports on the same Neutron network), because when an OVSDB port tag is set, OVS drops packets sent by a VM if they are already tagged [1].

Additionally, because its only applied after the NORMAL action, this local vlan is not usable in matches in br-int, this leads components such as the openvswitch SG firewall driver to keep track in an OVS register of which network a packet belongs to (the L2 openflow manager [2] will lead to other components ending up with the same need, other components such as networking-bagpipe worked around this limitation by placing rule in br-tun instead).

This RFE is here to discuss the idea of changing the design to not use OVSDB port tags anymore always use an OVS register instead, and use an explicit push_vlan action for traffic going towards br-ex, br-int, br-tun .

[1] http://paste.openstack.org/show/702971/
[2] https://review.openstack.org/#/c/323963/

zhaobo (zhaobo6)
Changed in neutron:
importance: Undecided → Wishlist
Revision history for this message
YAMAMOTO Takashi (yamamoto) wrote :
Revision history for this message
Slawek Kaplonski (slaweq) wrote :

Hi Thomas,

Do You still plan to work on this RFE? If so, can You provide answer to YAMAMOTO's questions from comment #1?

Revision history for this message
Thomas Morin (tmmorin-orange) wrote :

Slawek: no I don't intend to work on this in the near future

(to you question yamamoto: yes, this is what I had in mind)

Revision history for this message
Slawek Kaplonski (slaweq) wrote :

Thx for info Thomas. So I will now move this rfe to postponed state.

tags: added: rfe-postponed
removed: rfe
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.