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

lctl pool operations cannot be run on separate MGS system

Details

    • 3
    • 9223372036854775807

    Description

      The "lctl pool_new", "lctl pool_add", and related commands cannot properly be run on a system where the the MGS is separate from the MDS. The pool commands must be run on the MGS in order to modify the configuration, but the pool configuration is not instantiated on the MGS so it does not have corresponding /proc/fs/lustre/lov/*/pool/ files that lctl uses to verify whether the pool and OSTs exist or not.

      Rather than using the /proc or /sys file existence to verify the pool commands on the MGS, it would be possible to instead parse the filesystem configuration directly from the YAML filesystem configuration output of "lctl llog_print <fsname>-client". This would also avoid the need for lctl to wait for the pool files to appear in /proc after the command is run, which can often take a few seconds.

      Attachments

        Issue Links

          Activity

            [LU-11743] lctl pool operations cannot be run on separate MGS system

            Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/36314/
            Subject: LU-11743 utils: allow lctl pool_list on separate MGS
            Project: fs/lustre-release
            Branch: b2_12
            Current Patch Set:
            Commit: 2b659e3a5c693943c147f713d832fc82d5a254fe

            gerrit Gerrit Updater added a comment - Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/36314/ Subject: LU-11743 utils: allow lctl pool_list on separate MGS Project: fs/lustre-release Branch: b2_12 Current Patch Set: Commit: 2b659e3a5c693943c147f713d832fc82d5a254fe

            Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/35804/
            Subject: LU-11743 utils: allow lctl pool commands on separate MGS
            Project: fs/lustre-release
            Branch: b2_12
            Current Patch Set:
            Commit: ef1a94ee7fd859ea5c3af6b3028df78e6acea8c7

            gerrit Gerrit Updater added a comment - Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/35804/ Subject: LU-11743 utils: allow lctl pool commands on separate MGS Project: fs/lustre-release Branch: b2_12 Current Patch Set: Commit: ef1a94ee7fd859ea5c3af6b3028df78e6acea8c7

            Minh Diep (mdiep@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/36314
            Subject: LU-11743 utils: allow lctl pool_list on separate MGS
            Project: fs/lustre-release
            Branch: b2_12
            Current Patch Set: 1
            Commit: 1bcc27c39066363afbe52a8d5f3ad3cb074ef533

            gerrit Gerrit Updater added a comment - Minh Diep (mdiep@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/36314 Subject: LU-11743 utils: allow lctl pool_list on separate MGS Project: fs/lustre-release Branch: b2_12 Current Patch Set: 1 Commit: 1bcc27c39066363afbe52a8d5f3ad3cb074ef533
            pjones Peter Jones added a comment -

            Everything landed now

            pjones Peter Jones added a comment - Everything landed now

            Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/35895/
            Subject: LU-11743 utils: allow lctl pool_list on separate MGS
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: d908fe9686bc1e583da7434856d9c06e6cbbc4fd

            gerrit Gerrit Updater added a comment - Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/35895/ Subject: LU-11743 utils: allow lctl pool_list on separate MGS Project: fs/lustre-release Branch: master Current Patch Set: Commit: d908fe9686bc1e583da7434856d9c06e6cbbc4fd

            Still one patch to land.

            adilger Andreas Dilger added a comment - Still one patch to land.

            Emoly Liu (emoly@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/35895
            Subject: LU-11743 utils: allow lctl pool_list on separate MGS
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 1f3157d4df3e11f17e4630b55093296a98c4c3e1

            gerrit Gerrit Updater added a comment - Emoly Liu (emoly@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/35895 Subject: LU-11743 utils: allow lctl pool_list on separate MGS Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 1f3157d4df3e11f17e4630b55093296a98c4c3e1

            Minh Diep (mdiep@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/35804
            Subject: LU-11743 utils: allow lctl pool commands on separate MGS
            Project: fs/lustre-release
            Branch: b2_12
            Current Patch Set: 1
            Commit: a091f4da87a80cce9ff4144270d3aef7eb2437c0

            gerrit Gerrit Updater added a comment - Minh Diep (mdiep@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/35804 Subject: LU-11743 utils: allow lctl pool commands on separate MGS Project: fs/lustre-release Branch: b2_12 Current Patch Set: 1 Commit: a091f4da87a80cce9ff4144270d3aef7eb2437c0
            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/34110/
            Subject: LU-11743 utils: allow lctl pool commands on separate MGS
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 4a003a1f554602265630637080f65d9b4474f822

            gerrit Gerrit Updater added a comment - Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/34110/ Subject: LU-11743 utils: allow lctl pool commands on separate MGS Project: fs/lustre-release Branch: master Current Patch Set: Commit: 4a003a1f554602265630637080f65d9b4474f822

            People

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

              Dates

                Created:
                Updated:
                Resolved: