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

allow "lfs find --ost" to accept an OST range

Details

    • 9223372036854775807

    Description

      It would be convenient to allow specifying a range of OSTs to find objects on, like "lfs find /mnt/testfs -type f --ost 8-15". Currently, one has to specify all of the OSTs in a comma-separated list like "... --ost 8,9,10,11,12,13,14,15", which can be inconvenient for large OST lists. Using the bash range syntax like "{8..15}" doesn't work because this will expand to a space-separated list of indices.

      Attachments

        Issue Links

          is related to

          Bug - A problem which impairs or prevents the functions of the product. LU-5170 lfs usability

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

          Activity

            [LU-16622] allow "lfs find --ost" to accept an OST range
            pjones Peter Jones made changes -
            Fix Version/s New: Lustre 2.17.0 [ 16192 ]
            Resolution New: Fixed [ 1 ]
            Status Original: Open [ 1 ] New: Resolved [ 5 ]
            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/+/55902/
            Subject: LU-16622 utils: 'lfs find --ost' supports index range
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 70d8107c48abe2f82f714ca82e488d953642ce06

            gerrit Gerrit Updater added a comment - "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/55902/ Subject: LU-16622 utils: 'lfs find --ost' supports index range Project: fs/lustre-release Branch: master Current Patch Set: Commit: 70d8107c48abe2f82f714ca82e488d953642ce06
            adilger Andreas Dilger made changes -
            Labels Original: easy lug23dd lug24dd ububntu New: easy lug23dd lug24dd utils
            adilger Andreas Dilger made changes -
            Labels Original: easy lug23dd lug24dd New: easy lug23dd lug24dd ububntu
            gerrit Gerrit Updater added a comment -

            "Feng Lei <flei@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/55902
            Subject: LU-16622 utils: 'lfs find --ost' supports index range
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: ea82ed326f74896c0739f3476477e57ba6804f95

            gerrit Gerrit Updater added a comment - "Feng Lei <flei@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/55902 Subject: LU-16622 utils: 'lfs find --ost' supports index range Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: ea82ed326f74896c0739f3476477e57ba6804f95
            flei Feng Lei made changes -
            Assignee Original: Core Lustre Triage [ core-lustre-triage ] New: Feng Lei [ flei ]
            adilger Andreas Dilger made changes -
            Assignee Original: Amit [ ahkumarsmu ] New: Core Lustre Triage [ core-lustre-triage ]
            adilger Andreas Dilger made changes -
            Labels Original: easy lad23dd lug23dd lug24dd New: easy lug23dd lug24dd
            adilger Andreas Dilger added a comment -

            Hi ahkumarsmu, did you ever make any progress developing a patch for this enhancement? If yes, even if you didn't get it quite working, it would be useful to push it to Gerrit for review, either with "Test-Parameters: fortestonly" if it is substantially working, or "Test-Parameters: ignore" if it is not even at the stage where it can build. Possibly I could help you finish off the remaining issues, or someone else could take this over at the LUG'24 Hackathon. Please let me know either way.

            adilger Andreas Dilger added a comment - Hi ahkumarsmu , did you ever make any progress developing a patch for this enhancement? If yes, even if you didn't get it quite working, it would be useful to push it to Gerrit for review, either with " Test-Parameters: fortestonly " if it is substantially working, or " Test-Parameters: ignore " if it is not even at the stage where it can build. Possibly I could help you finish off the remaining issues, or someone else could take this over at the LUG'24 Hackathon. Please let me know either way.

            People

              flei Feng Lei
              adilger Andreas Dilger
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: