Details

    • New Feature
    • Resolution: Fixed
    • Minor
    • Lustre 2.17.0
    • None
    • 3
    • 9223372036854775807

    Description

      The command lctl nodemap_info only presents information about which property names exist for a given nodemap. It does not show the values for each property, and the command's usefulness is therefore limited. It would be beneficial if lctl nodemap_info would present the user with in-depth information, including the global nodemap state (active/inactive), property availability and descriptions, all defined nodemaps, and the values for all nodemap properties.

      This task aims to significantly improve the usability for querying nodemap information rather than relying on the lctl get_param-type interface (which is used today). This will further make the user API more consistent with lctl nodemap_* commands since lctl set_param is discouraged since the changes via the latter interface are not persistent and only local to the MGS.

      The proposed API is as follows:
      (extended) lctl nodemap_info [-l/--list] [-n/--name <nodemap_name>] [-p/--property <value>]

      Use the command without any parameters lists all nodemaps and their properties. The previous positional parameters list|all|nodemap_name are retained for compatibility and should be deprecated in the future.

      Attachments

        Issue Links

          Activity

            [LU-18715] Improve lctl nodemap_info
            pjones Peter Jones added a comment -

            Merged for 2.17

            pjones Peter Jones added a comment - Merged for 2.17

            "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/58052/
            Subject: LU-18715 utils: Extend lctl nodemap_info with property values
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: b8e3b427f47876ffa65b4198e096219aaa9798d4

            gerrit Gerrit Updater added a comment - "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/58052/ Subject: LU-18715 utils: Extend lctl nodemap_info with property values Project: fs/lustre-release Branch: master Current Patch Set: Commit: b8e3b427f47876ffa65b4198e096219aaa9798d4

            "Marc Vef <mvef@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/58052
            Subject: LU-18715 utils: Extend lctl nodemap_info with property values
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 0099f21e9c8b3c1bc26e552bbaff951841e821f8

            gerrit Gerrit Updater added a comment - "Marc Vef <mvef@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/58052 Subject: LU-18715 utils: Extend lctl nodemap_info with property values Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 0099f21e9c8b3c1bc26e552bbaff951841e821f8

            People

              mvef Marc Vef
              mvef Marc Vef
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: