|
 |
773311 |
From 356501f3246ddb99aef8cd6016467b7c1861b3ff Mon Sep 17 00:00:00 2001
|
|
|
773311 |
From: Dumitru Ceara <dceara@redhat.com>
|
|
|
773311 |
Date: Tue, 24 Mar 2020 14:35:41 +0100
|
|
|
773311 |
Subject: [PATCH ovn] ovn.at: Fix ARP test that fails due to timing.
|
|
|
773311 |
|
|
|
773311 |
The test for "ARP/ND request broadcast limiting" checks that injected
|
|
|
773311 |
ARP packets are not flooded using the MC_FLOOD multicast group. However,
|
|
|
773311 |
this introduces a race condition in the test because GARPs generated by
|
|
|
773311 |
OVN would also hit the same openflow rules.
|
|
|
773311 |
|
|
|
773311 |
Remove the checks that use the MC_FLOOD group. They are also redundant
|
|
|
773311 |
as the rest of the test checks that packets are forwarded according to
|
|
|
773311 |
the newly added, higher priority rules.
|
|
|
773311 |
|
|
|
773311 |
Fixes: 32f5ebb06226 ("ovn-northd: Limit ARP/ND broadcast domain whenever possible.")
|
|
|
773311 |
Signed-off-by: Dumitru Ceara <dceara@redhat.com>
|
|
|
773311 |
Signed-off-by: Numan Siddique <numans@ovn.org>
|
|
|
773311 |
(cherry picked from upstream commit 598a07cd240d7d01de3d7f04ca7abc58a33977a1)
|
|
|
773311 |
|
|
|
773311 |
Change-Id: I4519f441245a5f1ecf7da73257f97aff0c7bc967
|
|
|
773311 |
---
|
|
|
773311 |
tests/ovn.at | 33 ---------------------------------
|
|
|
773311 |
1 file changed, 33 deletions(-)
|
|
|
773311 |
|
|
|
773311 |
diff --git a/tests/ovn.at b/tests/ovn.at
|
|
|
773311 |
index 4baf2e9..9a44f0a 100644
|
|
|
773311 |
--- a/tests/ovn.at
|
|
|
773311 |
+++ b/tests/ovn.at
|
|
|
773311 |
@@ -18015,9 +18015,6 @@ r1_dp_key=$(ovn-sbctl --bare --columns tunnel_key list datapath_binding rtr1)
|
|
|
773311 |
r1_tnl_key=$(ovn-sbctl --bare --columns tunnel_key list port_binding sw-rtr1)
|
|
|
773311 |
r2_tnl_key=$(ovn-sbctl --bare --columns tunnel_key list port_binding sw-rtr2)
|
|
|
773311 |
|
|
|
773311 |
-mc_key=$(ovn-sbctl --bare --columns tunnel_key find multicast_group datapath=${sw_dp_uuid} name="_MC_flood")
|
|
|
773311 |
-mc_key=$(printf "%04x" $mc_key)
|
|
|
773311 |
-
|
|
|
773311 |
match_sw_metadata="metadata=0x${sw_dp_key}"
|
|
|
773311 |
match_r1_metadata="metadata=0x${r1_dp_key}"
|
|
|
773311 |
|
|
|
773311 |
@@ -18042,11 +18039,6 @@ OVS_WAIT_UNTIL([
|
|
|
773311 |
grep n_packets=1 -c)
|
|
|
773311 |
test "0" = "${pkts_to_rtr2}"
|
|
|
773311 |
])
|
|
|
773311 |
-OVS_WAIT_UNTIL([
|
|
|
773311 |
- pkts_flooded=$(ovs-ofctl dump-flows br-int | \
|
|
|
773311 |
- grep -E "${match_sw_metadata}" | grep ${mc_key} | grep -v n_packets=0 -c)
|
|
|
773311 |
- test "0" = "${pkts_flooded}"
|
|
|
773311 |
-])
|
|
|
773311 |
|
|
|
773311 |
# Inject ND_NS for ofirst router owned IP address.
|
|
|
773311 |
src_ipv6=00100000000000000000000000000254
|
|
|
773311 |
@@ -18069,11 +18061,6 @@ OVS_WAIT_UNTIL([
|
|
|
773311 |
grep n_packets=1 -c)
|
|
|
773311 |
test "0" = "${pkts_to_rtr2}"
|
|
|
773311 |
])
|
|
|
773311 |
-OVS_WAIT_UNTIL([
|
|
|
773311 |
- pkts_flooded=$(ovs-ofctl dump-flows br-int | \
|
|
|
773311 |
- grep -E "${match_sw_metadata}" | grep ${mc_key} | grep -v n_packets=0 -c)
|
|
|
773311 |
- test "0" = "${pkts_flooded}"
|
|
|
773311 |
-])
|
|
|
773311 |
|
|
|
773311 |
# Configure load balancing on both routers.
|
|
|
773311 |
ovn-nbctl lb-add lb1-v4 10.0.0.11 42.42.42.1
|
|
|
773311 |
@@ -18108,11 +18095,6 @@ OVS_WAIT_UNTIL([
|
|
|
773311 |
grep n_packets=1 -c)
|
|
|
773311 |
test "0" = "${pkts_to_rtr2}"
|
|
|
773311 |
])
|
|
|
773311 |
-OVS_WAIT_UNTIL([
|
|
|
773311 |
- pkts_flooded=$(ovs-ofctl dump-flows br-int | \
|
|
|
773311 |
- grep -E "${match_sw_metadata}" | grep ${mc_key} | grep -v n_packets=0 -c)
|
|
|
773311 |
- test "0" = "${pkts_flooded}"
|
|
|
773311 |
-])
|
|
|
773311 |
|
|
|
773311 |
# Inject ND_NS for first router owned VIP address.
|
|
|
773311 |
src_ipv6=00100000000000000000000000000254
|
|
|
773311 |
@@ -18135,11 +18117,6 @@ OVS_WAIT_UNTIL([
|
|
|
773311 |
grep n_packets=1 -c)
|
|
|
773311 |
test "0" = "${pkts_to_rtr2}"
|
|
|
773311 |
])
|
|
|
773311 |
-OVS_WAIT_UNTIL([
|
|
|
773311 |
- pkts_flooded=$(ovs-ofctl dump-flows br-int | \
|
|
|
773311 |
- grep -E "${match_sw_metadata}" | grep ${mc_key} | grep -v n_packets=0 -c)
|
|
|
773311 |
- test "0" = "${pkts_flooded}"
|
|
|
773311 |
-])
|
|
|
773311 |
|
|
|
773311 |
# Configure NAT on both routers.
|
|
|
773311 |
ovn-nbctl lr-nat-add rtr1 dnat_and_snat 10.0.0.111 42.42.42.1
|
|
|
773311 |
@@ -18175,11 +18152,6 @@ OVS_WAIT_UNTIL([
|
|
|
773311 |
grep n_packets=1 -c)
|
|
|
773311 |
test "0" = "${pkts_to_rtr2}"
|
|
|
773311 |
])
|
|
|
773311 |
-OVS_WAIT_UNTIL([
|
|
|
773311 |
- pkts_flooded=$(ovs-ofctl dump-flows br-int | \
|
|
|
773311 |
- grep -E "${match_sw_metadata}" | grep ${mc_key} | grep -v n_packets=0 -c)
|
|
|
773311 |
- test "0" = "${pkts_flooded}"
|
|
|
773311 |
-])
|
|
|
773311 |
|
|
|
773311 |
# Inject ARP request for FIP1.
|
|
|
773311 |
send_arp_request 1 0 ${src_mac} $(ip_to_hex 10 0 0 254) $(ip_to_hex 10 0 0 121)
|
|
|
773311 |
@@ -18242,11 +18214,6 @@ OVS_WAIT_UNTIL([
|
|
|
773311 |
grep n_packets=1 -c)
|
|
|
773311 |
test "0" = "${pkts_to_rtr2}"
|
|
|
773311 |
])
|
|
|
773311 |
-OVS_WAIT_UNTIL([
|
|
|
773311 |
- pkts_flooded=$(ovs-ofctl dump-flows br-int | \
|
|
|
773311 |
- grep -E "${match_sw_metadata}" | grep ${mc_key} | grep -v n_packets=0 -c)
|
|
|
773311 |
- test "0" = "${pkts_flooded}"
|
|
|
773311 |
-])
|
|
|
773311 |
|
|
|
773311 |
# Inject ND_NS for FIP1.
|
|
|
773311 |
src_ipv6=00100000000000000000000000000254
|
|
|
773311 |
--
|
|
|
773311 |
1.8.3.1
|
|
|
773311 |
|