[ovs-dev] [PATCH ovn v2 3/6] tests: Improve checking in "test unixctl" test.

[Numan Siddique]

On Thu, Nov 5, 2020 at 10:41 AM Ben Pfaff <blp at ovn.org> wrote:
>
> It seems worthwhile to ensure that this call does in fact time out.
>
> Signed-off-by: Ben Pfaff <blp at ovn.org>

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

Numan

> ---
>  tests/ovn-northd.at | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/tests/ovn-northd.at b/tests/ovn-northd.at
> index 0bf20c1a7053..26376c367178 100644
> --- a/tests/ovn-northd.at
> +++ b/tests/ovn-northd.at
> @@ -258,8 +258,11 @@ AT_CHECK([ovn-nbctl --wait=sb lsp-del p1])
>  OVS_APP_EXIT_AND_WAIT_BY_TARGET(["$ovs_base"/northd/ovn-northd.ctl], ["$ovs_base"/northd/ovn-northd.pid])
>
>  # Check no port_binding entry for new port as ovn-northd is not running
> +#
> +# 142 is 128+14, the exit status that the shell reports when a
> +# process exits due to SIGARLM (signal 14).
>  ovn-nbctl lsp-add sw p2
> -ovn-nbctl --timeout=10 --wait=sb sync
> +AT_CHECK([ovn-nbctl --timeout=10 --wait=sb sync], [142], [], [ignore])
>  check_row_count Port_Binding 0 logical_port=p2
>
>  # test default unixctl path
> --
> 2.26.2
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>