diff options
| author | Gert Doering | 2016-09-13 22:04:58 +0200 |
|---|---|---|
| committer | David Sommerseth | 2016-09-16 17:05:36 +0200 |
| commit | 0043ddbbe6768ac560996e9879f47381e043a116 (patch) | |
| tree | 8af3e0278b78c2815961b261c821bd24992bf9b9 | |
| parent | 0b4707472675e809b924fd8ca907534e0da16dbb (diff) | |
| download | openvpn-0043ddbbe6768ac560996e9879f47381e043a116.zip openvpn-0043ddbbe6768ac560996e9879f47381e043a116.tar.gz | |
Do not abort t_client run if OpenVPN instance does not start.
Basically, an oversight - if one test instance does not start at all
(due to "tap driver not loaded") the whole script would exit, instead
of logging the failing instance and proceeding to the next test run.
Signed-off-by: Gert Doering <gert@greenie.muc.de>
Acked-by: David Sommerseth <davids@openvpn.net>
Message-Id: 20160913200458.9906-1-gert@greenie.muc.de
URL: http://www.mail-archive.com/search?l=mid&q=20160913200458.9906-1-gert@greenie.muc.de
Signed-off-by: David Sommerseth <davids@openvpn.net>
(cherry picked from commit a7b02f7f660707f765881f35867b4d23d89b390f)
| -rwxr-xr-x | tests/t_client.sh.in | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/tests/t_client.sh.in b/tests/t_client.sh.in index ccf6b26..032d0d5 100755 --- a/tests/t_client.sh.in +++ b/tests/t_client.sh.in @@ -249,10 +249,14 @@ do if $RUN_SUDO kill -0 $opid then : else - echo -e "OpenVPN process has failed to start up, check log ($LOGDIR/$SUF:openvpn.log). FAIL.\ntail of logfile follows:\n..." >&2 - tail $LOGDIR/$SUF:openvpn.log >&2 + fail "OpenVPN process has failed to start up, check log ($LOGDIR/$SUF:openvpn.log)." + echo "tail -5 $SUF:openvpn.log" >&2 + tail -5 $LOGDIR/$SUF:openvpn.log >&2 + echo -e "\nFAIL. skip rest of sub-tests for test run $SUF.\n" >&2 trap - 0 1 2 3 15 - exit 10 + SUMMARY_FAIL="$SUMMARY_FAIL $SUF" + exit_code=30 + continue fi # compare whether anything changed in ifconfig/route setup? |