Before this patch, each system test was supposed to run four times:
- with and without northd-parallelization
- with and without monitor-all
However, while the titles of the test were updated, system tests were never
run with monitor-all=true (i.e. they were run twice w/o monitor-all).

This is now fixed, and the four flavors of the tssts are run.

Signed-off-by: Xavier Simonart <[email protected]>
---
 tests/system-dpdk-macros.at      | 4 ++++
 tests/system-kmod-macros.at      | 3 +++
 tests/system-userspace-macros.at | 3 +++
 3 files changed, 10 insertions(+)

diff --git a/tests/system-dpdk-macros.at b/tests/system-dpdk-macros.at
index 8843baa25..440908af7 100644
--- a/tests/system-dpdk-macros.at
+++ b/tests/system-dpdk-macros.at
@@ -47,6 +47,10 @@ m4_define([OVS_TRAFFIC_VSWITCHD_START],
    dnl Add bridges, ports, etc.
    OVS_WAIT_WHILE([ip link show br0])
    AT_CHECK([ovs-vsctl -- _ADD_BR([br0]) -- $1 m4_if([$2], [], [], [| 
uuidfilt])], [0], [$2])
+
+   if test OVN_MONITOR_ALL = yes; then
+        ovs-vsctl set open . external_ids:ovn-monitor-all=true
+   fi
 ])
 
 # OVS_TRAFFIC_VSWITCHD_STOP([WHITELIST], [extra_cmds])
diff --git a/tests/system-kmod-macros.at b/tests/system-kmod-macros.at
index 787a59c97..6f6670199 100644
--- a/tests/system-kmod-macros.at
+++ b/tests/system-kmod-macros.at
@@ -27,6 +27,9 @@ m4_define([OVS_TRAFFIC_VSWITCHD_START],
    _OVS_VSWITCHD_START([])
    dnl Add bridges, ports, etc.
    AT_CHECK([ovs-vsctl -- _ADD_BR([br0]) -- $1 m4_if([$2], [], [], [| 
uuidfilt])], [0], [$2])
+   if test OVN_MONITOR_ALL = yes; then
+        ovs-vsctl set open . external_ids:ovn-monitor-all=true
+   fi
 ])
 
 # OVS_TRAFFIC_VSWITCHD_STOP([WHITELIST], [extra_cmds])
diff --git a/tests/system-userspace-macros.at b/tests/system-userspace-macros.at
index 86ed22622..73ca2cce3 100644
--- a/tests/system-userspace-macros.at
+++ b/tests/system-userspace-macros.at
@@ -19,6 +19,9 @@ m4_define([OVS_TRAFFIC_VSWITCHD_START],
    dnl Add bridges, ports, etc.
    OVS_WAIT_WHILE([ip link show br0])
    AT_CHECK([ovs-vsctl -- _ADD_BR([br0]) -- $1 m4_if([$2], [], [], [| 
uuidfilt])], [0], [$2])
+   if test OVN_MONITOR_ALL = yes; then
+        ovs-vsctl set open . external_ids:ovn-monitor-all=true
+   fi
 ])
 
 # OVS_TRAFFIC_VSWITCHD_STOP([WHITELIST], [extra_cmds])
-- 
2.31.1

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to