You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by el...@apache.org on 2023/02/23 03:22:54 UTC

[airflow] branch main updated: Make cncf.kubernetes required for flink provider (#29710)

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

eladkal 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 68412e1664 Make cncf.kubernetes required for flink provider (#29710)
68412e1664 is described below

commit 68412e166414cbf6228385e1e118ec0939857496
Author: Jarek Potiuk <ja...@potiuk.com>
AuthorDate: Thu Feb 23 04:22:43 2023 +0100

    Make cncf.kubernetes required for flink provider (#29710)
    
    The #29707 added version to optional cncf.kubernetes dependency, but
    in fact the kubernetes provider is a required dependency so we need
    to change it to be so.
---
 airflow/providers/apache/flink/provider.yaml | 15 +--------------
 generated/provider_dependencies.json         |  4 ++--
 2 files changed, 3 insertions(+), 16 deletions(-)

diff --git a/airflow/providers/apache/flink/provider.yaml b/airflow/providers/apache/flink/provider.yaml
index a8d826f5ff..a6a38f0167 100644
--- a/airflow/providers/apache/flink/provider.yaml
+++ b/airflow/providers/apache/flink/provider.yaml
@@ -27,15 +27,7 @@ versions:
 dependencies:
   - apache-airflow>=2.3.0
   - cryptography>=2.0.0
-  # The Kubernetes API is known to introduce problems when upgraded to a MAJOR version. Airflow Core
-  # Uses Kubernetes for Kubernetes executor, and we also know that Kubernetes Python client follows SemVer
-  # (https://github.com/kubernetes-client/python#compatibility). This is a crucial component of Airflow
-  # So we should limit it to the next MAJOR version and only deliberately bump the version when we
-  # tested it, and we know it can be bumped. Bumping this version should also be connected with
-  # limiting minimum airflow version supported in cncf.kubernetes provider, due to the
-  # potential breaking changes in Airflow Core as well (kubernetes is added as extra, so Airflow
-  # core is not hard-limited via install-requirements, only by extra).
-  - kubernetes>=21.7.0,<24
+  - apache-airflow-providers-cncf-kubernetes>=5.1.0
 
 integrations:
   - integration-name: Apache Flink
@@ -54,8 +46,3 @@ sensors:
   - integration-name: Apache Flink
     python-modules:
       - airflow.providers.apache.flink.sensors.flink_kubernetes
-
-additional-extras:
-  - name: cncf.kubernetes
-    dependencies:
-      - apache-airflow-providers-cncf-kubernetes>=5.1.0
diff --git a/generated/provider_dependencies.json b/generated/provider_dependencies.json
index 0deeb0a53f..e4749bf1c2 100644
--- a/generated/provider_dependencies.json
+++ b/generated/provider_dependencies.json
@@ -80,9 +80,9 @@
   },
   "apache.flink": {
     "deps": [
+      "apache-airflow-providers-cncf-kubernetes>=5.1.0",
       "apache-airflow>=2.3.0",
-      "cryptography>=2.0.0",
-      "kubernetes>=21.7.0,<24"
+      "cryptography>=2.0.0"
     ],
     "cross-providers-deps": [
       "cncf.kubernetes"