Navigation Menu

Skip to content

Commit

Permalink
Fix loss of CRI test failure status in CI
Browse files Browse the repository at this point in the history
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 <estesp@linux.vnet.ibm.com>
  • Loading branch information
estesp committed Aug 1, 2018
1 parent efb04a3 commit 9622369
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion .travis.yml
Expand Up @@ -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:
Expand Down

0 comments on commit 9622369

Please sign in to comment.