[LU-18193] sanity/300l fails frequently on a local setup

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.16.0
• Affects Version/s: Lustre 2.16.0
• Severity: 3
• Assignee: Lai Siyao
• Reporter: Alex Zhuravlev

Description

# OSTCOUNT=2 MDSCOUNT=2 ONLY=300l bash sanity.sh
....
== sanity test 300l: non-root user to create dir under striped dir with stale layout ========================================================== 14:46:41 (1718894801)
striped dir -i0 -c2 -H fnv_1a_64 /mnt/lustre/d300l.sanity/striped_dir
fail_loc=0x80000158
running as uid/gid/euid/egid 500/500/500/500, groups:
 [mkdir] [/mnt/lustre/d300l.sanity/striped_dir/test_dir]
mkdir: cannot create directory '/mnt/lustre/d300l.sanity/striped_dir/test_dir': Operation not permitted
 sanity test_300l: @@@@@@ FAIL: create dir fails 
  Trace dump:
  = ./../tests/test-framework.sh:7090:error()
  = sanity.sh:26901:test_300l()
  = ./../tests/test-framework.sh:7435:run_one()
  = ./../tests/test-framework.sh:7498:run_one_logged()
  = ./../tests/test-framework.sh:7321:run_test()
  = sanity.sh:26907:main()
Dumping lctl log to /tmp/ltest-logs/sanity.test_300l.*.1718894801.log
Dumping logs only on local client.
FAIL 300l (0s)

I bisected this problem to commit: LU-12998 lod: statfs upon nocreate check

Issue Links

• is related to: LU-6341 Do not check security when accessing default stripeEA
• is related to: LU-12998 DNE3: tunable to disable directory creation on MDT

Comments

[Peter Jones]

Merged for 2.16

[Gerrit Updater]

"Oleg Drokin <[e-mail]>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/56252/
Subject: LU-18193 test: fix sanity 300l
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: bdea0d8dcfd8f1726dbe5713606fabace7ecef5d

[Gerrit Updater]

"Lai Siyao <[e-mail]>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/56252
Subject: LU-18193 test: fix sanity 300l
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 9dd077c285d0c8dc421bd96aa89ee6ff66235646

[Lai Siyao]

This looks to be test script issue: test_300l run mkdir with 'RUNAS', but it's a remote directory, which requires enable_remote_dir_gid=-1. But it doesn't make sense to RUNAS here, mkdir as ROOT is okay and won't cause confusion.

[Andreas Dilger]

Do you think this is due to the extra obd_statfs() RPCs being sent, or because of the logic change in lid_declare_create()? I'm hoping that the statfs calls are rate-limited so that the clients are not spamming the MDS when the MDT is deactivated...