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

tests: llmount.sh does not use fs-specific paths for the ost and mdt it mounts

Details

    • Bug
    • Resolution: Fixed
    • Minor
    • Lustre 2.9.0
    • None
    • 3
    • 9223372036854775807

    Description

      When using llmount.sh with its default configuration, the paths to the ost(s) and mdt(s) looks like this:
      /mnt/<component><id> (for example "/mnt/ost1")

      Because those paths are not specific to the filesystem they are related to, when mounting two different filesystems with llmount.sh, the first one's ost(s) and mdt(s) are replaced by the second's.

      Attachments

        Issue Links

          Activity

            [LU-8132] tests: llmount.sh does not use fs-specific paths for the ost and mdt it mounts

            Patch landed to master (pre-2.9.0). Work related to this ticket is tracked under LU-8140.

            jamesanunez James Nunez (Inactive) added a comment - Patch landed to master (pre-2.9.0). Work related to this ticket is tracked under LU-8140 .

            Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/20188/
            Subject: LU-8132 tests: do not truncate fsname from paths to ost/mdt
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 710f5ea458ee0db188bb7303ec18feb99e1ce1e0

            gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/20188/ Subject: LU-8132 tests: do not truncate fsname from paths to ost/mdt Project: fs/lustre-release Branch: master Current Patch Set: Commit: 710f5ea458ee0db188bb7303ec18feb99e1ce1e0

            Ok, thanks. Here is the related ticket: LU-8140.

            bougetq Quentin Bouget (Inactive) added a comment - Ok, thanks. Here is the related ticket: LU-8140 .

            Quentin - Thank you for the patch. You may certainly add functionality to the existing patch, but I prefer patches that do a single thing since this is easier for reviewers and may help with the landing process. My request would be to upload a second patch with your proposed change of introducing NOFORMAT_MGS and NOSETUP_MGS.

            jamesanunez James Nunez (Inactive) added a comment - Quentin - Thank you for the patch. You may certainly add functionality to the existing patch, but I prefer patches that do a single thing since this is easier for reviewers and may help with the landing process. My request would be to upload a second patch with your proposed change of introducing NOFORMAT_MGS and NOSETUP_MGS.

            Actually since the ultimate goal is to be able to mount two different filesystems in local mode with llmount.sh, would it be ok for this patch to introduce some other modifications?

            For that, the changes in the current fix are not enough. I would need to add an option "NOFORMAT_MGS" and "NOSETUP_MGS" (similar to the NOFORMAT and NOSETUP variables used in llmount.sh) to protect the mgs created by the first llmount.sh execution from the second one.

            bougetq Quentin Bouget (Inactive) added a comment - Actually since the ultimate goal is to be able to mount two different filesystems in local mode with llmount.sh, would it be ok for this patch to introduce some other modifications? For that, the changes in the current fix are not enough. I would need to add an option "NOFORMAT_MGS" and "NOSETUP_MGS" (similar to the NOFORMAT and NOSETUP variables used in llmount.sh) to protect the mgs created by the first llmount.sh execution from the second one.

            Quentin Bouget (quentin.bouget.ocre@cea.fr) uploaded a new patch: http://review.whamcloud.com/20188
            Subject: LU-8132 tests: do not truncate fsname from paths to ost/mdt
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 9a744f95f19620b0adb8276c00165dd784cee084

            gerrit Gerrit Updater added a comment - Quentin Bouget (quentin.bouget.ocre@cea.fr) uploaded a new patch: http://review.whamcloud.com/20188 Subject: LU-8132 tests: do not truncate fsname from paths to ost/mdt Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 9a744f95f19620b0adb8276c00165dd784cee084
            pjones Peter Jones added a comment -

            Do you intend to push a patch to fix this?

            pjones Peter Jones added a comment - Do you intend to push a patch to fix this?

            This could be corrected by changing the paths to something like:
            /mnt/<fsname>-<component><id>

            bougetq Quentin Bouget (Inactive) added a comment - This could be corrected by changing the paths to something like: /mnt/<fsname>-<component><id>

            People

              jamesanunez James Nunez (Inactive)
              cealustre CEA
              Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: