eladkal commented on PR #30718: URL: https://github.com/apache/airflow/pull/30718#issuecomment-1519125487
> IMHO we can merge this PR without updating the operators GKEStartPodOperator and EksPodOperator, and I can create two PRs which do the same thing for these two operators and change the default value for pod_deletion to be consistent with KPO, and we can add them to the next major release of each provider with a breaking change note. WDYT? It's OK to update multiple providers in one PR. I see no reason to break backward compatibility in any provider. This PR add new parameter and deprecate the old one. We should do the same for the other providers that inherit from KPO. -- 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]
