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

DNE3: mkdir() automatically create remote directory on MDS which has more space

Details

    • Improvement
    • Resolution: Fixed
    • Major
    • Lustre 2.15.0
    • Lustre 2.13.0
    • 3
    • 9223372036854775807

    Description

      Since the patches from LU-11213 landed for 2.13.0, I thought "lfs setdirstripe -i -1 /mnt/lustre" on the root or other existing directory would allow creation of remote directories on other MDTs using plain "mkdir" commands. This is different than the case of "lfs setdirstripe -i -1 -c N /mnt/lustre" selecting stripes on less-full MDTs that was landed via patch https://review.whamcloud.com/35825 "LU-12624 lod: alloc dir stripes by QoS", but this patch also removed the "space" hash, so I thought that regular mkdir of a directory could be allowed to balance across MDTs?

      However, I can't seem to get this to work. On current master (2.13.52-259, just before 2.13.53) I'm not able to use "lfs setdirstripe -i -1 /path/to/dir" on an existing directory. It seems to select the less-full MDT if I explicitly run "lfs mkdir -i -1" for a new directory, but that was also true in 2.12 using patch https://review.whamcloud.com/30598 "LU-10277 utils: 'lfs mkdir -i -1' pick the less full MDTs", so it isn't clear how to enable the LU-11213 functionality to balance directories across MDTs?

      There should be a way for "mkdir(2)" from a normal application (not "lfs mkdir -i -1") to be able to create remote (1-stripe) directories in the filesystem, and it should be possible to set this by default on the root directory (per LU-11213). This is critical for being able to use multiple MDTs effectively without users knowing the details of how to configure striped/remote directories manually, or be forced to set all directories as striped (unwelcome due to performance overhead).

      The default mdt_qos_threshold_rr value should be reduced significantly (e.g. 1-2% and/or modified so that some amount of MDT balancing is active when the filesystem is balanced, at least in the root directory by default. Otherwise, without users understanding the details of DNE MDT0000 will hold all of the inodes, when it would be better if the top 1 or 2 levels of directories should be distributed across MDTs.

      Maybe this is mostly a documentation issue, and the "lfs-setdirstripe.1" man page needs to be updated to be more clear so I can understand what needs to be done to enable this? (also the usage message for setdirstripe/mkdir should remove the "This can only be done on MDT0 with the right of administrator" message.)

      Attachments

        Issue Links

          Activity

            [LU-13417] DNE3: mkdir() automatically create remote directory on MDS which has more space
            pjones Peter Jones added a comment -

            Landed for 2.15

            pjones Peter Jones added a comment - Landed for 2.15

            Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/38553/
            Subject: LU-13417 mdd: set default LMV on ROOT
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 3e04b0fd6c3dd36372f33c54ea5f401c27485d60

            gerrit Gerrit Updater added a comment - Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/38553/ Subject: LU-13417 mdd: set default LMV on ROOT Project: fs/lustre-release Branch: master Current Patch Set: Commit: 3e04b0fd6c3dd36372f33c54ea5f401c27485d60

            Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/44315/
            Subject: LU-13417 test: mkdir_on_mdt0() in more tests
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 618625af42b9ff0427b096996ddf07a327689ec8

            gerrit Gerrit Updater added a comment - Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/44315/ Subject: LU-13417 test: mkdir_on_mdt0() in more tests Project: fs/lustre-release Branch: master Current Patch Set: Commit: 618625af42b9ff0427b096996ddf07a327689ec8

            Andreas Dilger (adilger@whamcloud.com) merged in patch https://review.whamcloud.com/44384/
            Subject: LU-13417 test: generate uneven MDTs early for sanity 413
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 233344d451e567c71726bcb071f45cf8f1c6ef3e

            gerrit Gerrit Updater added a comment - Andreas Dilger (adilger@whamcloud.com) merged in patch https://review.whamcloud.com/44384/ Subject: LU-13417 test: generate uneven MDTs early for sanity 413 Project: fs/lustre-release Branch: master Current Patch Set: Commit: 233344d451e567c71726bcb071f45cf8f1c6ef3e

            Lai Siyao (lai.siyao@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/44384
            Subject: LU-13417 test: generate uneven MDTs early for sanity 413
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: efa31b02f470cf90281504ca49185a37006523d8

            gerrit Gerrit Updater added a comment - Lai Siyao (lai.siyao@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/44384 Subject: LU-13417 test: generate uneven MDTs early for sanity 413 Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: efa31b02f470cf90281504ca49185a37006523d8

            Lai Siyao (lai.siyao@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/44315
            Subject: LU-13417 test: mkdir_on_mdt0() in more tests
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 2bd9a10b17db3df7f87b4068a3746bc114069b46

            gerrit Gerrit Updater added a comment - Lai Siyao (lai.siyao@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/44315 Subject: LU-13417 test: mkdir_on_mdt0() in more tests Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 2bd9a10b17db3df7f87b4068a3746bc114069b46

            People

              laisiyao Lai Siyao
              adilger Andreas Dilger
              Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: