eladkal commented on PR #30718: URL: https://github.com/apache/airflow/pull/30718#issuecomment-1573610089
> Using `on_finish_action` as an argument for the super function won't work with the older version of the KPO. Additionally, providing `is_delete_operator_pod` will result in a deprecation warning being logged, even if the user provides `on_finish_action` for the GKE/EKS operator. Is this a concern only for google provider? if so I think we can just bump the k8s provider version dependency for google. I'm suggesting it because currently google provider depended on https://github.com/apache/airflow/blob/dc5bf3fd02c589578209cb0dd5b7d005b1516ae9/airflow/providers/google/provider.yaml#L1121-L1123 which means it must use the latest version which is 7.0.0 I think bumping the dependency as `apache-airflow-providers-cncf-kubernetes>7.0.0` the difference between 7.0.0 to 7.1.0 will be very little and it will solve your concern. -- 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]
