[LU-13709] 'lfs mkdir -i -1' may not initialize target MDTs correctly Created: 24/Jun/20 Updated: 04/Mar/21 Resolved: 19/Nov/20 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | Lustre 2.12.0 |
| Fix Version/s: | Lustre 2.14.0, Lustre 2.12.6 |
| Type: | Bug | Priority: | Minor |
| Reporter: | Lai Siyao | Assignee: | Lai Siyao |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Issue Links: |
|
||||||||
| Severity: | 3 | ||||||||
| Rank (Obsolete): | 9223372036854775807 | ||||||||
| 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 |
| Comment by Gerrit Updater [ 24/Jun/20 ] |
|
Lai Siyao (lai.siyao@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/39165 |
| Comment by Gerrit Updater [ 11/Jul/20 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/39165/ |
| Comment by Peter Jones [ 11/Jul/20 ] |
|
Landed for 2.12.6 |
| Comment by Gerrit Updater [ 20/Jul/20 ] |
|
Olaf Faaland-LLNL (faaland1@llnl.gov) uploaded a new patch: https://review.whamcloud.com/39457 |
| Comment by Olaf Faaland [ 20/Jul/20 ] |
|
Needs a test |
| Comment by Olaf Faaland [ 12/Oct/20 ] |
|
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. |
| Comment by Lai Siyao [ 12/Oct/20 ] |
|
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. |
| Comment by Gerrit Updater [ 19/Nov/20 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/39457/ |
| Comment by Peter Jones [ 19/Nov/20 ] |
|
OK. So now everything Is landed to master but we should consider back porting the test to b2_12? |
| Comment by Gerrit Updater [ 30/Nov/20 ] |
|
Olaf Faaland-LLNL (faaland1@llnl.gov) uploaded a new patch: https://review.whamcloud.com/40801 |
| Comment by Olaf Faaland [ 30/Nov/20 ] |
|
Peter, |
| Comment by Gerrit Updater [ 04/Mar/21 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/40801/ |