ovs-appctl can wait indefinitely while executing an exit for a dead service. Let's add a timeout (10 seconds should be reasonable) to exit calls to avoid hanging up of the testsuite in such cases.
Signed-off-by: Ilya Maximets <[email protected]> --- tests/ovs-macros.at | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/ovs-macros.at b/tests/ovs-macros.at index dbce0a5..54c096c 100644 --- a/tests/ovs-macros.at +++ b/tests/ovs-macros.at @@ -196,8 +196,8 @@ m4_define([OVS_APP_EXIT_AND_WAIT], [AT_CHECK([test -e $OVS_RUNDIR/$1.pid]) TMPPID=$(cat $OVS_RUNDIR/$1.pid) AT_CHECK(m4_if([$1],[ovs-vswitchd], - [ovs-appctl -t $1 exit --cleanup], - [ovs-appctl -t $1 exit])) + [ovs-appctl --timeout=10 -t $1 exit --cleanup], + [ovs-appctl --timeout=10 -t $1 exit])) OVS_WAIT_WHILE([kill -0 $TMPPID 2>/dev/null])]) dnl OVS_APP_EXIT_AND_WAIT_BY_TARGET(TARGET, PIDFILE) @@ -207,7 +207,7 @@ dnl argument), and then wait for it to exit. m4_define([OVS_APP_EXIT_AND_WAIT_BY_TARGET], [AT_CHECK([test -e $2]) TMPPID=$(cat $2) - AT_CHECK([ovs-appctl --target=$1 exit]) + AT_CHECK([ovs-appctl --timeout=10 --target=$1 exit]) OVS_WAIT_WHILE([kill -0 $TMPPID 2>/dev/null])]) dnl on_exit "COMMAND" -- 2.7.4 _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
