From 9622369f0eaee16a9ac752c216b88eb8863dc31a Mon Sep 17 00:00:00 2001 From: Phil Estes Date: Tue, 31 Jul 2018 23:04:15 -0400 Subject: [PATCH] Fix loss of CRI test failure status in CI Prior PR fixed the wrong use of `exit` built-in within a Travis script, but lost the reporting of a failure result of CRI testing in the process. Signed-off-by: Phil Estes --- .travis.yml | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/.travis.yml b/.travis.yml index 1bfe3a214..99315fe5c 100644 --- a/.travis.yml +++ b/.travis.yml @@ -74,8 +74,10 @@ script: sudo PATH=$PATH containerd -log-level debug &> /tmp/containerd-cri.log & sudo ctr version ; sudo PATH=$PATH GOPATH=$GOPATH critest --runtime-endpoint=/var/run/containerd/containerd.sock --parallel=8 ; - test $? -ne 0 && cat /tmp/containerd-cri.log ; + TEST_RC=$? ; + test $TEST_RC -ne 0 && cat /tmp/containerd-cri.log ; sudo pkill containerd ; + test $TEST_RC -eq 0 || /bin/false ; fi after_success: