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

pool_list by path incorrectly prints path as pool prefix

Details

    • Bug
    • Resolution: Fixed
    • Minor
    • Lustre 2.13.0
    • Lustre 2.10.0, Lustre 2.12.2
    • 3
    • 9223372036854775807

    Description

      The lfs/lctl pool_list command incorrectly re-uses the path as pool prefix when printing pools, e.g. with a fresh llmount:

       

      [root@vm0 ~]# lctl pool_new lustre.foo
      lfs Pool lustre.foo created
      
      [root@vm0 ~]# lctl pool_list /mnt/lustre
      Pools from /mnt/lustre:
      /mnt/lustre.foo
      

      /mnt/lustre.foo is not a valid pool name and scripts looking at this output to use directly with lfs setstripe -p will now fail

      On old 2.7 at least, this would correctly print lustre.foo instead

       
      The fix is very much straightforward, I'll submit something right away.

      Attachments

        Issue Links

          Activity

            [LU-12473] pool_list by path incorrectly prints path as pool prefix
            pjones Peter Jones added a comment -

            Landed for 2.13

            pjones Peter Jones added a comment - Landed for 2.13

            Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/35320/
            Subject: LU-12473 llapi: fix pool_list by path
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 3fa730299876a612695270a6c65e1ab1d3762a1f

            gerrit Gerrit Updater added a comment - Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/35320/ Subject: LU-12473 llapi: fix pool_list by path Project: fs/lustre-release Branch: master Current Patch Set: Commit: 3fa730299876a612695270a6c65e1ab1d3762a1f
            pjones Peter Jones added a comment -

            Thanks Dominique! 

            pjones Peter Jones added a comment - Thanks Dominique! 

            Dominique Martinet (dominique.martinet@cea.fr) uploaded a new patch: https://review.whamcloud.com/35320
            Subject: LU-12473 llapi: fix pool_list by path
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: b29e777b0cd8a74aaaba649e72ac50be41180084

            gerrit Gerrit Updater added a comment - Dominique Martinet (dominique.martinet@cea.fr) uploaded a new patch: https://review.whamcloud.com/35320 Subject: LU-12473 llapi: fix pool_list by path Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: b29e777b0cd8a74aaaba649e72ac50be41180084

            People

              martinetd Dominique Martinet (Inactive)
              cealustre CEA
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: