potiuk commented on code in PR #35714:
URL: https://github.com/apache/airflow/pull/35714#discussion_r1451790580
##########
airflow/providers/google/provider.yaml:
##########
@@ -151,7 +151,7 @@ additional-extras:
- apache-beam[gcp]
- name: cncf.kubernetes
dependencies:
- - apache-airflow-providers-cncf-kubernetes>=7.2.0
+ - apache-airflow-providers-cncf-kubernetes>=7.14.0
Review Comment:
Yeah. No big issue here. Though I think we should review fhose checks and
add runtime checks/warnings, because those dependencies are rarely actually
used and they are mostly informative.
But I think (as a separate PR) in the future providers we could implement a
runtime warning that would be logged at provider import in case we have two
providers that are essentially incompatible.
It should be rather easy to add - just next to our "hard" failure for
minimum airllow version.
Created an issue here: https://github.com/apache/airflow/issues/36773
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
