[ovs-git] [ovn-org/ovn] 114858: Don't suppress localport traffic directed to exter...

[Ihar Hrachyshka]

  Branch: refs/heads/master
  Home:   https://github.com/ovn-org/ovn
  Commit: 1148580290d0ace803f20aeaa0241dd51c100630
      https://github.com/ovn-org/ovn/commit/1148580290d0ace803f20aeaa0241dd51c100630
  Author: Ihar Hrachyshka <ihrachys at redhat.com>
  Date:   2021-07-14 (Wed, 14 Jul 2021)

  Changed paths:
    M controller/binding.c
    M controller/ovn-controller.c
    M controller/ovn-controller.h
    M controller/physical.c
    M tests/ovn.at

  Log Message:
  -----------
  Don't suppress localport traffic directed to external port

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

Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=1974062

Signed-off-by: Ihar Hrachyshka <ihrachys at redhat.com>
Signed-off-by: Numan Siddique <numans at ovn.org>