[LU-9383] ost-pools test_24: Stripe count -1 not on /mnt/lustre/d24.ost-pools/dir4/f24.ost-pools0:2 Created: 21/Apr/17  Updated: 02/May/17  Resolved: 22/Apr/17

Status: Resolved
Project: Lustre
Component/s: None
Affects Version/s: None
Fix Version/s: Lustre 2.10.0

Type: Bug Priority: Critical
Reporter: Maloo Assignee: John Hammond
Resolution: Fixed Votes: 0
Labels: easy

Issue Links:
Related
is related to LU-7819 sanityn test_71: @@@@@@ FAIL: data is... Resolved
Severity: 3
Rank (Obsolete): 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



 Comments   
Comment by John Hammond [ 21/Apr/17 ]

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.

Comment by Gerrit Updater [ 21/Apr/17 ]

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

Comment by Bob Glossman (Inactive) [ 21/Apr/17 ]

another on master:
https://testing.hpdd.intel.com/test_sets/4304c198-26ae-11e7-b742-5254006e85c2

Comment by Gerrit Updater [ 22/Apr/17 ]

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

Comment by Peter Jones [ 22/Apr/17 ]

Landed for 2.10

Generated at Sat Feb 10 02:25:43 UTC 2024 using Jira 9.4.14#940014-sha1:734e6822bbf0d45eff9af51f82432957f73aa32c.