[LU-14281] sanity-sec test 56 fails with 'filefrag /mnt/lustre/d56.sanity-sec/f56.sanity-sec failed'

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.14.0
• Affects Version/s: Lustre 2.14.0
• Labels: rhel8.3
• Environment: RHEL8.3, ZFS, DNE
• Severity: 3
• Assignee: Sebastien Buisson
• Reporter: James Nunez (Inactive)

Description

The patch https://review.whamcloud.com/#/c/40852/ “LU-14149 llite: fiemap set flags for encrypted files” adds test_56 to sanity-sec. We are seeing this test fail, so far, only for RHEL8.3, ZFS, and DNE.

Looking at the suite_log for the failure at https://testing.whamcloud.com/test_sets/55445adc-5706-47c6-a6ea-9f5df56a0fe4, we see

== sanity-sec test 56: FIEMAP on encrypted file ====================================================== 23:45:05 (1609199105)
10.9.4.110@tcp:/lustre /mnt/lustre lustre rw,flock,user_xattr,lazystatfs,encrypt 0 0
CMD: trevis-10vm1.trevis.whamcloud.com grep -c /mnt/lustre' ' /proc/mounts
Stopping client trevis-10vm1.trevis.whamcloud.com /mnt/lustre (opts:)
CMD: trevis-10vm1.trevis.whamcloud.com lsof -t /mnt/lustre
CMD: trevis-10vm1.trevis.whamcloud.com umount  /mnt/lustre 2>&1
Starting client: trevis-10vm1.trevis.whamcloud.com:  -o user_xattr,flock,test_dummy_encryption trevis-10vm4@tcp:/lustre /mnt/lustre
CMD: trevis-10vm1.trevis.whamcloud.com mkdir -p /mnt/lustre
CMD: trevis-10vm1.trevis.whamcloud.com mount -t lustre -o user_xattr,flock,test_dummy_encryption trevis-10vm4@tcp:/lustre /mnt/lustre
3+0 records in
3+0 records out
3145728 bytes (3.1 MB, 3.0 MiB) copied, 1.26556 s, 2.5 MB/s
/mnt/lustre/d56.sanity-sec/f56.sanity-sec: FIBMAP unsupported
Filesystem type is: bd00bd0
File size of /mnt/lustre/d56.sanity-sec/f56.sanity-sec is 3145728 (3072 blocks of 1024 bytes)
 sanity-sec test_56: @@@@@@ FAIL: filefrag /mnt/lustre/d56.sanity-sec/f56.sanity-sec failed 
  Trace dump:
  = /usr/lib64/lustre/tests/test-framework.sh:6273:error()
  = /usr/lib64/lustre/tests/sanity-sec.sh:4162:test_56()

Logs for more failures at
https://testing.whamcloud.com/test_sets/4c131661-0929-415f-8d4a-71d2c47f97cb
https://testing.whamcloud.com/test_sets/31562510-3417-4b4e-b272-80499b2d4d3a

Issue Links

• is related to: LU-14149 FIEMAP should set FIEMAP_EXTENT_DATA_ENCRYPTED for fscrypt files

Comments

[Peter Jones]

Landed for 2.14

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/41130/
Subject: LU-14281 tests: fix sanity-sec test_56 for ZFS
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 98b7df0be162581a143bfd4e60ac921b5d0b7394

[Gerrit Updater]

Sebastien Buisson ([e-mail]) uploaded a new patch: https://review.whamcloud.com/41130
Subject: LU-14281 tests: fix sanity-sec test_56
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 1937eb9954778bc49fd097ce3db1120b5e769df8

[Peter Jones]

Sébastien

Could you please advise?

Peter

[James Nunez (Inactive)]

The e2fsprogs version looks correct. In node-provisioning for this test session, we see

Package e2fsprogs-1.45.6.wc3-0.el8.x86_64 is already installed.

[Andreas Dilger]

Is there somewhere in that test which is showing the e2fsprogs version installed? It needs to be the Lustre version if the file has anything other than a 1-stripe file.

Since all we care about is whether the "encrypted" flag is set at all, the "lfs setstripe" should be changed to just use "lfs setstripe -c 1 $DIR/$tfile" and then the count of "encrypted" and "encoded" should be ">= 1".