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