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

lfs quota -s START_QID -e END_QID needs documentation

Details

    • Bug
    • Resolution: Not a Bug
    • Minor
    • None
    • None
    • 3
    • 9223372036854775807

    Description

      lfs quota -s START_QID -e END_QID is not documented in the man pages.

      sanity-quota.sh test 49a and 87 should be improved as well.

      Attachments

        Issue Links

          is related to

          Improvement - An improvement or enhancement to an existing feature or task. LU-14535 add option for LFS to get all quota information

          • 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-17702 'lfs quota -u $USER' should print quota for all Lustre mountpoints

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

          Activity

            [LU-19132] lfs quota -s START_QID -e END_QID needs documentation
            gerrit Gerrit Updater added a comment -

            "Frederick Dilger <fdilger@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/60172
            Subject: LU-19132 utils: added error msgs for quota opts
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: e787a7d82d046b8296bb0a3d8fae61a56524acca

            gerrit Gerrit Updater added a comment - "Frederick Dilger <fdilger@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/60172 Subject: LU-19132 utils: added error msgs for quota opts Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: e787a7d82d046b8296bb0a3d8fae61a56524acca
            adilger Andreas Dilger added a comment -

            I think in this case it makes sense to print an error, as much as I don't like it, because these two options do quite different things, and it might make sense for them to both be used together in the future, but if they don't work together now it is best to make that clear.

            adilger Andreas Dilger added a comment - I think in this case it makes sense to print an error, as much as I don't like it, because these two options do quite different things, and it might make sense for them to both be used together in the future, but if they don't work together now it is best to make that clear.
            gerrit Gerrit Updater added a comment -

            "Frederick Dilger <fdilger@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/60149
            Subject: LU-19132 doc: fix 'lfs-quota(1)' man page
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 0be97994d2f9dab37d6e5c9a507d62625deba2ad

            gerrit Gerrit Updater added a comment - "Frederick Dilger <fdilger@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/60149 Subject: LU-19132 doc: fix 'lfs-quota(1)' man page Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 0be97994d2f9dab37d6e5c9a507d62625deba2ad
            fdilger Fred Dilger added a comment - - edited

            for some reason 'lfs quota -t -a -U' is a valid command and will just use the most recent command that is specified instead of giving the user a message like "-t and -a cannot be specified at the same time" adilger not sure what you would want the solution to be here because I know you dislike it when the system becomes yappy

            fdilger Fred Dilger added a comment - - edited for some reason 'lfs quota -t -a -U' is a valid command and will just use the most recent command that is specified instead of giving the user a message like "-t and -a cannot be specified at the same time" adilger not sure what you would want the solution to be here because I know you dislike it when the system becomes yappy
            adilger Andreas Dilger added a comment -

            Turns out this is a big in the patch...0

            adilger Andreas Dilger added a comment - Turns out this is a big in the patch...0

            People

              fdilger Fred Dilger
              fdilger Fred Dilger
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: