eladkal commented on PR #30718: URL: https://github.com/apache/airflow/pull/30718#issuecomment-1519120383
> > Are you planning to change the operators based on KPO in a followup PR? Just want to make sure it isn't missed. For example, the [eks pod operator](https://github.com/apache/airflow/blob/4c5cebf04555aa7b892a50a85f2f488acc4c057b/airflow/providers/amazon/aws/operators/eks.py#L685-L697). > > @jedcunningham yes, in this PR, I have ensured that it is fully backward compatible. This means that users can update the `cncf.kubernetes` provider version and still use it with older versions of Amazon or Google providers: > > https://github.com/apache/airflow/blob/a99e3ba37ee52a2a63998979d492213e0a68a24c/airflow/providers/cncf/kubernetes/operators/pod.py#L387-L393 > > Once this PR is merged, I'll create a separate PR for each provider to deprecate the argument `is_delete_operator_pod` and use the new one. We can do it in this PR if you prefer.. no reason to keep it in seperated PRs unless that is how you prefer. -- 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]
