[ovs-git] [ovn-org/ovn] 287207: ovn-northd: Fix multiple ARP replies for SNAT entr...

numansiddique noreply at github.com
Mon Sep 14 17:50:51 UTC 2020


  Branch: refs/heads/branch-20.06
  Home:   https://github.com/ovn-org/ovn
  Commit: 28720796fae9b5d9ff4f2a8b448df18d9fcdefb4
      https://github.com/ovn-org/ovn/commit/28720796fae9b5d9ff4f2a8b448df18d9fcdefb4
  Author: Numan Siddique <numans at ovn.org>
  Date:   2020-09-14 (Mon, 14 Sep 2020)

  Changed paths:
    M TODO.rst
    M northd/ovn-northd.c
    M tests/ovn-northd.at
    M tests/ovn.at

  Log Message:
  -----------
  ovn-northd: Fix multiple ARP replies for SNAT entries configured on a distributed router.

The commit in the Fixes tag, while addressing the issue to send ARP replies for
the SNAT entries, didn't take into account the gateway router port scenario.
Because of this, all the chassis which have bridge mappings configured reply
to ARP request for SNAT entries. This patch fixes the issue by adding
"is_chassis_resident()" condition for such flows so that only the gateway chassis
which claims the gateway router port responds to the ARP request.

Note: This patch doesn't require any changes to ovn-northd.8.xml as it was already
documented with the desired flows for SNAT entries.

Reported-at: https://mail.openvswitch.org/pipermail/ovs-discuss/2020-September/050679.html
Reported-by: Chris <kklimonda at syntaxhighlighted.com>
Fixes: e2aa124ff7c2("ovn-northd: Add ARP responder flows for SNAT entries.")
Acked-by: Dumitru Ceara <dceara at redhat.com>
Signed-off-by: Numan Siddique <numans at ovn.org>




More information about the git mailing list