Thanks Ales and Mohammad. I pushed this change to main.

On 8/10/22 08:05, Mohammad Heib wrote:
Acked-by: Mohammad Heib <[email protected]>

thanks,


On Wed, Aug 10, 2022 at 3:02 PM Ales Musil <[email protected]> wrote:

After switch to parallel jobs some tests
were missing due to way how filtering in autotest
works. Add additional jobs that filter the !ovn-northd
which is currently always present when the test calls
OVN_FOR_EACH_NORTHD.

Fixes: d093905 ("OVN-CI: ovn unit tests run in parallel jobs.")
Signed-off-by: Ales Musil <[email protected]>
---
v3: Switch to !ovn-northd after suggestion from Mohammad.
---
  .github/workflows/test.yml | 5 +++++
  1 file changed, 5 insertions(+)

diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
index afa984345..7a59cd478 100644
--- a/.github/workflows/test.yml
+++ b/.github/workflows/test.yml
@@ -40,22 +40,27 @@ jobs:
          - { compiler: gcc, testsuite: test, testsuite_kw:
"parallelization=yes,ovn_monitor_all=no" }
          - { compiler: gcc, testsuite: test, testsuite_kw:
"parallelization=no,ovn_monitor_all=yes" }
          - { compiler: gcc, testsuite: test, testsuite_kw:
"parallelization=no,ovn_monitor_all=no" }
+        - { compiler: gcc, testsuite: test, testsuite_kw: "!ovn-northd" }
          - { compiler: clang, testsuite: test, sanitizers: sanitizers,
testsuite_kw: "parallelization=yes,ovn_monitor_all=yes" }
          - { compiler: clang, testsuite: test, sanitizers: sanitizers,
testsuite_kw: "parallelization=yes,ovn_monitor_all=no" }
          - { compiler: clang, testsuite: test, sanitizers: sanitizers,
testsuite_kw: "parallelization=no,ovn_monitor_all=yes" }
          - { compiler: clang, testsuite: test, sanitizers: sanitizers,
testsuite_kw: "parallelization=no,ovn_monitor_all=no" }
+        - { compiler: clang, testsuite: test, sanitizers: sanitizers,
testsuite_kw: "!ovn-northd" }
          - { compiler: gcc, testsuite: test, libs: -ljemalloc,
testsuite_kw: "parallelization=yes,ovn_monitor_all=yes" }
          - { compiler: gcc, testsuite: test, libs: -ljemalloc,
testsuite_kw: "parallelization=yes,ovn_monitor_all=no" }
          - { compiler: gcc, testsuite: test, libs: -ljemalloc,
testsuite_kw: "parallelization=no,ovn_monitor_all=yes" }
          - { compiler: gcc, testsuite: test, libs: -ljemalloc,
testsuite_kw: "parallelization=no,ovn_monitor_all=no" }
+        - { compiler: gcc, testsuite: test, libs: -ljemalloc,
testsuite_kw: "!ovn-northd" }
          - { compiler: clang, testsuite: test, libs: -ljemalloc,
testsuite_kw: "parallelization=yes,ovn_monitor_all=yes" }
          - { compiler: clang, testsuite: test, libs: -ljemalloc,
testsuite_kw: "parallelization=yes,ovn_monitor_all=no" }
          - { compiler: clang, testsuite: test, libs: -ljemalloc,
testsuite_kw: "parallelization=no,ovn_monitor_all=yes" }
          - { compiler: clang, testsuite: test, libs: -ljemalloc,
testsuite_kw: "parallelization=no,ovn_monitor_all=no" }
+        - { compiler: clang, testsuite: test, libs: -ljemalloc,
testsuite_kw: "!ovn-northd" }
          - { compiler: gcc, testsuite: system-test, testsuite_kw:
"parallelization=yes,ovn_monitor_all=yes" }
          - { compiler: gcc, testsuite: system-test, testsuite_kw:
"parallelization=yes,ovn_monitor_all=no" }
          - { compiler: gcc, testsuite: system-test, testsuite_kw:
"parallelization=no,ovn_monitor_all=yes" }
          - { compiler: gcc, testsuite: system-test, testsuite_kw:
"parallelization=no,ovn_monitor_all=no" }
+        - { compiler: gcc, testsuite: system-test, testsuite_kw:
"!ovn-northd" }
          - { compiler: gcc,  m32: m32, opts: --disable-ssl}

      steps:
--
2.37.1

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


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


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

Reply via email to