[LU-9422] sanity test 234 is skipped incorrectly for attr rpm version number

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.11.0
• Affects Version/s: Lustre 2.10.0
• Labels: test
• Severity: 3
• Assignee: James Nunez (Inactive)
• Reporter: James Nunez (Inactive)

Description

sanity test_234 is being skipped with the message

SKIP: sanity test_234 LU-3703: attr version getfattr 2.4.46 too old

Looking at the test code, the test is skipped due to the check

13747         # output of the form: attr 2 4 44 3 fc13 x86_64
13748         V=($(IFS=".-" rpm -q attr))
13749         if [[ ${V[1]} > 2 || ${V[2]} > 4 || ${V[3]} > 44 ||
13750               ${V[1]} = 2 && ${V[2]} = 4 && ${V[3]} = 44 && ${V[4]} > 6 ]]; then
13751                 # attr pre-2.4.44-7 had a bug with rc
13752                 # LU-3703 - SLES 11 and FC13 clients have older attr
13753                 getfattr -n user.attr $DIR/$tdir/$tfile &&
13754                         error "getfattr should have failed with ENOMEM"
13755         else
13756                 skip "LU-3703: attr version $(getfattr --version) too old"
13757         fi

From this description, this test should not be skipped with attr version 2.4.46. The problem is setting the Internal Field Separator (IFS) this way isn’t actually setting the correct values to split the string on.

This is easily fixed by setting the IFS outside the assignment/call to rpm, but is this check necessary any more?

Issue Links

• is related to: LU-3703 Failure on test suite sanity test_234: getfattr should have failed with ENOMEM

Comments

[Peter Jones]

Landed for 2.11

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/26906/
Subject: LU-9422 tests: generalize SLES version check
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 2182db2564fa6810bc3c35f51e0493a88a2f39a9

[Gerrit Updater]

James Nunez ([e-mail]) uploaded a new patch: https://review.whamcloud.com/26906
Subject: LU-9422 test: sanity 234 correct attr version check
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 7b9c18a824f4e224050da2bf3e7fbc9ea919fbc8