error_ignore is called when a test and an error is encountered
that is supposed to be ignored. The first argument to
error_ignore is a bug number, justification as to why the error
should be ignored, that the routine assumes is a bugzilla bug
number and automatically attaches a “bz” prefix to the number.
This patch removes the assumption that all bug numbers are bugzilla
numbers and moves adding the “bz” to the bug number to the
individual calls to error_ignore. Now, we can use “LU” bug
numbers with error ignore.
Signed-off-by: James Nunez <james.a.nunez@intel.com>
Change-Id: I40c47ccf6af76e76bff99c67816e0d2aebd37e71
Reviewed-on: http://review.whamcloud.com/7791
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Minh Diep <minh.diep@intel.com>
Reviewed-by: Jian Yu <jian.yu@intel.com>
# is / should be
max=30
diff=$((($files1 - $files2) * 100 / $files1))
# is / should be
max=30
diff=$((($files1 - $files2) * 100 / $files1))
- echo "No pool / wide pool: $diff %."
- [ $diff -gt $max ] &&
- error_ignore 23408 "Degradation with wide pool is $diff% > $max%"
+ echo "No pool / wide pool: $diff %."
+ [ $diff -gt $max ] &&
+ error_ignore bz23408 "Degradation with wide pool is $diff% > $max%"
- max=30
- diff=$((($files1 - $files3) * 100 / $files1))
- echo "No pool / missing pool: $diff %."
- [ $diff -gt $max ] &&
- error_ignore 23408 "Degradation with wide pool is $diff% > $max%"
+ max=30
+ diff=$((($files1 - $files3) * 100 / $files1))
+ echo "No pool / missing pool: $diff %."
+ [ $diff -gt $max ] &&
+ error_ignore bz23408 "Degradation with wide pool is $diff% > $max%"
}
run_test 18 "File create in a directory which references a deleted pool"
}
run_test 18 "File create in a directory which references a deleted pool"
rc=$?
lctl set_param fail_loc=0x0
client_reconnect
rc=$?
lctl set_param fail_loc=0x0
client_reconnect
- [ $rc -eq 0 ] && error_ignore 5494 "multiop didn't fail fsync: rc $rc" || true
+ [ $rc -eq 0 ] &&
+ error_ignore bz5494 "multiop didn't fail fsync: rc $rc" || true
}
run_test 24a "fsync error (should return error)"
}
run_test 24a "fsync error (should return error)"
lctl set_param fail_loc=0x0
client_reconnect
[ $rc1 -eq 0 -o $rc2 -eq 0 ] &&
lctl set_param fail_loc=0x0
client_reconnect
[ $rc1 -eq 0 -o $rc2 -eq 0 ] &&
- error_ignore 5494 "multiop didn't fail fsync: $rc1 or close: $rc2" ||
+ error_ignore bz5494 "multiop didn't fail fsync: $rc1 or close: $rc2" ||
- dmesg | grep "dirty page discard:" || \
+ dmesg | grep "dirty page discard:" ||
error "no discarded dirty page found!"
}
run_test 24b "test dirty page discard due to client eviction"
error "no discarded dirty page found!"
}
run_test 24b "test dirty page discard due to client eviction"
rc=$?
echo writemany returned $rc
#these may fail because of eviction due to slow AST response.
rc=$?
echo writemany returned $rc
#these may fail because of eviction due to slow AST response.
- [ $rc -eq 0 ] || error_ignore 13652 "writemany returned rc $rc" || true
+ [ $rc -eq 0 ] ||
+ error_ignore bz13652 "writemany returned rc $rc" || true
}
run_test 50 "failover MDS under load"
}
run_test 50 "failover MDS under load"
# and recovery was interrupted
sleep $TIMEOUT
kill -USR1 $CLIENT_PID
# and recovery was interrupted
sleep $TIMEOUT
kill -USR1 $CLIENT_PID
rc=$?
echo writemany returned $rc
rc=$?
echo writemany returned $rc
- [ $rc -eq 0 ] || error_ignore 13652 "writemany returned rc $rc" || true
+ [ $rc -eq 0 ] ||
+ error_ignore bz13652 "writemany returned rc $rc" || true
}
run_test 51 "failover MDS during recovery"
}
run_test 51 "failover MDS during recovery"
run_test 59 "Read cancel race on client eviction"
err17935 () {
run_test 59 "Read cancel race on client eviction"
err17935 () {
- # we assume that all md changes are in the MDT0 changelog
- if [ $MDSCOUNT -gt 1 ]; then
- error_ignore 17935 $*
- else
- error $*
- fi
+ # we assume that all md changes are in the MDT0 changelog
+ if [ $MDSCOUNT -gt 1 ]; then
+ error_ignore bz17935 $*
+ else
+ error $*
+ fi
CONN2=$(lctl get_param -n osc.*.stats | awk '/_connect/ {total+=$2} END {print total}')
ATTEMPTS=$(($CONN2 - $CONN1))
echo "$ATTEMPTS osc reconnect attempts on gradual slow"
CONN2=$(lctl get_param -n osc.*.stats | awk '/_connect/ {total+=$2} END {print total}')
ATTEMPTS=$(($CONN2 - $CONN1))
echo "$ATTEMPTS osc reconnect attempts on gradual slow"
- [ $ATTEMPTS -gt 0 ] && error_ignore 13721 "AT should have prevented reconnect"
- return 0
+ [ $ATTEMPTS -gt 0 ] &&
+ error_ignore bz13721 "AT should have prevented reconnect"
+ return 0
}
run_test 67a "AT: verify slow request processing doesn't induce reconnects"
}
run_test 67a "AT: verify slow request processing doesn't induce reconnects"
echo "ok"
else
echo "failed - QOS mode won't be used"
echo "ok"
else
echo "failed - QOS mode won't be used"
- error_ignore 0000 "QOS imbalance criteria not met"
+ skip "QOS imbalance criteria not met"
simple_cleanup_common
return
fi
simple_cleanup_common
return
fi
[ $MINC -gt 0 ] &&
echo "Wrote ${FILL}% more files to larger OST $MAXI1"
[ $MAXC -gt $MINC ] ||
[ $MINC -gt 0 ] &&
echo "Wrote ${FILL}% more files to larger OST $MAXI1"
[ $MAXC -gt $MINC ] ||
- error_ignore 0000 "stripe QOS didn't balance free space"
-
+ error_ignore LU-9 "stripe QOS didn't balance free space"
simple_cleanup_common
}
run_test 116a "stripe QOS: free space balance ==================="
simple_cleanup_common
}
run_test 116a "stripe QOS: free space balance ==================="
BEFORE=`roc_hit`
cat $file >/dev/null
AFTER=`roc_hit`
BEFORE=`roc_hit`
cat $file >/dev/null
AFTER=`roc_hit`
- if ! let "AFTER - BEFORE == 0"; then
- error_ignore 20762 "IN CACHE: before: $BEFORE, after: $AFTER"
- else
- log "cache hits:: before: $BEFORE, after: $AFTER"
- fi
-
+ if ! let "AFTER - BEFORE == 0"; then
+ error_ignore bz20762 "IN CACHE: before: $BEFORE, after: $AFTER"
+ else
+ log "cache hits:: before: $BEFORE, after: $AFTER"
+ fi
log "Turn on the read cache and turn off the write cache"
set_cache read on
log "Turn on the read cache and turn off the write cache"
set_cache read on
cancel_lru_locks osc
cat $file >/dev/null
AFTER=`roc_hit`
cancel_lru_locks osc
cat $file >/dev/null
AFTER=`roc_hit`
- if ! let "AFTER - BEFORE == 0"; then
- error_ignore 20762 "IN CACHE: before: $BEFORE, after: $AFTER"
- else
- log "cache hits:: before: $BEFORE, after: $AFTER"
- fi
+ if ! let "AFTER - BEFORE == 0"; then
+ error_ignore bz20762 "IN CACHE: before: $BEFORE, after: $AFTER"
+ else
+ log "cache hits:: before: $BEFORE, after: $AFTER"
+ fi
log "Read again; it should be satisfied from the cache."
BEFORE=`roc_hit`
log "Read again; it should be satisfied from the cache."
BEFORE=`roc_hit`
#Changelogs
err17935 () {
if [ $MDSCOUNT -gt 1 ]; then
#Changelogs
err17935 () {
if [ $MDSCOUNT -gt 1 ]; then
+ error_ignore bz17935 $*
# use only if we are ignoring failures for this test, bugno required.
# (like ALWAYS_EXCEPT, but run the test and ignore the results.)
# use only if we are ignoring failures for this test, bugno required.
# (like ALWAYS_EXCEPT, but run the test and ignore the results.)
-# e.g. error_ignore 5494 "your message"
+# e.g. error_ignore bz5494 "your message" or
+# error_ignore LU-5494 "your message"
- local TYPE="IGNORE (bz$1)"
+ local TYPE="IGNORE ($1)"
shift
error_noexit "$@"
}
shift
error_noexit "$@"
}