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

Reply via email to