Thanks. (Confirmed the other test case that covers tunnel exhaustion
scenarios already uses `eval`.)

Reviewed-By: Ihar Hrachyshka <i...@redhat.com>

On Mon, Jul 15, 2024 at 5:17 AM Ales Musil <amu...@redhat.com> wrote:

> The test tries to setup 4097 LSs via single ovn-nbctl command,
> this is fine however it might lead to "Unreasonably long Xms poll
> interval" on slower systems thus failing the test. Remove the check
> as the test still checks if all switches were created right under
> so it should be completely fine and this should avoid faliures
> on slower systems.
>
> Fixes: 7e99500e60bf ("northd: Add support for disabling vxlan mode.")
> Signed-off-by: Ales Musil <amu...@redhat.com>
> ---
>  tests/ovn-northd.at | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/ovn-northd.at b/tests/ovn-northd.at
> index a389d1988..49a145e7d 100644
> --- a/tests/ovn-northd.at
> +++ b/tests/ovn-northd.at
> @@ -2887,7 +2887,7 @@ for i in {1..4097..1}; do
>      cmd="${cmd} -- ls-add lsw-${i}"
>  done
>
> -check $cmd
> +eval $cmd
>
>  check_row_count nb:Logical_Switch 4097
>  wait_row_count sb:Datapath_Binding 4095
> --
> 2.45.2
>
> _______________________________________________
> dev mailing list
> d...@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>
>
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to