[ovs-git] [ovn-org/ovn] 3100c6: Support vlan-passthru for tag=0 logical switch ports

Ihar Hrachyshka noreply at github.com
Fri Jul 30 18:00:21 UTC 2021


  Branch: refs/heads/branch-21.03
  Home:   https://github.com/ovn-org/ovn
  Commit: 3100c6ea268c6d8580a298c6d529b75e3430d9a1
      https://github.com/ovn-org/ovn/commit/3100c6ea268c6d8580a298c6d529b75e3430d9a1
  Author: Ihar Hrachyshka <ihrachys at redhat.com>
  Date:   2021-07-30 (Fri, 30 Jul 2021)

  Changed paths:
    M controller/physical.c
    M northd/ovn-northd.c
    M ovn-nb.xml
    M tests/ovn.at

  Log Message:
  -----------
  Support vlan-passthru for tag=0 logical switch ports

When vlan-passthru is true for LS, for untagged localnet ports, tagged
VLAN traffic originating from VIFs should be passed through intact since
the VLAN header belongs to VIF user, not the localnet port fabric.

This patch adds multinode test cases to cover scenarios where packets
traverse through fabric layer, for both tagged and untagged (tag=0)
localnet ports.

Conflicts:
      NEWS
      northd/ovn_northd.dl
      tests/ovn.at

Signed-off-by: Ihar Hrachyshka <ihrachys at redhat.com>
Signed-off-by: Numan Siddique <numans at ovn.org>
(cherry picked from commit 962740bdb3541eb04618ba1d1c912e2985a87b0c)


  Commit: 317f52fd95c27f2b4d1b0777dc352b11c7d84b0f
      https://github.com/ovn-org/ovn/commit/317f52fd95c27f2b4d1b0777dc352b11c7d84b0f
  Author: Ihar Hrachyshka <ihrachys at redhat.com>
  Date:   2021-07-30 (Fri, 30 Jul 2021)

  Changed paths:
    M northd/ovn-northd.8.xml
    M northd/ovn-northd.c
    M tests/ovn.at

  Log Message:
  -----------
  Disable ARP/NA responders for vlan-passthru switches

When vlan-passthru is on, VIFs may attach different VLAN tags. In this
case, VIFs are not guaranteed to belong to the same L2 broadcast domain.
Because of that, we don't know if a peer port on the switch has the same
tag used and should not allow the local responder to generate neighbour
traffic. Instead, pass ARP and ND requests to the peer port owner and
allow it to reply, if needed.

Conflicts:
      northd/ovn_northd.dl

Signed-off-by: Ihar Hrachyshka <ihrachys at redhat.com>
Signed-off-by: Numan Siddique <numans at ovn.org>
Acked-by: Mark Michelson <mmichels at redhat.com>
(cherry picked from commit ea57f666f6eef1eb1d578f0e975baa14c5d23ec9)
(cherry picked from commit 0cfd85b758ac0bd1f063a82734ef45359bc502bc)


Compare: https://github.com/ovn-org/ovn/compare/7b90deff5386...317f52fd95c2


More information about the git mailing list