Whamcloud - gitweb
LU-15589 tests: skip sanity test_230s for interop 98/46598/2
authorAndreas Dilger <adilger@whamcloud.com>
Wed, 23 Feb 2022 19:24:12 +0000 (12:24 -0700)
committerOleg Drokin <green@whamcloud.com>
Thu, 17 Mar 2022 07:02:34 +0000 (07:02 +0000)
Sanity test_230s was added in 2.14.52 but incorrectly checked for
MDS version 2.13.57 for interop, likely because that was the version
present at the time the patch was originally written, but it was
only landed later.

Test-Parameters: trivial serverversion=2.14.0 testlist=sanity env=ONLY=230s
Fixes: 65e3e4050ec5 ("LU-14366 mdt: lfs mkdir should return -EEXIST if exists")
Signed-off-by: Andreas Dilger <adilger@whamcloud.com>
Change-Id: I5d046ad224a29558866453972c9c33b5da3a9037
Reviewed-on: https://review.whamcloud.com/46598
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Lai Siyao <lai.siyao@whamcloud.com>
Reviewed-by: Alena Nikitenko <anikitenko@ddn.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
lustre/tests/sanity.sh

index f4eea8e..22fde7f 100755 (executable)
@@ -20447,8 +20447,8 @@ test_230r() {
 run_test 230r "migrate with too many local locks"
 
 test_230s() {
-       [ $MDS1_VERSION -ge $(version_code 2.13.57) ] ||
-               skip "Need MDS version at least 2.13.57"
+       (( $MDS1_VERSION >= $(version_code 2.14.52) )) ||
+               skip "Need MDS version at least 2.14.52"
 
        local mdts=$(comma_list $(mdts_nodes))
        local restripe_status=$(do_facet mds1 $LCTL get_param -n \
@@ -20462,7 +20462,7 @@ test_230s() {
                do_nodes $mdts "$LCTL set_param mdt.*.enable_dir_restripe=$st"
                test_mkdir $DIR/$tdir
                $LFS mkdir $DIR/$tdir |& grep "File exists" ||
-                       error "$LFS mkdir doesn't return -EEXIST if target exists"
+                       error "$LFS mkdir should return EEXIST if target exists"
                rmdir $DIR/$tdir
        done
 }