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 d6df0786cf Make kerberos an optional and devel dependency for impala
and fab (#41616)
d6df0786cf is described below
commit d6df0786cfe3b7e7ded30c7fd786d685811cac52
Author: Jarek Potiuk <[email protected]>
AuthorDate: Tue Aug 20 19:02:24 2024 +0200
Make kerberos an optional and devel dependency for impala and fab (#41616)
The improved compatibility tests detected that FAB provider tests
have implicit dependency on kerberos - similar as impala. This change
make kerberos an optional dependency of FAB as well as it as
development dependency for both impala and FAB.
---
.github/workflows/basic-tests.yml | 4 ++--
airflow/providers/apache/impala/provider.yaml | 2 ++
airflow/providers/fab/provider.yaml | 8 ++++++++
generated/provider_dependencies.json | 8 ++++++--
4 files changed, 18 insertions(+), 4 deletions(-)
diff --git a/.github/workflows/basic-tests.yml
b/.github/workflows/basic-tests.yml
index 9828a14993..8acb271d8f 100644
--- a/.github/workflows/basic-tests.yml
+++ b/.github/workflows/basic-tests.yml
@@ -154,8 +154,8 @@ jobs:
env:
HATCH_ENV: "test"
working-directory: ./clients/python
- - name: "Install Airflow in editable mode with fab for webserver tests"
- run: pip install -e ".[fab]"
+ - name: "Install Airflow with fab for webserver tests"
+ run: pip install ".[fab]"
- name: "Install Python client"
run: pip install ./dist/apache_airflow_client-*.whl
- name: "Initialize Airflow DB and start webserver"
diff --git a/airflow/providers/apache/impala/provider.yaml
b/airflow/providers/apache/impala/provider.yaml
index e2cdfd1332..2000a2cd8b 100644
--- a/airflow/providers/apache/impala/provider.yaml
+++ b/airflow/providers/apache/impala/provider.yaml
@@ -48,6 +48,8 @@ additional-extras:
dependencies:
- kerberos>=1.3.0
+devel-dependencies:
+ - kerberos>=1.3.0
integrations:
- integration-name: Apache Impala
diff --git a/airflow/providers/fab/provider.yaml
b/airflow/providers/fab/provider.yaml
index 741297d446..7b7cc4c9a7 100644
--- a/airflow/providers/fab/provider.yaml
+++ b/airflow/providers/fab/provider.yaml
@@ -57,6 +57,14 @@ dependencies:
- google-re2>=1.0
- jmespath>=0.7.0
+additional-extras:
+ - name: kerberos
+ dependencies:
+ - kerberos>=1.3.0
+
+devel-dependencies:
+ - kerberos>=1.3.0
+
config:
fab:
description: This section contains configs specific to FAB provider.
diff --git a/generated/provider_dependencies.json
b/generated/provider_dependencies.json
index 96a2671f2b..99de0b489f 100644
--- a/generated/provider_dependencies.json
+++ b/generated/provider_dependencies.json
@@ -207,7 +207,9 @@
"apache-airflow>=2.8.0",
"impyla>=0.18.0,<1.0"
],
- "devel-deps": [],
+ "devel-deps": [
+ "kerberos>=1.3.0"
+ ],
"plugins": [],
"cross-providers-deps": [
"common.sql"
@@ -556,7 +558,9 @@
"google-re2>=1.0",
"jmespath>=0.7.0"
],
- "devel-deps": [],
+ "devel-deps": [
+ "kerberos>=1.3.0"
+ ],
"plugins": [],
"cross-providers-deps": [],
"excluded-python-versions": [],
