hussein-awala commented on PR #30718: URL: https://github.com/apache/airflow/pull/30718#issuecomment-1519119487
> 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. -- 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]
