[DVR] Modify `in_port` field of packets which from remote qr-* port

Bug #1831706 reported by yangjianfeng
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
In Progress
Undecided
yangjianfeng

Bug Description

When the packets which from remote qr-* port enter `br-int` bridge, their `in_port` field will become to `patch-tun`(vxlan network) or `int-br-vlan`(vlan network), the way will result in these packets mustn't be processed by ovs flow `NORMAL` action, because of the `NORMAL` action will result in the ovs bridge update `fdb` table, but these packets `in_port` field is not correct, so ovs bridge will record an error MAC forward entry. Although, the current design about DVR is OK(It use `outpt` action for these packets). But, I think we can modify these packets' `in_port` field as local qr-* so that we can simplify the flow tables.
For example, the below flows can be deleted:
  table=60, priority=4,dl_vlan=288,dl_dst=fa:16:3e:2d:d7:85 actions=strip_vlan,output:"tap0500d7e2-87"
  table=60, priority=4,dl_vlan=287,dl_dst=fa:16:3e:bc:ec:67 actions=strip_vlan,output:"test-vlan02"
  table=60, priority=4,dl_vlan=287,dl_dst=fa:16:3e:bf:0c:35 actions=strip_vlan,output:"tapbcd046ef-26"
  table=60, priority=4,dl_vlan=287,dl_dst=fa:16:3e:b6:03:cd actions=strip_vlan,output:"test-vlan"
  table=60, priority=4,dl_vlan=203,dl_dst=fa:16:3e:8f:88:bc actions=strip_vlan,output:"tap2610c99a-52"
I think the below flow is enough if these packets' `in_port` field be modified as appropriate value.
  table=60, priority=3 actions=NORMAL

One more thing, I think it can be a resolution about the patch: https://review.opendev.org/#/c/639009

tags: added: l3-dvr-backlog
Changed in neutron:
assignee: nobody → yangjianfeng (yangjianfeng)
status: New → In Progress
Revision history for this message
LIU Yulong (dragon889) wrote :
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to neutron (master)

Related fix proposed to branch: master
Review: https://review.opendev.org/665647

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

Change abandoned by Slawek Kaplonski (<email address hidden>) on branch: master
Review: https://review.opendev.org/665647
Reason: This review is > 4 weeks without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Change abandoned by "liuyulong <email address hidden>" on branch: master
Review: https://review.opendev.org/c/openstack/neutron/+/663008
Reason: Plz restore if this still needed.

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.