moiseenkov commented on PR #44568:
URL: https://github.com/apache/airflow/pull/44568#issuecomment-2522918768

   > This is one option yes. There is another option. You can add the 
`cncf.kubernetes` provider to the list of "chicken-egg" providers in 
`src/airflow_breeze/global_constants.py` -> similarly to what I've done in 
#44714 (about to be merged).
   
   Thank you for the suggestion! I followed this idea.
   
   > Why not just using AirflowOptionalProviderFeatureException?
   
   I've also considered this, but unfortunately provided changes aren't 
optional. For example the `GKEStartKueueInsideClusterOperator` now inherits 
from `KubernetesInstallKueueOperator` which was introduced in this PR. At the 
same time the `GKEStartKueueInsideClusterOperator` is a bit older and will be 
broken if the Kubernetes provider isn't upgraded. Thus I had to bump the 
cncf-kubernetes provider version in google provider's dependencies.


-- 
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: commits-unsubscr...@airflow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to