This is an automated email from the ASF dual-hosted git repository.
vatsrahul1001 pushed a commit to branch v3-2-test
in repository https://gitbox.apache.org/repos/asf/airflow.git
commit f17c389d7a0cf9b8de662ae6bc85e660e7e8ca7d
Author: github-actions[bot]
<41898282+github-actions[bot]@users.noreply.github.com>
AuthorDate: Mon Apr 20 06:36:22 2026 +0200
[v3-2-test] Revert "Stop masking quarantined unit test failures (#65500)"
(#65515) (#65516)
* 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.
(cherry picked from commit b7fcda5136f0600ce74f600129ac800a43177df1)
Co-authored-by: Jarek Potiuk <[email protected]>
---
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 92a5a29be40..1858b09adba 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
