nprasad2077/airflow
1
0 Fork 0
mirror of https://github.com/apache/airflow.git synced 2026-11-04 07:12:48 +00:00
Code Issues Packages Projects Releases Wiki Activity

Remove FAB-related exclusion in SQLA tests (#74180)

Browse Source
This commit is contained in:
Dev-iL authored and GitHub committed 2026-10-05 08:59:29 -04:00
1 parent ae082ab651
commit fa65343163
3 files changed
+4 -13
Download Patch File Download Diff File Expand all files Collapse all files

No files matched your search

Dockerfile.ci
+2 -5
View File
@@ -1479,16 +1479,13 @@ function check_boto_upgrade() {
}
function check_upgrade_sqlalchemy() {
# The python version constraint is a TEMPORARY WORKAROUND to exclude all FAB tests. Is should be removed once we
# upgrade FAB to v5 (PR #50960).
if [[ "${UPGRADE_SQLALCHEMY=}" != "true" || ${PYTHON_MAJOR_MINOR_VERSION} != "3.13" ]]; then
if [[ "${UPGRADE_SQLALCHEMY=}" != "true" ]]; then
return
fi
echo
echo "${COLOR_BLUE}Upgrading sqlalchemy to the latest version to run tests with it${COLOR_RESET}"
echo
uv sync --all-packages --no-install-package apache-airflow-providers-fab --resolution highest \
--no-python-downloads --no-managed-python
uv sync --all-packages --resolution highest --no-python-downloads --no-managed-python
}
function check_downgrade_sqlalchemy() {
docker-tests/tests/docker_tests/test_prod_image.py
-3
View File
@@ -242,9 +242,6 @@ class TestPythonPackages:
if package_name == "providers":
excluded_imports = {f"airflow.providers.{pid}" for pid in excluded_ids}
import_names = [name for name in import_names if name not in excluded_imports]
# FAB provider has import issues on Python 3.13
if python_minor == "3.13":
import_names = [name for name in import_names if name != "airflow.providers.fab"]
if import_names:
run_python_in_docker(f"import {','.join(import_names)}", image=default_docker_image)
scripts/docker/entrypoint_ci.sh
+2 -5
View File
@@ -346,16 +346,13 @@ function check_boto_upgrade() {
# Upgrade sqlalchemy to the latest version to run tests with it
function check_upgrade_sqlalchemy() {
# The python version constraint is a TEMPORARY WORKAROUND to exclude all FAB tests. Is should be removed once we
# upgrade FAB to v5 (PR #50960).
if [[ "${UPGRADE_SQLALCHEMY=}" != "true" || ${PYTHON_MAJOR_MINOR_VERSION} != "3.13" ]]; then
if [[ "${UPGRADE_SQLALCHEMY=}" != "true" ]]; then
return
fi
echo
echo "${COLOR_BLUE}Upgrading sqlalchemy to the latest version to run tests with it${COLOR_RESET}"
echo
uv sync --all-packages --no-install-package apache-airflow-providers-fab --resolution highest \
--no-python-downloads --no-managed-python
uv sync --all-packages --resolution highest --no-python-downloads --no-managed-python
}
# Download minimum supported version of sqlalchemy to run tests with it