From 0a2c722ca8c5123b9b60b61bb8cbc5b94b4e291e Mon Sep 17 00:00:00 2001 From: Andreas Dilger Date: Mon, 12 Feb 2024 21:01:20 -0700 Subject: [PATCH] EX-9125 tests: ignore sanity-compr/1008 failures 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 Change-Id: I199613b1e5b1ee8ea7ca4287a6dfe090257ed72f Reviewed-on: https://review.whamcloud.com/c/ex/lustre-release/+/54019 Tested-by: jenkins Tested-by: Maloo Reviewed-by: Artem Blagodarenko Reviewed-by: Jian Yu --- lustre/tests/sanity-compr.sh | 22 ++++++++++++++++------ 1 file changed, 16 insertions(+), 6 deletions(-) diff --git a/lustre/tests/sanity-compr.sh b/lustre/tests/sanity-compr.sh index 37a9e49..36d9823 100644 --- a/lustre/tests/sanity-compr.sh +++ b/lustre/tests/sanity-compr.sh @@ -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 -- 1.8.3.1