(airflow) branch main updated: Remove FAB-related exclusion in SQLA tests (#74180)

[vincbeck]

This is an automated email from the ASF dual-hosted git repository.

vincbeck 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 fa65343163d Remove FAB-related exclusion in SQLA tests (#74180)
fa65343163d is described below

commit fa65343163dc8ece48898cb6ce92ee6cd1b6ffb1
Author: Dev-iL <6509619+dev...@users.noreply.github.com>
AuthorDate: Mon Oct 5 15:59:29 2026 +0300

    Remove FAB-related exclusion in SQLA tests (#74180)
---
 Dockerfile.ci                                      | 7 ++-----
 docker-tests/tests/docker_tests/test_prod_image.py | 3 ---
 scripts/docker/entrypoint_ci.sh                    | 7 ++-----
 3 files changed, 4 insertions(+), 13 deletions(-)

diff --git a/Dockerfile.ci b/Dockerfile.ci
index 75bb1e19c82..b2125f7c38d 100644
--- a/Dockerfile.ci
+++ b/Dockerfile.ci
@@ -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() {
diff --git a/docker-tests/tests/docker_tests/test_prod_image.py 
b/docker-tests/tests/docker_tests/test_prod_image.py
index 43b5f3cea96..25e8783276c 100644
--- a/docker-tests/tests/docker_tests/test_prod_image.py
+++ b/docker-tests/tests/docker_tests/test_prod_image.py
@@ -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)
 
diff --git a/scripts/docker/entrypoint_ci.sh b/scripts/docker/entrypoint_ci.sh
index b27b986df65..d385d6ef9be 100755
--- a/scripts/docker/entrypoint_ci.sh
+++ b/scripts/docker/entrypoint_ci.sh
@@ -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