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

numansiddique noreply at github.com
Mon Sep 14 17:32:48 UTC 2020


  Branch: refs/heads/master
  Home:   https://github.com/ovn-org/ovn
  Commit: b5fa8d34fbb8fa91590b371571f5e61c275aa9f8
      https://github.com/ovn-org/ovn/commit/b5fa8d34fbb8fa91590b371571f5e61c275aa9f8
  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