From 7aece3c75abba4da6f80f855515970531de4a3bf Mon Sep 17 00:00:00 2001 From: rustfs-ci Date: Thu, 29 Oct 2026 14:49:50 +0800 Subject: [PATCH] fix(kms): unsupported cases must not fail a plain lane run #113's validation migration made any UNSUPPORTED topology result set overall_fail, so every kms topology exited 1 despite PASS=42/43 FAIL=0 (6-7 rc-restart-gated skips per topology are the expected lane shape). Keep teardown failures fatal everywhere; treat UNSUPPORTED as incomplete only in validation mode. --- rustfs-kms-test.sh | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/rustfs-kms-test.sh b/rustfs-kms-test.sh index 7391bf7e..6f102231 100755 --- a/rustfs-kms-test.sh +++ b/rustfs-kms-test.sh @@ -2373,7 +2373,13 @@ main() { fi fi if [ "${FAIL}" -gt 0 ]; then overall_fail=1; fi - if [ "${UNSUPPORTED}" -gt 0 ] || [ "${cleanup_status}" != PASS ]; then overall_fail=1; fi + # A teardown failure is a real infra error in any mode. UNSUPPORTED + # cases (environment gates such as the rc-restart exit-7 skip) are + # expected in plain lane runs — the validation flow is the only mode + # that treats them as incomplete. Failing the lane on them made every + # topology exit 1 despite PASS=42/43 FAIL=0 (chain 40796474356). + if [ "${cleanup_status}" != PASS ]; then overall_fail=1; fi + if [ -n "${VALIDATION_CASE}" ] && [ "${UNSUPPORTED}" -gt 0 ]; then overall_fail=1; fi done done final_summary