[LU-11215] conf-sanity test_61: Invalid filesystem option set: large_xattr, Clearing filesystem feature 'ea_inode' not supported

• Status: Resolved
• Project: Lustre
• Fix Version/s: Lustre 2.12.0, Lustre 2.10.6
• Type: Bug
• Priority: Major
• Reporter: Maloo
• Assignee: Dongyang Li
• Resolution: Fixed
• Severity: 3

Description

This issue was created by maloo for Dongyang Li <[e-mail]>

This issue relates to the following test suite run: https://testing.whamcloud.com/test_sets/010e8a40-97d1-11e8-87f3-52540065bddc

test_61 failed with the following error:

tune2fs on mds 1 failed

The e2fsprogs upstream large_xattr patch doesn't add "large_xattr" as the feature name.
It is still used by our test scripts.

VVVVVVV DO NOT REMOVE LINES BELOW, Added by Maloo for auto-association VVVVVVV
conf-sanity test_61 - tune2fs on mds 1 failed

Issue Links

• is related to: LU-1732 enable wide striping by default
• is related to: LU-11246 New lustre e2fsprogs 1.44 issues
• is related to: LU-11440 Make e2fsprogs-1.44.3-wc1 release
• is related to: LU-12154 interop: conf-sanity test_61: Clearin...

Comments

[Andreas Dilger]

Dongyang, can you submit a Lustre patch that replaces the use of large_xattr with ea_inode. That should work for all versions of Lustre that we are doing interop testing with.

[Andreas Dilger]

Ah, I see your comments in https://review.whamcloud.com/32932 - I agree we can revive patch https://review.whamcloud.com/4315 instead of making a separate one.

[Andreas Dilger]

Dongyang, it might make sense to separate out the "ea_inode" parts of https://review.whamcloud.com/4315 into a separate patch, since there appear to be other problems with that patch interacting badly with another recently-added patch. That would fix the test scripts in one patch that can be marked with "Test-Parameters: trivial testlist=conf-sanity", and a second patch that enables the ea_inode feature that can land once we've resolved problems with enabling that feature.

[Gerrit Updater]

Li Dongyang ([e-mail]) uploaded a new patch: https://review.whamcloud.com/33012
Subject: LU-11215 tests: replace "large_xattr" with "ea_inode"
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: e9212bf78259151b159bd090260cdf9065719bbd

[Dongyang Li]

I'll modify https://review.whamcloud.com/4315 to enable ea_inode by default once the problems are fixed.

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/33012/
Subject: LU-11215 tests: replace "large_xattr" with "ea_inode"
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 900a1b6e4271a6c1903d6723082a01c98defe7b2

[Andreas Dilger]

Reopen to track patch https://review.whamcloud.com/32932 "libe2p: add large_xattr as an alias of ea_inode" landing.

[Gerrit Updater]

Andreas Dilger ([e-mail]) merged in patch https://review.whamcloud.com/32932/
Subject: LU-11215 libe2p: add large_xattr as an alias of ea_inode
Project: tools/e2fsprogs
Branch: master-lustre-test
Current Patch Set:
Commit: b0dce56164783250497dec0b725f90cf877602b1

[Gerrit Updater]

Jian Yu ([e-mail]) uploaded a new patch: https://review.whamcloud.com/33503
Subject: LU-11215 tests: replace "large_xattr" with "ea_inode"
Project: fs/lustre-release
Branch: b2_10
Current Patch Set: 1
Commit: dadf5122efedde4715a236d0a9d8065c36bdd6f1

[Gerrit Updater]

John L. Hammond ([e-mail]) merged in patch https://review.whamcloud.com/33503/
Subject: LU-11215 tests: replace "large_xattr" with "ea_inode"
Project: fs/lustre-release
Branch: b2_10
Current Patch Set:
Commit: c7ac1c5416e9011cb7c36bc7bdf072ff3065f55a

[Andreas Dilger]

Later failures reported (for search purposes):

Clearing filesystem feature 'ea_inode' not supported.