[ovs-dev] [PATCH v2 ovn] Don't suppress localport traffic directed to external port

[Dumitru Ceara]

On 7/7/21 5:03 PM, Ihar Hrachyshka wrote:
> Recently, we stopped leaking localport traffic through localnet ports
> into fabric to avoid unnecessary flipping between chassis hosting the
> same localport.
> 
> Despite the type name, in some scenarios localports are supposed to
> talk outside the hosting chassis. Specifically, in OpenStack [1]
> metadata service for SR-IOV ports is implemented as a localport hosted
> on another chassis that is exposed to the chassis owning the SR-IOV
> port through an "external" port. In this case, "leaking" localport
> traffic into fabric is desirable.
> 
> This patch inserts a higher priority flow per external port on the
> same datapath that avoids dropping localport traffic.
> 
> Fixes: 96959e56d634 ("physical: do not forward traffic from localport
> to a localnet one")
> 
> [1] https://docs.openstack.org/neutron/latest/admin/ovn/sriov.html
> 
> Signed-off-by: Ihar Hrachyshka <ihrachys at redhat.com>
> 
> --

Hi Ihar,

Looks like our emails crossed, but I left some comments on the v1 of
this patch and I think they still apply to v2:

https://mail.openvswitch.org/pipermail/ovs-dev/2021-July/385130.html

Regards,
Dumitru