[ovs-dev] [PATCH 1/2] ovn-performance.at: Missing steps for connecting LS to LR.

Han Zhou zhouhan at gmail.com
Wed Jul 10 04:23:10 UTC 2019

From: Han Zhou <hzhou8 at ebay.com>

The test creates 2 logical switches and connect them with a logical router.
However, it didn't set the option "router-port", so the 2 LS datapaths
were not connected. This results in missing test coverage for port-binding
incremental processing: assume I-P has a bug and port-binding change always
trigger recompute, since each HV monitors only its own datapath (i.e.
HV1 -> ls1, HV2 -> ls2) then it never got notification of the other
port-binding change, thus recompute is never triggered when port-binding
is updated on the other datapath. With this fix, each HV's local datapaths
will include both ls1 and ls2, so port-binding change notification will
be received properly and unexpected recompute would be captured.

Signed-off-by: Han Zhou <hzhou8 at ebay.com>
 tests/ovn-performance.at | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/tests/ovn-performance.at b/tests/ovn-performance.at
index 6a213eb..5e061ee 100644
--- a/tests/ovn-performance.at
+++ b/tests/ovn-performance.at
@@ -275,6 +275,10 @@ for i in 1 2; do
         [hv1 hv2], [lflow_run],
+        [ovn-nbctl --wait=hv lsp-set-options $lsp router-port=$lrp]
+    )
+        [hv1 hv2], [lflow_run],
         [ovn-nbctl --wait=hv lsp-set-addresses $lsp router]

More information about the dev mailing list