diff --git a/lustre/tests/sanity.sh b/lustre/tests/sanity.sh index 484c7e626243a027556d9a6f0ee7355f1bd8f4f7..fce0b63947e444b49a509135781e560576b321f8 100644 --- a/lustre/tests/sanity.sh +++ b/lustre/tests/sanity.sh @@ -5158,9 +5158,10 @@ test_127() { # bug 15521 $LCTL get_param osc.*0000-osc-*.stats | grep samples > $DIR/${tfile}.tmp while read NAME COUNT SAMP UNIT MIN MAX SUM SUMSQ; do - eval $NAME=$COUNT echo "got $COUNT $NAME" - + [ ! $MIN ] && error "Missing min value for $NAME proc entry" + eval $NAME=$COUNT || error "Wrong proc format" + case $NAME in read_bytes|write_bytes) [ $MIN -lt 4096 ] && error "min is too small: $MIN" @@ -5178,8 +5179,10 @@ test_127() { # bug 15521 done < $DIR/${tfile}.tmp #check that we actually got some stats - [ "$read_bytes" ] || error "no read done" - [ "$write_bytes" ] || error "no write done" + [ "$read_bytes" ] || error "Missing read_bytes stats" + [ "$write_bytes" ] || error "Missing write_bytes stats" + [ "$read_bytes" = 0 ] && error "no read done" + [ "$write_bytes" = 0 ] && error "no write done" } run_test 127 "verify the client stats are sane"