Whamcloud - gitweb
LU-15152 tests: auster reports wrong testsuite status 43/45343/3
authorChris Horn <chris.horn@hpe.com>
Fri, 22 Oct 2021 01:51:40 +0000 (01:51 +0000)
committerOleg Drokin <green@whamcloud.com>
Sat, 20 Nov 2021 06:28:44 +0000 (06:28 +0000)
auster always reports testsuites returned 0 even when there
are failures.

Test-Parameters: trivial testlist=sanity-lnet env=ONLY=230,ONLY_REPEAT=20
Signed-off-by: Chris Horn <chris.horn@hpe.com>
Change-Id: I7a8101d6cfd854d8419edf55c18a72e211f5e5c8
Reviewed-on: https://review.whamcloud.com/45343
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: James Nunez <jnunez@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
lustre/tests/auster

index 7e2d61c..423eec2 100755 (executable)
@@ -214,27 +214,30 @@ run_suite() {
        [[ ! -f $TF_SKIP ]] || rm -f $TF_SKIP
 
        reset_lustre
+
+       return $rc
 }
 
 run_suite_logged() {
-    local suite_name=${1%.sh}
-    local suite=$(echo ${suite_name} | tr "[:lower:]-" "[:upper:]_")
+       local suite_name=${1%.sh}
+       local suite=$(echo ${suite_name} | tr "[:lower:]-" "[:upper:]_")
 
-    suite_script=$(find_script_in_path $suite_name $LUSTRE/tests)
+       suite_script=$(find_script_in_path $suite_name $LUSTRE/tests)
 
-    if [[ -z $suite_script ]]; then
-        echo "Can't find test script for $suite_name"
-        return 1
-    fi
+       if [[ -z $suite_script ]]; then
+               echo "Can't find test script for $suite_name"
+               return 1
+       fi
 
-    echo "run_suite $suite_name $suite_script"
-    local log_name=${suite_name}.suite_log.$(hostname -s).log
-    if $verbose; then
-        run_suite $suite_name $suite_script 2>&1 |tee  $LOGDIR/$log_name
-    else
-        run_suite $suite_name $suite_script > $LOGDIR/$log_name 2>&1
-    fi
+       echo "run_suite $suite_name $suite_script"
+       local log_name=${suite_name}.suite_log.$(hostname -s).log
+       if $verbose; then
+               run_suite $suite_name $suite_script 2>&1 |tee  $LOGDIR/$log_name
+       else
+               run_suite $suite_name $suite_script > $LOGDIR/$log_name 2>&1
+       fi
 
+       return ${PIPESTATUS[0]}
 }
 
 #