[LU-3880] Make error_ignore accept a general string for bug number Created: 04/Sep/13  Updated: 25/Oct/13  Resolved: 25/Oct/13

Status: Resolved
Project: Lustre
Component/s: None
Affects Version/s: Lustre 2.4.1, Lustre 2.5.0, Lustre 2.6.0
Fix Version/s: Lustre 2.6.0

Type: Improvement Priority: Minor
Reporter: James Nunez (Inactive) Assignee: James Nunez (Inactive)
Resolution: Fixed Votes: 0
Labels: None

Issue Links:
Related
is related to LU-3764 sanity test_116a: stripe QOS didn't b... Resolved
Rank (Obsolete): 10063

 Description   

The error_ignore is a routine that allows a test to run and, if an error occurs, the error is ignored, but a message is printed. The message is composed of a bug number and user defined message. The bug number is assumed to be a bugzilla number:

local TYPE="IGNORE (bz$1)"

The error_ignore routine should be changed from assuming that the first parameter is a bugzilla number to just take the whole ticket number (bzXXX or LU-XXX or any string).

There are 13 calls to error_ignore that will need to be modified:
ost-pools.sh: error_ignore 23408 "Degradation with wide pool is $diff% > $max%"
ost-pools.sh: error_ignore 23408 "Degradation with wide pool is $diff% > $max%"
recovery-small.sh: [ $rc -eq 0 ] && error_ignore 5494 "multiop didn't fail fsync: rc $rc" || true
recovery-small.sh: error_ignore 5494 "multiop didn't fail fsync: $rc1 or close: $rc2" ||
recovery-small.sh: [ $rc -eq 0 ] || error_ignore 13652 "writemany returned rc $rc" || true
recovery-small.sh: [ $rc -eq 0 ] || error_ignore 13652 "writemany returned rc $rc" || true
recovery-small.sh: error_ignore 17935 $*
replay-single.sh: [ $ATTEMPTS -gt 0 ] && error_ignore 13721 "AT should have prevented reconnect"
sanity.sh: error_ignore "QOS imbalance criteria not met"
sanity.sh: [ $MAXC -gt $MINC ] || error_ignore "stripe QOS didn't balance free space"
sanity.sh: error_ignore 20762 "IN CACHE: before: $BEFORE, after: $AFTER"
sanity.sh: error_ignore 20762 "IN CACHE: before: $BEFORE, after: $AFTER"
sanity.sh: error_ignore 17935 $*



 Comments   
Comment by James Nunez (Inactive) [ 25/Sep/13 ]

Patch is ready, but waiting for LU-3640 patch ( http://review.whamcloud.com/#/c/7132/ ) to land before submitting since both patches modify some common lines of code.

Comment by James Nunez (Inactive) [ 30/Sep/13 ]

Proposed Patch at: http://review.whamcloud.com/#/c/7791/

Comment by James Nunez (Inactive) [ 25/Oct/13 ]

Landed to master.

Generated at Sat Feb 10 01:37:43 UTC 2024 using Jira 9.4.14#940014-sha1:734e6822bbf0d45eff9af51f82432957f73aa32c.