[LU-13709] 'lfs mkdir -i -1' may not initialize target MDTs correctly

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.14.0, Lustre 2.12.6
• Affects Version/s: Lustre 2.12.0
• Severity: 3
• Assignee: Lai Siyao
• Reporter: Lai Siyao

Description

When 'lfs mkdir -i -1' is used to create directory with balanced MDT usage, it may not initialize target MDTs correctly, which will cause verify_dir_param() report "stripe offset x is not in target list".

From 2.13 MDT object QoS allocation is supported, and these codes are removed, so this needs to be fixed in 2.12 only.

Comments

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/40801/
Subject: LU-13709 tests: test lfs mkdir -c without -i
Project: fs/lustre-release
Branch: b2_12
Current Patch Set:
Commit: 404bf138a326b6039ea1f2b46ef0a5b15e8680dd

[Olaf Faaland]

Peter,
Yes, I think we should.

[Gerrit Updater]

Olaf Faaland-LLNL ([e-mail]) uploaded a new patch: https://review.whamcloud.com/40801
Subject: LU-13709 tests: test lfs mkdir -c without -i
Project: fs/lustre-release
Branch: b2_12
Current Patch Set: 1
Commit: c018178faf605437e97da958fdeef1cc7dd322f8

[Peter Jones]

OK. So now everything Is landed to master but we should consider back porting the test to b2_12?

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/39457/
Subject: LU-13709 tests: test lfs mkdir -c without -i
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 2c89cc6c25549cb6748c7c9f5a209c7e38387eb4

[Lai Siyao]

It's always good to add new test, though sanity 413a and 413b verified MDT object QoS allocation, and 413a uses "lfs mkdir" without "-i" to verify directories are created by QoS.

[Olaf Faaland]

Lai, master has more instances of "mkdir" without "-i" argument in the test suite than b2_12 does, but even in master I don't see a test that sanity checks the stripe placement; so I think https://review.whamcloud.com/39457 is still useful.  If you agree, can you review the patch?  And if you think it's redundant, let me know and I'll abandon it?  Thanks.