[ovs-build] Failed: dceara/ovn#98 (arp_nd_broadcast_limit-v7 - 60f55f7)

Travis CI builds at travis-ci.org
Tue Nov 12 10:07:02 UTC 2019


Build Update for dceara/ovn
-------------------------------------

Build: #98
Status: Failed

Duration: 9 mins and 56 secs
Commit: 60f55f7 (arp_nd_broadcast_limit-v7)
Author: Dumitru Ceara
Message: ovn-northd: Limit ARP/ND broadcast domain whenever possible.

ARP request and ND NS packets for router owned IPs were being
flooded in the complete L2 domain (using the MC_FLOOD multicast group).
However this creates a scaling issue in scenarios where aggregation
logical switches are connected to more logical routers (~350). The
logical pipelines of all routers would have to be executed before the
packet is finally replied to by a single router, the owner of the IP
address.

This commit limits the broadcast domain by bypassing the L2 Lookup stage
for ARP requests that will be replied by a single router. The packets
are forwarded only to the router port that owns the target IP address.

IPs that are owned by the routers and for which this fix applies are:
- IP addresses configured on the router ports.
- VIPs.
- NAT IPs.

Reported-at: https://bugzilla.redhat.com/1756945
Reported-by: Anil Venkata <vkommadi at redhat.com>
Signed-off-by: Dumitru Ceara <dceara at redhat.com>

---
v7:
- Address Han's comments:
    - Remove flooding for all ARPs received on VLAN networks. To avoid
      that we now identify self originated (G)ARPs by matching on source
      MAC address too.
    - Rename REGBIT_NOT_VXLAN to FLAGBIT_NOT_VXLAN.
- Fix ovn-sb manpage.
- Split patch in a series of 2:
    - patch1: fixes the get_router_load_balancer_ips() function.
    - patch2: limits the ARP/ND broadcast domain.
v6:
- Address Han's comments:
    - remove flooding of ARPs targeting OVN owned IP addresses.
    - update ovn-architecture documentation.
    - rename ARP handling functions.
    - Adapt "ovn -- 3 HVs, 3 LS, 3 lports/LS, 1 LR" autotest to take into
    account the new way of forwarding ARPs.
- Also, properly deal with ARP packets on VLAN-backed networks.
v5: Address Numan's comments: update comments & make autotest more
    robust.
v4: Rebase.
v3: Properly deal with VXLAN traffic. Address review comments from
    Numan (add autotests). Fix function get_router_load_balancer_ips.
    Rebase -> deal with IPv6 NAT too.
v2: Move ARP broadcast domain limiting to table S_SWITCH_IN_L2_LKUP to
address localnet ports too.

View the changeset: https://github.com/dceara/ovn/compare/2b047e64bfb8...60f55f78cbb2

View the full build log and details: https://travis-ci.org/dceara/ovn/builds/610773957?utm_medium=notification&utm_source=email

--

You can unsubscribe from build emails from the dceara/ovn repository going to https://travis-ci.org/account/preferences/unsubscribe?repository=25507942&utm_medium=notification&utm_source=email.
Or unsubscribe from *all* email updating your settings at https://travis-ci.org/account/preferences/unsubscribe?utm_medium=notification&utm_source=email.
Or configure specific recipients for build notifications in your .travis.yml file. See https://docs.travis-ci.com/user/notifications.

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openvswitch.org/pipermail/ovs-build/attachments/20191112/d2968c0a/attachment-0001.html>


More information about the build mailing list