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

add options for 'lfs quota' to replace getquota() argument parsing

Details

    • 3
    • 9223372036854775807

    Description

      sanity-quota.sh is using getquota() to parse individual arguments from the table. The table used to have varying formats before LU-17702 where the Filesystem would be put on a new line if it exceeded the formatted space.

      The ability to obtain the value of individual arguments like 'lfs getstripe' would not only replace the unnecessary parsing with getquota() but would also be a good tool for users.

      Attachments

        Issue Links

          has to be done before

          Task - A task that needs to be done. LU-18441 replace getquota() in sanity-quota.sh

          • 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-18220 "lfs quota" output is difficult to read

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

          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-18079] add options for 'lfs quota' to replace getquota() argument parsing
            pjones Peter Jones added a comment -

            Merged for 2.17

            pjones Peter Jones added a comment - Merged for 2.17
            gerrit Gerrit Updater added a comment -

            "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/56231/
            Subject: LU-18079 tests: updating getquota() in sanity-quota
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: f37e9527ef80ffcadcc849980d56ec83b039679e

            gerrit Gerrit Updater added a comment - "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/56231/ Subject: LU-18079 tests: updating getquota() in sanity-quota Project: fs/lustre-release Branch: master Current Patch Set: Commit: f37e9527ef80ffcadcc849980d56ec83b039679e
            gerrit Gerrit Updater added a comment -

            "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/55894/
            Subject: LU-18079 utils: argument parse opts for lfs quota
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 7c02893e12e207ac543a15da15bdf5fa9597df9b

            gerrit Gerrit Updater added a comment - "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/55894/ Subject: LU-18079 utils: argument parse opts for lfs quota Project: fs/lustre-release Branch: master Current Patch Set: Commit: 7c02893e12e207ac543a15da15bdf5fa9597df9b
            gerrit Gerrit Updater added a comment -

            "Frederick Dilger <fdilger@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/56232
            Subject: LU-18079 tests: updating sanity-quota code style
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 2bcdbe2033f08b536c2599ca1be9b2e3a222d553

            gerrit Gerrit Updater added a comment - "Frederick Dilger <fdilger@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/56232 Subject: LU-18079 tests: updating sanity-quota code style Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 2bcdbe2033f08b536c2599ca1be9b2e3a222d553
            gerrit Gerrit Updater added a comment -

            "Frederick Dilger <fdilger@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/56231
            Subject: LU-18079 tests: replacing getquota() in sanity-quota
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 4981ce8281476197582f5cddc5cb4dc1e26548c2

            gerrit Gerrit Updater added a comment - "Frederick Dilger <fdilger@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/56231 Subject: LU-18079 tests: replacing getquota() in sanity-quota Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 4981ce8281476197582f5cddc5cb4dc1e26548c2
            gerrit Gerrit Updater added a comment -

            "Frederick Dilger <fdilger@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/55894
            Subject: LU-18079 utils: argument parse opts for lfs quota
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 20c0fd6fe7afe8f3c61c7d291c66ad8188f3ae66

            gerrit Gerrit Updater added a comment - "Frederick Dilger <fdilger@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/55894 Subject: LU-18079 utils: argument parse opts for lfs quota Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 20c0fd6fe7afe8f3c61c7d291c66ad8188f3ae66

            People

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

              Dates

                Created:
                Updated:
                Resolved: