It seems worthwhile to ensure that this call does in fact time out.

Signed-off-by: Ben Pfaff <[email protected]>
---
 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

Reply via email to