Whamcloud - gitweb
EX-9125 tests: ignore sanity-compr/1008 failures
authorAndreas Dilger <adilger@whamcloud.com>
Tue, 13 Feb 2024 04:01:20 +0000 (21:01 -0700)
committerAndreas Dilger <adilger@whamcloud.com>
Wed, 14 Feb 2024 19:17:59 +0000 (19:17 +0000)
Temporarily ignore test failures for sanity-compr.sh test_1008 on
SLES15 and aarch64/ppc64le due to very high failure rates on those
systems.

Rename the *second* test_1008 to test_1080 so it can run, and allow
other new tests to start using disjoint numbers to avoid conflicts.

Test-Parameters: trivial testlist=sanity-compr.sh env=ONLY="1008 1080"
Fixes: bd462ce8e4 ("EX-7795 tests: add sanity-compr test for dir compression")
Fixes: 7546ae79e9 ("EX-8688 csdc: Add header checksum verification calculation")
Signed-off-by: Andreas Dilger <adilger@whamcloud.com>
Change-Id: I199613b1e5b1ee8ea7ca4287a6dfe090257ed72f
Reviewed-on: https://review.whamcloud.com/c/ex/lustre-release/+/54019
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Artem Blagodarenko <ablagodarenko@ddn.com>
Reviewed-by: Jian Yu <yujian@whamcloud.com>
lustre/tests/sanity-compr.sh

index 37a9e49..36d9823 100644 (file)
@@ -1177,14 +1177,24 @@ test_1008() {
        done # compr_level
        done # compr_type
 
-       (( failed_estimates <= total_estimates * failed_margin / 100 )) ||
-               error "failed estimates > ${failed_margin}% of total estimates"
+       if (( failed_estimates > total_estimates * failed_margin / 100 )); then
+               local arch=$(uname -m)
+               # these should be fixed, but ignore so other patches can land.
+               # better to continue running tests on these systems than skip.
+               if [[ $arch == aarch64 || $arch == ppcle64 ]]; then
+                       error_ignore EX-9125 "failed > $failed_margin% on $arch"
+               elif [[ -r /etc/SuSE-release || -r /etc/SUSE-brand ]]; then
+                       error_ignore EX-9125 "failed > $failed_margin% on SLES"
+               else
+                       error "failed > ${failed_margin}% of total estimates"
+               fi
+       fi
 }
 run_test 1008 "validate directory space usage reduction with compression"
 
-test_1008() {
-       (( MDS1_VERSION >= $(version_code 2.14.0-ddn121) )) ||
-               skip "Need MDS version at least 2.14.0-ddn121"
+test_1080() {
+       (( MDS1_VERSION >= $(version_code 2.14.0-ddn128) )) ||
+               skip "Need MDS version at least 2.14.0-ddn128"
 
        test_mkdir -p $DIR/$tdir
        local tf=$DIR/$tdir/$tfile
@@ -1276,7 +1286,7 @@ test_1008() {
        hexdump -C $tf2 | head
        $LCTL set_param fail_loc=0x0
 }
-run_test 1008 "Compression header error tolerance"
+run_test 1080 "Compression header error tolerance"
 
 complete_test $SECONDS
 check_and_cleanup_lustre