b=24426 run_one(): run error() once

there is no reason to run error() (and lctl dk thereby) more than once
second lctl dk overwrites the most important logs obtained on first lctl dk

i=grev

diff --git a/lustre/tests/test-framework.sh b/lustre/tests/test-framework.sh
index b5c0bf5..8cfda77 100644 (file)
--- a/lustre/tests/test-framework.sh
+++ b/lustre/tests/test-framework.sh
@@ -2824,9 +2824,9 @@ run_one() {
     #check_mds
     cd $SAVE_PWD
     reset_fail_loc
-    check_grant ${testnum} || error "check_grant $testnum failed with $?"
-    check_catastrophe || error "LBUG/LASSERT detected"
-    ps auxww | grep -v grep | grep -q multiop && error "multiop still running"
+    check_grant ${testnum} || $TEST_FAILED || error "check_grant $testnum failed"
+    check_catastrophe || $TEST_FAILED || error "LBUG/LASSERT detected"
+    ps auxww | grep -v grep | grep -q multiop && ($TEST_FAILED || error "multiop still running")
     pass "($((`date +%s` - $BEFORE))s)"
     TEST_FAILED=false
     unset TESTNAME