mbox series

[ovs-dev,ovn,v2,0/3] Avoid ARP flow explosion.

Message ID 1596611114-118204-1-git-send-email-hzhou@ovn.org
Headers show
Series Avoid ARP flow explosion. | expand

Message

Han Zhou Aug. 5, 2020, 7:05 a.m. UTC
This patch series addresses the problem discussed in [0]. Below options need to
be configured for the gateway routers:

options:always_learn_from_arp_request = false
options:dynamic_neigh_routers = true

[0] - https://mail.openvswitch.org/pipermail/ovs-discuss/2020-May/049994.html

v1->v2:
The v1 series had 7 patches, 4 of them were merged.
In v2, a new test case is added for the patch 1/3.

Han Zhou (3):
  ovn-northd: Support optionally avoid static neighbor flows in routers.
  actions: Implement new actions lookup_arp_ip and lookup_nd_ip.
  ovn-northd.c: Support optionally disabling neighbor learning from ARP
    request/NS.

 controller/lflow.c      |   4 +-
 include/ovn/actions.h   |  10 +++
 lib/actions.c           | 112 ++++++++++++++++++++++++++++++
 northd/ovn-northd.8.xml | 114 +++++++++++++++++++++++++-----
 northd/ovn-northd.c     | 102 ++++++++++++++++++++++-----
 ovn-nb.xml              |  40 +++++++++++
 ovn-sb.xml              |  55 +++++++++++++++
 tests/ovn.at            | 180 ++++++++++++++++++++++++++++++++++++++++++++++++
 utilities/ovn-trace.c   |  54 ++++++++++++++-
 9 files changed, 633 insertions(+), 38 deletions(-)

Comments

Numan Siddique Aug. 10, 2020, 12:46 p.m. UTC | #1
On Wed, Aug 5, 2020 at 12:35 PM Han Zhou <hzhou@ovn.org> wrote:
>
> This patch series addresses the problem discussed in [0]. Below options need to
> be configured for the gateway routers:
>
> options:always_learn_from_arp_request = false
> options:dynamic_neigh_routers = true
>
> [0] - https://mail.openvswitch.org/pipermail/ovs-discuss/2020-May/049994.html
>
> v1->v2:
> The v1 series had 7 patches, 4 of them were merged.
> In v2, a new test case is added for the patch 1/3.

Hi Han,

As per the discussions we had in our previous irc meeting, I went
ahead and applied all your 7 patches
in this series to branch-20.06.

Thanks
Numan

>
> Han Zhou (3):
>   ovn-northd: Support optionally avoid static neighbor flows in routers.
>   actions: Implement new actions lookup_arp_ip and lookup_nd_ip.
>   ovn-northd.c: Support optionally disabling neighbor learning from ARP
>     request/NS.
>
>  controller/lflow.c      |   4 +-
>  include/ovn/actions.h   |  10 +++
>  lib/actions.c           | 112 ++++++++++++++++++++++++++++++
>  northd/ovn-northd.8.xml | 114 +++++++++++++++++++++++++-----
>  northd/ovn-northd.c     | 102 ++++++++++++++++++++++-----
>  ovn-nb.xml              |  40 +++++++++++
>  ovn-sb.xml              |  55 +++++++++++++++
>  tests/ovn.at            | 180 ++++++++++++++++++++++++++++++++++++++++++++++++
>  utilities/ovn-trace.c   |  54 ++++++++++++++-
>  9 files changed, 633 insertions(+), 38 deletions(-)
>
> --
> 2.1.0
>
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>