[LU-6891] fix grow_xattr() defect Created: 21/Jul/15  Updated: 17/May/16  Resolved: 10/Aug/15

Status: Resolved
Project: Lustre
Component/s: None
Affects Version/s: Lustre 2.7.0
Fix Version/s: Lustre 2.8.0

Type: Bug Priority: Minor
Reporter: Elena Gryaznova Assignee: WC Triage
Resolution: Fixed Votes: 0
Labels: None

Epic/Theme: patch, test
Severity: 3
Rank (Obsolete): 9223372036854775807

 Description    
grow_xattr() {
...
   [ -z $(lctl get_param -n mdc.*.connect_flags | grep xattr) ] &&
         skip "must have user_xattr" && return 0
..
}

string needs to be quoted to avoid syntax error on DNE setup:
mdscount=2 :

# lctl get_param  mdc.*.connect_flags | grep xattr
xattr
xattr
# find /proc/fs/lustre/ -name connect_flags | grep mdc
/proc/fs/lustre/mdc/lustre-MDT0001-mdc-ffff88005a998000/connect_flags
/proc/fs/lustre/mdc/lustre-MDT0000-mdc-ffff88005a998000/connect_flags

xtrace:

+ grow_xattr 65536
+ local xsize=65536
+ local file=/mnt/lustre/f102ha.sanity
++ grep xattr
++ lctl get_param -n 'mdc.*.connect_flags'
+ '[' -z xattr xattr ']'
sanity.sh: line 6447: [: xattr: binary operator expected

mdscount=3 :

++ lctl get_param -n 'mdc.*.connect_flags'
+ '[' -z xattr xattr xattr ']'
sanity.sh: line 6447: [: too many arguments


 Comments   
Comment by Gerrit Updater [ 21/Jul/15 ]

Elena Gryaznova (elena.gryaznova@seagate.com) uploaded a new patch: http://review.whamcloud.com/15672
Subject: LU-6891 test: fix grow_xattr() defect
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 1b0efdb3fd02978741e77e30adcdf1f2404a3e26

Comment by Gerrit Updater [ 09/Aug/15 ]

Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/15672/
Subject: LU-6891 test: fix grow_xattr() defect
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 683b4b29540bf100725adf9b9b4412b6324734e9

Comment by Peter Jones [ 10/Aug/15 ]

Landed for 2.8

Generated at Sat Feb 10 02:04:10 UTC 2024 using Jira 9.4.14#940014-sha1:734e6822bbf0d45eff9af51f82432957f73aa32c.