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

'lfs mkdir -i -1' may not initialize target MDTs correctly

Details

    • Bug
    • Resolution: Fixed
    • Minor
    • Lustre 2.14.0, Lustre 2.12.6
    • Lustre 2.12.0
    • None
    • 3
    • 9223372036854775807

    Description

      When 'lfs mkdir -i -1' is used to create directory with balanced MDT usage, it may not initialize target MDTs correctly, which will cause verify_dir_param() report "stripe offset x is not in target list".

      From 2.13 MDT object QoS allocation is supported, and these codes are removed, so this needs to be fixed in 2.12 only.

      Attachments

        Activity

          [LU-13709] 'lfs mkdir -i -1' may not initialize target MDTs correctly
          gerrit Gerrit Updater added a comment -

          Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/40801/
          Subject: LU-13709 tests: test lfs mkdir -c without -i
          Project: fs/lustre-release
          Branch: b2_12
          Current Patch Set:
          Commit: 404bf138a326b6039ea1f2b46ef0a5b15e8680dd

          gerrit Gerrit Updater added a comment - Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/40801/ Subject: LU-13709 tests: test lfs mkdir -c without -i Project: fs/lustre-release Branch: b2_12 Current Patch Set: Commit: 404bf138a326b6039ea1f2b46ef0a5b15e8680dd
          ofaaland Olaf Faaland added a comment -

          Peter,
          Yes, I think we should.

          ofaaland Olaf Faaland added a comment - Peter, Yes, I think we should.
          gerrit Gerrit Updater added a comment -

          Olaf Faaland-LLNL (faaland1@llnl.gov) uploaded a new patch: https://review.whamcloud.com/40801
          Subject: LU-13709 tests: test lfs mkdir -c without -i
          Project: fs/lustre-release
          Branch: b2_12
          Current Patch Set: 1
          Commit: c018178faf605437e97da958fdeef1cc7dd322f8

          gerrit Gerrit Updater added a comment - Olaf Faaland-LLNL (faaland1@llnl.gov) uploaded a new patch: https://review.whamcloud.com/40801 Subject: LU-13709 tests: test lfs mkdir -c without -i Project: fs/lustre-release Branch: b2_12 Current Patch Set: 1 Commit: c018178faf605437e97da958fdeef1cc7dd322f8
          pjones Peter Jones added a comment -

          OK. So now everything Is landed to master but we should consider back porting the test to b2_12?

          pjones Peter Jones added a comment - OK. So now everything Is landed to master but we should consider back porting the test to b2_12?
          gerrit Gerrit Updater added a comment -

          Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/39457/
          Subject: LU-13709 tests: test lfs mkdir -c without -i
          Project: fs/lustre-release
          Branch: master
          Current Patch Set:
          Commit: 2c89cc6c25549cb6748c7c9f5a209c7e38387eb4

          gerrit Gerrit Updater added a comment - Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/39457/ Subject: LU-13709 tests: test lfs mkdir -c without -i Project: fs/lustre-release Branch: master Current Patch Set: Commit: 2c89cc6c25549cb6748c7c9f5a209c7e38387eb4
          laisiyao Lai Siyao added a comment -

          It's always good to add new test, though sanity 413a and 413b verified MDT object QoS allocation, and 413a uses "lfs mkdir" without "-i" to verify directories are created by QoS.

          laisiyao Lai Siyao added a comment - It's always good to add new test, though sanity 413a and 413b verified MDT object QoS allocation, and 413a uses "lfs mkdir" without "-i" to verify directories are created by QoS.
          ofaaland Olaf Faaland added a comment -

          Lai, master has more instances of "mkdir" without "-i" argument in the test suite than b2_12 does, but even in master I don't see a test that sanity checks the stripe placement; so I think https://review.whamcloud.com/39457 is still useful.  If you agree, can you review the patch?  And if you think it's redundant, let me know and I'll abandon it?  Thanks.

          ofaaland Olaf Faaland added a comment - Lai, master has more instances of "mkdir" without "-i" argument in the test suite than b2_12 does, but even in master I don't see a test that sanity checks the stripe placement; so I think https://review.whamcloud.com/39457  is still useful.  If you agree, can you review the patch?  And if you think it's redundant, let me know and I'll abandon it?  Thanks.
          ofaaland Olaf Faaland added a comment -

          Needs a test

          ofaaland Olaf Faaland added a comment - Needs a test
          gerrit Gerrit Updater added a comment -

          Olaf Faaland-LLNL (faaland1@llnl.gov) uploaded a new patch: https://review.whamcloud.com/39457
          Subject: LU-13709 tests: test lfs setdirstripe -c without -i
          Project: fs/lustre-release
          Branch: master
          Current Patch Set: 1
          Commit: d89a9b49298343cb20c7ee287789130d318a3e2d

          gerrit Gerrit Updater added a comment - Olaf Faaland-LLNL (faaland1@llnl.gov) uploaded a new patch: https://review.whamcloud.com/39457 Subject: LU-13709 tests: test lfs setdirstripe -c without -i Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: d89a9b49298343cb20c7ee287789130d318a3e2d
          pjones Peter Jones added a comment -

          Landed for 2.12.6

          pjones Peter Jones added a comment - Landed for 2.12.6

          People

            laisiyao Lai Siyao
            laisiyao Lai Siyao
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: