On Thu, Nov 5, 2020 at 10:41 AM Ben Pfaff <[email protected]> wrote: > > It seems worthwhile to ensure that this call does in fact time out. > > Signed-off-by: Ben Pfaff <[email protected]>
Acked-by: Numan Siddique <[email protected]> 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 > [email protected] > https://mail.openvswitch.org/mailman/listinfo/ovs-dev > _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
