Whamcloud - gitweb
LU-12175 tests: Partial revert of LU-11636 05/34705/3
authorPatrick Farrell <pfarrell@whamcloud.com>
Thu, 18 Apr 2019 16:42:24 +0000 (12:42 -0400)
committerOleg Drokin <green@whamcloud.com>
Thu, 18 Apr 2019 22:03:41 +0000 (22:03 +0000)
Since landing:
LU-11636/https://review.whamcloud.com/33611/
07b271c0972757772a129e9a6370dbb163f16a06

we have seen high failure rates in several tests:
sanity 208 (LU-12175)
sanity 133g (LU-12171)
recovery-small 107 (LU-12176)
recovery-small 134 (LU-11560)

Testing with a full revert of LU-11636 showed that removing
it stops those failure.  Our best guess is that the
randomization of stripe count and mdt index is the cause,
so this patch reverts just those changes.

Test-Parameters: trivial
Signed-off-by: Patrick Farrell <pfarrell@whamcloud.com>
Change-Id: Id6fa3079e29178555827f4e1f39d51cf8d62cf31
Reviewed-on: https://review.whamcloud.com/34705
Tested-by: Jenkins
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Reviewed-by: James Nunez <jnunez@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
lustre/tests/test-framework.sh

index fb831e1..158eeb1 100755 (executable)
@@ -8352,8 +8352,8 @@ generate_logname() {
 test_mkdir() {
        local path
        local p_option
-       local dirstripe_count=${DIRSTRIPE_COUNT:-"-1"}
-       local dirstripe_index=${DIRSTRIPE_INDEX:-"-1"}
+       local dirstripe_count=${DIRSTRIPE_COUNT:-"2"}
+       local dirstripe_index=${DIRSTRIPE_INDEX:-$((base % $MDSCOUNT))}
        local OPTIND=1
 
        while getopts "c:i:p" opt; do
@@ -8385,7 +8385,7 @@ test_mkdir() {
                local mdt_index
 
                if [ $dirstripe_index -eq -1 ]; then
-                       mdt_index=$((RANDOM % MDSCOUNT))
+                       mdt_index=$((base % MDSCOUNT))
                else
                        mdt_index=$dirstripe_index
                fi