[LU-14058] Create tests for large number of MDTs Created: 21/Oct/20 Updated: 20/Sep/22 |
|
| Status: | Reopened |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | Lustre 2.14.0 |
| Fix Version/s: | Lustre 2.16.0 |
| Type: | Improvement | Priority: | Minor |
| Reporter: | James Nunez (Inactive) | Assignee: | Sarah Liu |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | failing_tests, tests | ||
| Issue Links: |
|
||||||||
| Rank (Obsolete): | 9223372036854775807 | ||||||||
| Description |
|
Patch https://review.whamcloud.com/37759 " |
| Comments |
| Comment by Olaf Faaland [ 05/Nov/20 ] |
|
According to the commit message, the patch fixes the case where MDTs > 64. So It would be really good to have tests for 16 or 32 MDTs, but your mount test should have >64 MDTs. |
| Comment by Oleg Drokin [ 06/Nov/20 ] |
|
this probably makes sense to do as part of conf-sanity and since out VMs have relatively little memory we might not be able to run 32-64 MDTs on a single node so perhaps the test would need to spread out MDTs (off loopback mounted storage in tmp perhaps) across multiple nodes |
| Comment by Alex Kulyavtsev [ 17/Nov/20 ] |
|
confirm, you need > 64 MDT to be sure. You can trip the max in /proc/misc with fewer MDTs as there are ~18 devices created by linux. This number vary from system to system. Lustre creates two devices (obd,lnet) common for all lustre mounts in /proc/misc besides changelog devices changelog-<fsname>-MDT00NN (one per MDT). Probably Nmdt > 64-(18+2), that is Nmdt > 44 will be enough but you may want to use >64 to be sure. Note,
|
| Comment by Andreas Dilger [ 11/Feb/21 ] |
|
Andreas Dilger (adilger@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/41485 |
| Comment by Gerrit Updater [ 08/Mar/21 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/41485/ |
| Comment by Peter Jones [ 08/Mar/21 ] |
|
Landed for 2.15 |
| Comment by Andreas Dilger [ 09/Mar/21 ] |
|
There is still an unlanded patch for this ticket (it was originally pushed under Andreas Dilger (adilger@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/38058 |
| Comment by Gerrit Updater [ 21/Jan/22 ] |
|
"Wei Liu <sarah@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/46268 |