explicitly specify executor to be bash for tests
We use prove command to run tests. Sometimes tests fail rather strangely with error as shown below: Not a perl script at ./tests/bugs/core/bug-1111557.t line 1. ./tests/bugs/core/bug-1111557.t .. Dubious, test returned 25 (wstat 6400, 0x1900) No subtests run https://build.gluster.org/job/centos6-regression/3818/consoleFull https://build.gluster.org/job/centos6-regression/3819/consoleFull https://build.gluster.org/job/centos6-regression/3801/consoleFull Quote from man page `` "--exec" Normally you can just pass a list of Perl tests and the harness will know how to execute them. However, if your tests are not written in Perl or if you want all tests invoked exactly the same way, use the "-e", or "--exec" switch: prove --exec '/usr/bin/ruby -w' t/ ``` Hence, better to comply with recommended practice although it might not be the reason for this failure. Change-Id: If7a0baf20698f8497ef3e8fc422fa67063a4651f BUG: 1438858 Signed-off-by: Raghavendra Talur <rtalur@redhat.com> Reviewed-on: https://review.gluster.org/16996 Smoke: Gluster Build System <jenkins@build.gluster.org> Reviewed-by: N Balachandran <nbalacha@redhat.com> NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org> Reviewed-by: Nigel Babu <nigelb@redhat.com> CentOS-regression: Gluster Build System <jenkins@build.gluster.org> Reviewed-by: Jeff Darcy <jeff@pl.atyp.us>
This commit is contained in:
parent
c5a4a77848
commit
d6b88e9b8b
@ -268,7 +268,7 @@ function run_tests()
|
||||
total_run_tests=$((total_run_tests+1))
|
||||
echo "[$(date +%H:%M:%S)] Running tests in file $t"
|
||||
starttime="$(date +%s)"
|
||||
prove -vf $t
|
||||
prove -vfe '/bin/bash' $t
|
||||
TMP_RES=$?
|
||||
ELAPSEDTIMEMAP[$t]=`expr $(date +%s) - $starttime`
|
||||
if [ ${TMP_RES} -ne 0 ] && [ "x${retry}" = "xyes" ] ; then
|
||||
@ -280,7 +280,7 @@ function run_tests()
|
||||
echo " * we got some spurous failures *"
|
||||
echo " *********************************"
|
||||
echo ""
|
||||
prove -vf $t
|
||||
prove -vfe '/bin/bash' $t
|
||||
TMP_RES=$?
|
||||
fi
|
||||
if [ ${TMP_RES} -ne 0 ] ; then
|
||||
|
Loading…
x
Reference in New Issue
Block a user