Uploaded image for project: 'Lustre'
  1. Lustre
  2. LU-7299

allow mkfs.lustre --index to be specified in hex

Details

    • Improvement
    • Resolution: Fixed
    • Minor
    • Lustre 2.8.0
    • None
    • 9223372036854775807

    Description

      The mkfs.lustre --index argument should be able to handle hex index values as well as decimal values, especially since the OSTxxxx identifiers are printed in hexadecimal as well.

      The current code uses atol() currently, but should be changed to strtoul(,,0) so that it can also accept index values of the form 0xnnnn and handle better error checking.

      Attachments

        Issue Links

          is related to

          Improvement - An improvement or enhancement to an existing feature or task. LU-2475 Improve the documentation of the "--index" mkfs.lustre option

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Resolved

          Activity

            [LU-7299] allow mkfs.lustre --index to be specified in hex
            jgmitter Joseph Gmitter (Inactive) added a comment -

            Landed for 2.8

            jgmitter Joseph Gmitter (Inactive) added a comment - Landed for 2.8
            gerrit Gerrit Updater added a comment -

            Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/16831/
            Subject: LU-7299 utils: allow mkfs.lustre --index to specify in hex/dec
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 069a25b61ecdfc7b01edfa302ff49b1a6897928a

            gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/16831/ Subject: LU-7299 utils: allow mkfs.lustre --index to specify in hex/dec Project: fs/lustre-release Branch: master Current Patch Set: Commit: 069a25b61ecdfc7b01edfa302ff49b1a6897928a
            adilger Andreas Dilger added a comment -

            Thomas, thanks for the patch.

            adilger Andreas Dilger added a comment - Thomas, thanks for the patch.
            gerrit Gerrit Updater added a comment -

            Thomas Stibor (t.stibor@gsi.de) uploaded a new patch: http://review.whamcloud.com/16831
            Subject: LU-7299 utils: allow mkfs.lustre --index to specify in hex/dec
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: bb233adaa34689930cf0038877589a9a6c588db0

            gerrit Gerrit Updater added a comment - Thomas Stibor (t.stibor@gsi.de) uploaded a new patch: http://review.whamcloud.com/16831 Subject: LU-7299 utils: allow mkfs.lustre --index to specify in hex/dec Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: bb233adaa34689930cf0038877589a9a6c588db0
            adilger Andreas Dilger added a comment -

            This change should also be documented in the mkfs.lustre(8) man page.

            adilger Andreas Dilger added a comment - This change should also be documented in the mkfs.lustre(8) man page.

            People

              wc-triage WC Triage
              adilger Andreas Dilger
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: