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

   > @eladkal , @potiuk,
   > 
   > This PR contains changes for both Google and cncf-kubernetes providers, 
and the Google provider depends on changes in cncf-kuberneets. For this reason 
I had to bump cncf-kubernetes version in Google provider's 
[dependencies](https://github.com/apache/airflow/pull/44568/files#diff-bfccba1ff8b4b3e462131ef3d0869d7760d64a2f3b6f1a15b89157a4c7d68404L191)
 and it now breaks CI because it can't install cncf-kubernetes provider that 
wasn't released yet. Therefore, should I split this PR, so we could release 
these changes gradually (cncf first, and google later) or we are OK with 
merging it all together now.
   
   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 
https://github.com/apache/airflow/pull/44714 (about to be merged).


-- 
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