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

"lfs migrate" and "lfs_migrate" to accept list of FIDs

Details

    • Improvement
    • Resolution: Unresolved
    • Minor
    • None
    • None

    Description

      The "lfs migrate" and "lfs_migrate" commands should accept a list of FIDs instead of filenames on the command-line with the "--fids" option. This will allow them to use the "llapi_open_by_fid[_at]()" interface internally to open the files instead of doing pathname resolution, and improve integration into pipelines that produce FIDs instead of filenames (e.g. Changelogs) without the need to produce an intermediate pathname.

      Attachments

        Issue Links

          Activity

            [LU-15425] "lfs migrate" and "lfs_migrate" to accept list of FIDs

            "Feng Lei <flei@ddn.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/58260
            Subject: LU-15425 utils: lfs_migrate script can accept FID
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: af53854149040739c1ee2361567fa113e9a757a0

            gerrit Gerrit Updater added a comment - "Feng Lei <flei@ddn.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/58260 Subject: LU-15425 utils: lfs_migrate script can accept FID Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: af53854149040739c1ee2361567fa113e9a757a0

            "Feng Lei <flei@ddn.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/58224
            Subject: LU-15425 utils: lfs migrate command can accept FIDs
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: cefc19eb619835e6605678c5eb3bfd999ed2b8ca

            gerrit Gerrit Updater added a comment - "Feng Lei <flei@ddn.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/58224 Subject: LU-15425 utils: lfs migrate command can accept FIDs Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: cefc19eb619835e6605678c5eb3bfd999ed2b8ca

            Better to use llapi_open_by_fid_at() so that it works even in a subdirectory mount where the .lustre/fid directory does not exist.

            adilger Andreas Dilger added a comment - Better to use llapi_open_by_fid_at() so that it works even in a subdirectory mount where the .lustre/fid directory does not exist.
            flei Feng Lei added a comment - - edited

            adilger Is it OK to construct filenames <lustre_mount_point>/.lustre/fids/<FID> based on the given FIDs, then execute the command as before.

            flei Feng Lei added a comment - - edited adilger Is it OK to construct filenames <lustre_mount_point>/.lustre/fids/<FID> based on the given FIDs, then execute the command as before.
            flei Feng Lei added a comment - - edited

            I'm going to add --fid option so that all the filenames are explained as fids. The fids are listed on command line or by --files-from=LIST_FILE  as if they are still filenames.

            flei Feng Lei added a comment - - edited I'm going to add --fid option so that all the filenames are explained as fids. The fids are listed on command line or by --files-from=LIST_FILE   as if they are still filenames.

            People

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

              Dates

                Created:
                Updated: