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

ost-pools test_24: Stripe count -1 not on /mnt/lustre/d24.ost-pools/dir4/f24.ost-pools0:2

Details

    • Bug
    • Resolution: Fixed
    • Critical
    • Lustre 2.10.0
    • None
    • 3
    • 9223372036854775807

    Description

      This issue was created by maloo for John Hammond <john.hammond@intel.com>

      This issue relates to the following test suite run: https://testing.hpdd.intel.com/test_sets/7cf06cee-260f-11e7-9de9-5254006e85c2.

      The sub-test test_24 failed with the following error:

      Stripe count -1 not on /mnt/lustre/d24.ost-pools/dir4/f24.ost-pools0:2
      

      It looks like /.../dir4 is inheriting lov stripe count -1 from the FS root and the test session has OSTCOUNT=2 so the file is created with the correct stripe count but the subtest is checking that -1 == 2 which fails.

      Info required for matching: ost-pools 24

      Attachments

        Issue Links

          Activity

            [LU-9383] ost-pools test_24: Stripe count -1 not on /mnt/lustre/d24.ost-pools/dir4/f24.ost-pools0:2
            pjones Peter Jones added a comment -

            Landed for 2.10

            pjones Peter Jones added a comment - Landed for 2.10

            Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/26771/
            Subject: LU-9383 test: use a subdir in sanityn test_71b()
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 2870ed7687d40520f1d94b0e7d66507450a6a506

            gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/26771/ Subject: LU-9383 test: use a subdir in sanityn test_71b() Project: fs/lustre-release Branch: master Current Patch Set: Commit: 2870ed7687d40520f1d94b0e7d66507450a6a506
            bogl Bob Glossman (Inactive) added a comment - another on master: https://testing.hpdd.intel.com/test_sets/4304c198-26ae-11e7-b742-5254006e85c2

            John L. Hammond (john.hammond@intel.com) uploaded a new patch: https://review.whamcloud.com/26771
            Subject: LU-9383 test: use a subdir in sanityn test_71b()
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 6f4aaca267d1f6613dc88356c6b03b24e2958902

            gerrit Gerrit Updater added a comment - John L. Hammond (john.hammond@intel.com) uploaded a new patch: https://review.whamcloud.com/26771 Subject: LU-9383 test: use a subdir in sanityn test_71b() Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 6f4aaca267d1f6613dc88356c6b03b24e2958902
            jhammond John Hammond added a comment -

            Probably introduced by the following change to sanityn.sh:

            commit 8b8bc6eee05758cec346f1f9aeed92f3334d8cf3
            Author:     Ashish Purkar <ashish.purkar@seagate.com>
            AuthorDate: Thu Feb 25 20:55:47 2016 -0800
            Commit:     Oleg Drokin <oleg.drokin@intel.com>
            CommitDate: Wed Apr 19 04:45:34 2017 +0000
            
                LU-7819 tests: fix checkfiemap for stripecount > 1
                
                If sanityn test_71 is ran with stripecount > 1, it fails
                with "error while ioctl 95" i.e. EOPNOTSUPP.
                
                FIEMAP_FLAG_DEVICE_ORDER flag must be set while issuing
                FS_IOC_FIEMAP ioctl.
                
                New test case 71b to check fiemap for stripe count > 1
                Renamed original test_71 to 71a
            
            +test_71b() {
            +       local server_version=$(lustre_version_code $SINGLEMDS)
            +
            +       [[ $server_version -lt $(version_code 2.1.6) ]] &&
            +               skip "Need MDS version at least 2.1.6" && return
            +
            +       # Patch not applied to 2.2 and 2.3 branches
            +       [[ $server_version -ge $(version_code 2.2.0) ]] &&
            +       [[ $server_version -lt $(version_code 2.4.0) ]] &&
            +               skip "Need MDS version earlier than 2.2.0 or at least 2.4.0" &&
            +                       return
            +       [[ $OSTCOUNT -ge 2 ]] || { skip "need at least 2 osts"; return; }
            +
            +       checkfiemap --test ||
            +               { skip "error $?: checkfiemap failed" && return; }
            +
            +       $LFS setstripe -c -1 $DIR1 || error "setstripe failed"
            +       dd if=/dev/urandom of=$DIR1/$tfile bs=40K count=1
            +       [ "$(facet_fstype ost$(($($GETSTRIPE -i $DIR1/$tfile) + 1)))" = \
            +               "zfs" ] &&
            +               skip "ORI-366/LU-1941: FIEMAP unimplemented on ZFS" && return 0
            +       checkfiemap $DIR1/$tfile 40960 || error "checkfiemap failed"
            +}
            +run_test 71b "check fiemap support for stripecount > 1"
            

            This subtest should be using a subdir of the root.

            jhammond John Hammond added a comment - Probably introduced by the following change to sanityn.sh: commit 8b8bc6eee05758cec346f1f9aeed92f3334d8cf3 Author: Ashish Purkar <ashish.purkar@seagate.com> AuthorDate: Thu Feb 25 20:55:47 2016 -0800 Commit: Oleg Drokin <oleg.drokin@intel.com> CommitDate: Wed Apr 19 04:45:34 2017 +0000 LU-7819 tests: fix checkfiemap for stripecount > 1 If sanityn test_71 is ran with stripecount > 1, it fails with "error while ioctl 95" i.e. EOPNOTSUPP. FIEMAP_FLAG_DEVICE_ORDER flag must be set while issuing FS_IOC_FIEMAP ioctl. New test case 71b to check fiemap for stripe count > 1 Renamed original test_71 to 71a +test_71b() { + local server_version=$(lustre_version_code $SINGLEMDS) + + [[ $server_version -lt $(version_code 2.1.6) ]] && + skip "Need MDS version at least 2.1.6" && return + + # Patch not applied to 2.2 and 2.3 branches + [[ $server_version -ge $(version_code 2.2.0) ]] && + [[ $server_version -lt $(version_code 2.4.0) ]] && + skip "Need MDS version earlier than 2.2.0 or at least 2.4.0" && + return + [[ $OSTCOUNT -ge 2 ]] || { skip "need at least 2 osts"; return; } + + checkfiemap --test || + { skip "error $?: checkfiemap failed" && return; } + + $LFS setstripe -c -1 $DIR1 || error "setstripe failed" + dd if=/dev/urandom of=$DIR1/$tfile bs=40K count=1 + [ "$(facet_fstype ost$(($($GETSTRIPE -i $DIR1/$tfile) + 1)))" = \ + "zfs" ] && + skip "ORI-366/LU-1941: FIEMAP unimplemented on ZFS" && return 0 + checkfiemap $DIR1/$tfile 40960 || error "checkfiemap failed" +} +run_test 71b "check fiemap support for stripecount > 1" This subtest should be using a subdir of the root.

            People

              jhammond John Hammond
              maloo Maloo
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: