[LU-5868] lctl does not support listing nodemaps Created: 05/Nov/14 Updated: 10/Jun/15 Resolved: 10/Jun/15 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | Lustre 2.7.0 |
| Fix Version/s: | Lustre 2.8.0 |
| Type: | Bug | Priority: | Minor |
| Reporter: | John Hammond | Assignee: | John Hammond |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | nodemap, patch | ||
| Severity: | 3 |
| Rank (Obsolete): | 16417 |
| Description |
|
lctl provides the several nodemap related commands but none will list the currently added nodemaps. |
| Comments |
| Comment by Andreas Dilger [ 05/Nov/14 ] |
|
Kit, could you take a look at this? |
| Comment by Kit Westneat [ 05/Nov/14 ] |
|
Sure, it doesn't look like I can assign this task to myself, but I'll work on adding a command. At the moment, there is the hack of looking in /proc/fs/lustre/nodemap to see what directories exist, but I agree it should have a real command. |
| Comment by Andreas Dilger [ 06/Nov/14 ] |
|
Would it be enough to map this command internally to essentially be lctl list_param nodemap.* or similar? No point in making it too complex if there isn't a requirement for it. |
| Comment by Andreas Dilger [ 11/Dec/14 ] |
|
Kit, any progress on this? |
| Comment by Kit Westneat [ 12/Dec/14 ] |
|
Hi Andreas, Nothing substantial, I'll work on it for Tuesday. I started looking into using list_param. It looks like there is no way to ask list_param to only show directories, so it adds all the add_idmap, etc proc files. Would it be worth adding a flag to list_param to only list directories? Or should I just send an ioctl to get the nodemap list? |
| Comment by Gerrit Updater [ 12/Dec/14 ] |
|
Kit Westneat (kit.westneat@gmail.com) uploaded a new patch: http://review.whamcloud.com/13044 |
| Comment by Kit Westneat [ 12/Dec/14 ] |
|
I wrote a small patch that currently doesn't list, but it does display other information. It's basically just a wrapper for get_param: |
| Comment by Peter Jones [ 12/Dec/14 ] |
|
Thanks Kit! John Could you please assess this patch? Thanks Peter |
| Comment by Gerrit Updater [ 09/May/15 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/13044/ |
| Comment by Peter Jones [ 10/Jun/15 ] |
|
Landed for 2.8 |