currently there is no new changes applied to the ddlog code base in ovn and we keep skipping ddlog test cases in our ci runs which leads to so many skips lines printed to the ci logs and that cause us to miss some tests cases that were skipped because of missing packages or any other reason.
This patch will remove ddlog test cases from the OVN-CI, which will give us more clear ci output and less execution time. Signed-off-by: Mohammad Heib <[email protected]> --- tests/ovn-macros.at | 11 +++-------- tests/ovn.at | 3 --- tests/ovs-macros.at | 4 ---- 3 files changed, 3 insertions(+), 15 deletions(-) diff --git a/tests/ovn-macros.at b/tests/ovn-macros.at index c6f0f6251..df3f35554 100644 --- a/tests/ovn-macros.at +++ b/tests/ovn-macros.at @@ -748,22 +748,17 @@ m4_define([OVN_POPULATE_ARP], [AT_CHECK(ovn_populate_arp__, [0], [ignore])]) # Defines a versions of a test with all combinations of northd and # datapath groups. m4_define([OVN_FOR_EACH_NORTHD], - [m4_foreach([NORTHD_TYPE], [ovn-northd, ovn-northd-ddlog], + [m4_foreach([NORTHD_TYPE], [ovn-northd], [m4_foreach([NORTHD_USE_DP_GROUPS], [yes, no], [m4_foreach([NORTHD_USE_PARALLELIZATION], [yes, no], [$1 ])])])]) # Some tests aren't prepared for dp groups to be enabled. m4_define([OVN_FOR_EACH_NORTHD_WITHOUT_DP_GROUPS], - [m4_foreach([NORTHD_TYPE], [ovn-northd, ovn-northd-ddlog], + [m4_foreach([NORTHD_TYPE], [ovn-northd], [m4_foreach([NORTHD_USE_DP_GROUPS], [no], [m4_foreach([NORTHD_USE_PARALLELIZATION], [yes, no], [$1 ])])])]) # Some tests aren't prepared for ddlog to be enabled. -m4_define([OVN_FOR_EACH_NORTHD_WITHOUT_DDLOG], - [m4_foreach([NORTHD_TYPE], [ovn-northd], - [m4_foreach([NORTHD_USE_DP_GROUPS], [yes, no], - [m4_foreach([NORTHD_USE_PARALLELIZATION], [yes, no], [$1 -])])])]) - +m4_define([OVN_FOR_EACH_NORTHD_WITHOUT_DDLOG], [OVN_FOR_EACH_NORTHD($@)]) diff --git a/tests/ovn.at b/tests/ovn.at index 4461b840e..1dcd82762 100644 --- a/tests/ovn.at +++ b/tests/ovn.at @@ -22993,9 +22993,6 @@ AT_CLEANUP OVN_FOR_EACH_NORTHD([ AT_SETUP([interconnection]) -dnl This test has problems with ovn-northd-ddlog. -AT_SKIP_IF([test NORTHD_TYPE = ovn-northd-ddlog && test "$RUN_ANYWAY" != yes]) - ovn_init_ic_db n_az=5 n_ts=5 diff --git a/tests/ovs-macros.at b/tests/ovs-macros.at index 0482b7f5b..dc3f34dc9 100644 --- a/tests/ovs-macros.at +++ b/tests/ovs-macros.at @@ -7,7 +7,6 @@ dnl Make AT_SETUP automatically do some things for us: dnl - Run the ovs_init() shell function as the first step in every test. dnl - If NORTHD_TYPE is defined, then append it to the test name and dnl set it as a shell variable as well. -dnl - Skip the test if it's for ovn-northd-ddlog but it didn't get built. m4_rename([AT_SETUP], [OVS_AT_SETUP]) m4_define([AT_SETUP], [OVS_AT_SETUP($@[]m4_ifdef([NORTHD_TYPE], [ -- NORTHD_TYPE])[]m4_ifdef([NORTHD_USE_DP_GROUPS], [ -- dp-groups=NORTHD_USE_DP_GROUPS])[]m4_ifdef([NORTHD_USE_PARALLELIZATION], [ -- parallelization=NORTHD_USE_PARALLELIZATION])) @@ -19,9 +18,6 @@ m4_ifdef([NORTHD_USE_PARALLELIZATION], [[NORTHD_USE_PARALLELIZATION]=NORTHD_USE_ ])dnl m4_ifdef([NORTHD_DUMMY_NUMA], [[NORTHD_DUMMY_NUMA]=NORTHD_DUMMY_NUMA ])dnl -m4_if(NORTHD_TYPE, [ovn-northd-ddlog], [dnl -AT_SKIP_IF([test $TEST_DDLOG = no]) -])dnl ovs_init ]) -- 2.34.1 _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
