From 1de3c3455868670c289ed1d69f57244816ea39b5 Mon Sep 17 00:00:00 2001
From: Chris Horn <chris.horn@hpe.com>
Date: Fri, 22 Oct 2021 01:51:40 +0000
Subject: [PATCH] LU-15152 tests: auster reports wrong testsuite status

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 | 31 +++++++++++++++++--------------
 1 file changed, 17 insertions(+), 14 deletions(-)

diff --git a/lustre/tests/auster b/lustre/tests/auster
index 7e2d61c..423eec2 100755
--- a/lustre/tests/auster
+++ b/lustre/tests/auster
@@ -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]}
 }
 
 #
-- 
1.8.3.1