This is an automated email from the ASF dual-hosted git repository.
shahar1 pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new b7fcda5136f Revert "Stop masking quarantined unit test failures
(#65500)" (#65515)
b7fcda5136f is described below
commit b7fcda5136f0600ce74f600129ac800a43177df1
Author: Jarek Potiuk <[email protected]>
AuthorDate: Mon Apr 20 05:01:39 2026 +0200
Revert "Stop masking quarantined unit test failures (#65500)" (#65515)
* Revert "Stop masking quarantined unit test failures (#65500)"
This reverts commit 67e7cc1912c47b6a87657b5c78a02248750b5bc1.
* Document why quarantined tests use `|| true`
Adds a short comment next to both `breeze testing ... --test-type
"All-Quarantined" || true` calls explaining that the shell `|| true`
is intentional — quarantined tests are known-flaky and must not fail the
overall CI run; they are reported separately.
---
scripts/ci/testing/run_unit_tests.sh | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/scripts/ci/testing/run_unit_tests.sh
b/scripts/ci/testing/run_unit_tests.sh
index da130e17527..0d6fc76bbda 100755
--- a/scripts/ci/testing/run_unit_tests.sh
+++ b/scripts/ci/testing/run_unit_tests.sh
@@ -50,7 +50,9 @@ function core_tests() {
set +x
elif [[ "${TEST_SCOPE}" == "Quarantined" ]]; then
set -x
- breeze testing core-tests --test-type "All-Quarantined"
+ # The `|| true` is deliberate — quarantined tests are known-flaky and
we do not want a
+ # failing run here to fail the whole CI job; they are reported
separately. Do not remove.
+ breeze testing core-tests --test-type "All-Quarantined" || true
RESULT=$?
set +x
elif [[ "${TEST_SCOPE}" == "System" ]]; then
@@ -93,7 +95,9 @@ function providers_tests() {
set +x
elif [[ "${TEST_SCOPE}" == "Quarantined" ]]; then
set -x
- breeze testing providers-tests --test-type "All-Quarantined"
+ # The `|| true` is deliberate — quarantined tests are known-flaky and
we do not want a
+ # failing run here to fail the whole CI job; they are reported
separately. Do not remove.
+ breeze testing providers-tests --test-type "All-Quarantined" || true
RESULT=$?
set +x
else
