[LU-13154] Interop: sanity-quota test 66 fails with 'nonroot user should fail to set projid'

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.14.0, Lustre 2.12.5
• Affects Version/s: Lustre 2.14.0, Lustre 2.12.4
• Labels: interop
• Environment: Interop testing with servers older than clients (see the description)
• Severity: 3
• Assignee: Wang Shilong (Inactive)
• Reporter: James Nunez (Inactive)

Description

sanity-quota test_66 fails with 'nonroot user should fail to set projid'. Looking at the output in the client test_log, we see that we are unable to set the enable_chprojid_gid parameter:

CMD: trevis-44vm9 /usr/sbin/lctl get_param -n mdt.*.enable_chprojid_gid
trevis-44vm9: error: get_param: param_path 'mdt/*/enable_chprojid_gid': No such file or directory
CMD: trevis-44vm9 /usr/sbin/lctl set_param mdt.*.enable_chprojid_gid=0
trevis-44vm9: error: set_param: param_path 'mdt/*/enable_chprojid_gid': No such file or directory
lfs project -sp 1000 /mnt/lustre/d66.sanity-quota/foo
running as uid/gid/euid/egid 60000/60000/60000/60000, groups:
 [mkdir] [/mnt/lustre/d66.sanity-quota/foo/foo]
running as uid/gid/euid/egid 60000/60000/60000/60000, groups:
 [lfs] [project] [-p] [0] [/mnt/lustre/d66.sanity-quota/foo/foo]
 sanity-quota test_66: @@@@@@ FAIL: nonroot user should fail to set projid 

Looking at the patch for LU-12826, we can see that patch https://review.whamcloud.com/36544/ added this test and implemented the enable_chprojid_gid parameter. The test does not have a version check, but needs one. This patch landed to lustre-master with version 2.13.50.114 and for b2_12 with version 2.12.3.89.

Logs for failures are at
2.10.8 servers/2.12.3.93 clients: https://testing.whamcloud.com/test_sets/f2d41664-3461-11ea-bb75-52540065bddc
2.12.3 servers/2.13.50.206 clients: https://testing.whamcloud.com/test_sets/09bbb624-34a4-11ea-b0f4-52540065bddc
2.11.0 servers/2.12.3.93 clients: https://testing.whamcloud.com/test_sets/efa5ffda-346a-11ea-b1e8-52540065bddc

Comments

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/37350/
Subject: LU-13154 test: skip sanity-quota 66 if MDS version < 2.12.4
Project: fs/lustre-release
Branch: b2_12
Current Patch Set:
Commit: 3c3b95cca6d44be1ca0b515f589f2e95c8b26ea4

[Gerrit Updater]

Minh Diep ([e-mail]) uploaded a new patch: https://review.whamcloud.com/37350
Subject: LU-13154 test: skip sanity-quota 66 if MDS version < 2.12.4
Project: fs/lustre-release
Branch: b2_12
Current Patch Set: 1
Commit: 17d0838250741a36bcd5bd57105fa56eba109b84

[Peter Jones]

Landed for 2.14

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/37276/
Subject: LU-13154 test: skip sanity-quota 66 if MDS version < 2.12.4
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: f99fa029fd904ac13f33ab82de37fd07a69aea84

[Gerrit Updater]

Wang Shilong ([e-mail]) uploaded a new patch: https://review.whamcloud.com/37276
Subject: LU-13154 test: skip sanity-quota 66 if MDS version < 2.12.4
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 5f4b8e290e8054297fd085d6522aa35b0c062e20

[Peter Jones]

Shilong

Could you please add the missing version checks for the new test?

Thanks

Peter

[Andreas Dilger]

Since it looks like the patch was backported to b2_12, then both b2_12 and master should get a version check to skip testing for MDS < 2.12.4.