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

potiuk 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 063c550405 CI providers discovery update when airflow min 2.7.0 
(#42767)
063c550405 is described below

commit 063c550405e613b2d6ea50fd04bf63506410efd6
Author: Gopal Dirisala <39794726+dir...@users.noreply.github.com>
AuthorDate: Tue Oct 8 23:44:42 2024 +0530

    CI providers discovery update when airflow min 2.7.0 (#42767)
---
 scripts/in_container/verify_providers.py | 18 ++++--------------
 1 file changed, 4 insertions(+), 14 deletions(-)

diff --git a/scripts/in_container/verify_providers.py 
b/scripts/in_container/verify_providers.py
index b3602dead8..f67334c451 100755
--- a/scripts/in_container/verify_providers.py
+++ b/scripts/in_container/verify_providers.py
@@ -727,10 +727,6 @@ def verify_provider_classes() -> tuple[list[str], 
list[str]]:
 
 
 def run_provider_discovery():
-    import packaging.version
-
-    import airflow.version
-
     console.print("[bright_blue]List all providers[/]\n")
     subprocess.run(["airflow", "providers", "list"], check=True)
     console.print("[bright_blue]List all hooks[/]\n")
@@ -747,16 +743,10 @@ def run_provider_discovery():
     subprocess.run(["airflow", "providers", "secrets"], check=True)
     console.print("[bright_blue]List all auth backends[/]\n")
     subprocess.run(["airflow", "providers", "auth"], check=True)
-    if packaging.version.parse(airflow.version.version) >= 
packaging.version.parse("2.7.0.dev0"):
-        # CI also check if our providers are installable and discoverable in 
airflow older versions
-        # But the triggers command is not available till airflow 2.7.0
-        # TODO: Remove this condition once airflow dependency in providers are 
> 2.7.0
-        console.print("[bright_blue]List all triggers[/]\n")
-        subprocess.run(["airflow", "providers", "triggers"], check=True)
-        # CI also check if our providers are installable and discoverable in 
airflow older versions
-        # But the executors command is not available till airflow 2.7.0
-        console.print("[bright_blue]List all executors[/]\n")
-        subprocess.run(["airflow", "providers", "executors"], check=True)
+    console.print("[bright_blue]List all triggers[/]\n")
+    subprocess.run(["airflow", "providers", "triggers"], check=True)
+    console.print("[bright_blue]List all executors[/]\n")
+    subprocess.run(["airflow", "providers", "executors"], check=True)
 
 
 AIRFLOW_LOCAL_SETTINGS_PATH = Path("/opt/airflow") / 
"airflow_local_settings.py"

Reply via email to