This is an automated email from the ASF dual-hosted git repository.
jedcunningham 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 0568e9a604 Exit instead of return in case of Providers[fab] test is
ignored (#40089)
0568e9a604 is described below
commit 0568e9a604a12f29e79319f12ee6d86675eb4a0f
Author: Jarek Potiuk <[email protected]>
AuthorDate: Thu Jun 6 18:26:39 2024 +0200
Exit instead of return in case of Providers[fab] test is ignored (#40089)
The #40059 added an exception for fab tests not to run on old
airflow releases but it was wrong doing return instead of exit
---
Dockerfile.ci | 2 +-
scripts/docker/entrypoint_ci.sh | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/Dockerfile.ci b/Dockerfile.ci
index 873ec62f25..c2ebddc2f1 100644
--- a/Dockerfile.ci
+++ b/Dockerfile.ci
@@ -989,7 +989,7 @@ function determine_airflow_to_use() {
echo
echo "${COLOR_YELLOW}Skipping FAB tests on Airflow 2.7 and 2.8
because of FAB incompatibility with them${COLOR_RESET}"
echo
- return
+ exit 0
fi
python "${IN_CONTAINER_DIR}/install_airflow_and_providers.py"
# Some packages might leave legacy typing module which causes test
issues
diff --git a/scripts/docker/entrypoint_ci.sh b/scripts/docker/entrypoint_ci.sh
index 6b3130adba..9847cfdbb1 100755
--- a/scripts/docker/entrypoint_ci.sh
+++ b/scripts/docker/entrypoint_ci.sh
@@ -208,7 +208,7 @@ function determine_airflow_to_use() {
echo
echo "${COLOR_YELLOW}Skipping FAB tests on Airflow 2.7 and 2.8
because of FAB incompatibility with them${COLOR_RESET}"
echo
- return
+ exit 0
fi
python "${IN_CONTAINER_DIR}/install_airflow_and_providers.py"
# Some packages might leave legacy typing module which causes test
issues
