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

inconsistencies between "lfs find", "lfs getstripe", "lfs setstripe", "lfs setdirstripe"

    XMLWordPrintable

Details

    • 3
    • 9223372036854775807

    Description

      There are a number of inconsistencies in the command-line options between lfs find, lfs getstripe, and lfs setstripe/lfs migrate which are confusing and should be fixed, in a compatible manner, but deprecating the old options.

      • lfs find and lfs migrate use -m to for mdt_index, but lfs getstripe uses -M to indicate mdt_index. getstripe should use -m and deprecate -M, since find uses -M for mtime.
      • lfs setstripe uses --ost-list, but lfs find and lfs getstripe uses for --ost, but they are all conceptually the same thing. setstripe should just use --ost for this, and deprecate --ost-list as this option has only been around for a short time.
      • lfs find uses --mdt to search for a specific mdt_index, but lfs getstripe and lfs migrate use --mdt-index. find should add --mdt-index (and --mdt_index for ease of use, but not document it), and getstripe and migrate should add --mdt.

      I'm not sure of a clear winner between --mdt and --mdt-index, since I don't think --mdt is confusing and it parallels --ost for OST selection, and is short and convenient. Having both is not harmful I think.

      The lfs setdirstripe and lfs getdirstripe options should also be expanded to be consistent with those in lfs find and lfs getstripe, namely, add --mdt-count|-c, --mdt-index|-m, --mdt-hash as aliases for the existing --count|-c and --index|-i and --hash-type|-t options. The long -count and -index options should be deprecated with LUSTRE_VERSION_CODE checks as they were for lfs_setstripe().

      Attachments

        Issue Links

          is duplicated by

          Improvement - An improvement or enhancement to an existing feature or task. LU-9378 extract lfs-getstripe.1 from lfs.1

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

          Bug - A problem which impairs or prevents the functions of the product. LU-7503 "-v" is missing for "lfs migrate -m"

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

          Bug - A problem which impairs or prevents the functions of the product. LU-10856 running sanity now prints "warning: '-M' deprecated, use '--mdt-index' or '-m' instead" billions of times

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

          Bug - A problem which impairs or prevents the functions of the product. LU-11891 make --mdt-index usage consistent

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

          Improvement - An improvement or enhancement to an existing feature or task. LU-15276 DNE3: "lfs find" able to check default directory and file layout

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

          Improvement - An improvement or enhancement to an existing feature or task. LU-7502 add lfs find --mdt-count and --mdt-hash options

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

          Activity

            People

              ys Yang Sheng
              adilger Andreas Dilger
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: