[ovs-dev] [PATCH ovn] ovn-ipsec.at: Fix occasional failure of test "unsupported multiple ovn-encap-ip values".

[Numan Siddique]

On Wed, Apr 28, 2021 at 9:04 PM Han Zhou <hzhou at ovn.org> wrote:
>
> This test case is very likely to fail with address sanitizer enabled.
> This patch fixes the timing problem in the test case.
>
> Signed-off-by: Han Zhou <hzhou at ovn.org>

Thanks for fixing this.

Acked-by: Numan Siddique <numans at ovn.org>

Numan

> ---
>  tests/ovn-ipsec.at | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tests/ovn-ipsec.at b/tests/ovn-ipsec.at
> index 887281d5b..8f1cad54c 100644
> --- a/tests/ovn-ipsec.at
> +++ b/tests/ovn-ipsec.at
> @@ -99,6 +99,6 @@ ovs-vsctl \
>      -- set Open_vSwitch . other_config:private_key=dummy-privkey.pem \
>      -- set Open_vSwitch . other_config:ca_cert=dummy-cacert.pem
>
> -AT_CHECK([grep "ovn-encap-ip has been configured as a list. This is unsupported for IPsec." hv1/ovn-controller.log],[0], ignore)
> +OVS_WAIT_UNTIL([grep "ovn-encap-ip has been configured as a list. This is unsupported for IPsec." hv1/ovn-controller.log])
>
> -AT_CLEANUP
> \ No newline at end of file
> +AT_CLEANUP
> --
> 2.30.2
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>