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

argument parsing in lfs_migrate is broken

Details

    • Bug
    • Resolution: Fixed
    • Major
    • Lustre 2.12.0
    • Lustre 2.11.0
    • None
    • 3
    • 9223372036854775807

    Description

      Since the landing of patch https://review.whamcloud.com/20621 "LU-8235 scripts: pass unrecognized options to lfs migrate" any short options with adjacent arguments, such as -S1M or -E-1 are treated as separate options, like -S -1 -M or -E -1.

      Attachments

        Issue Links

          Activity

            [LU-10801] argument parsing in lfs_migrate is broken
            pjones Peter Jones added a comment -

            Landed for 2.12

            pjones Peter Jones added a comment - Landed for 2.12

            Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/32977/
            Subject: LU-10801 utils: fix lfs_migrate argument parsing
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: f4cf8114ac2330a1a22bb37e3789f1635db15f7e

            gerrit Gerrit Updater added a comment - Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/32977/ Subject: LU-10801 utils: fix lfs_migrate argument parsing Project: fs/lustre-release Branch: master Current Patch Set: Commit: f4cf8114ac2330a1a22bb37e3789f1635db15f7e

            Emoly Liu (emoly@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/32977
            Subject: LU-10801 utils: fix lfs_migrate argument parsing
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: a4c1a6a18013ceafd7fa7d2107eb82e5555069bf

            gerrit Gerrit Updater added a comment - Emoly Liu (emoly@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/32977 Subject: LU-10801 utils: fix lfs_migrate argument parsing Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: a4c1a6a18013ceafd7fa7d2107eb82e5555069bf
            emoly.liu Emoly Liu added a comment -

            Andreas, thanks for your patch. I will look into this issue.

            emoly.liu Emoly Liu added a comment - Andreas, thanks for your patch. I will look into this issue.

            Attached is a prototype patch to fix this issue. It is a bit outdated, and I didn't quite have time to get it working, but I think it is a reasonable approach.

            adilger Andreas Dilger added a comment - Attached is a prototype patch to fix this issue. It is a bit outdated, and I didn't quite have time to get it working, but I think it is a reasonable approach.
            pjones Peter Jones added a comment -

            Emoly

            Is this something that you could help with?

            Peter

            pjones Peter Jones added a comment - Emoly Is this something that you could help with? Peter

            People

              emoly.liu Emoly Liu
              adilger Andreas Dilger
              Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: