This is an automated email from the ASF dual-hosted git repository.
bneradt pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/trafficserver-ci.git
The following commit(s) were added to refs/heads/main by this push:
new c2347a5 regression.sh: ctest exit with 1 on failure (#338)
c2347a5 is described below
commit c2347a58d12909e447e7463667aa30a225ccedc1
Author: Brian Neradt <[email protected]>
AuthorDate: Tue Apr 23 21:37:59 2024 -0500
regression.sh: ctest exit with 1 on failure (#338)
Without the explicit `exit 1` the script finishes with an exit code of 8
which was maybe making it so the tests didn't fail. Maybe? In any case,
this shouldn't hurt and may make the failure explicit in jenkins.
---
jenkins/bin/regression.sh | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/jenkins/bin/regression.sh b/jenkins/bin/regression.sh
index bc01738..80ddf90 100755
--- a/jenkins/bin/regression.sh
+++ b/jenkins/bin/regression.sh
@@ -34,7 +34,7 @@ echo -n "Unit tests started at " && date
if [ -d cmake ]
then
pushd build
- ctest -j${NPROC} --output-on-failure --no-compress-output -T Test
+ ctest -j${NPROC} --output-on-failure --no-compress-output -T Test ||
exit 1
popd
else
${ATS_MAKE} -j${NPROC} check VERBOSE=Y V=1 || exit 1
