[GitHub] [airflow] ikholopov-omni commented on issue #28751: KubernetesExecutor leaves failed pods due to deepcopy issue with Google providers

["ikholopov-omni (via GitHub)" <gi...@apache.org>]

ikholopov-omni commented on issue #28751:
URL: https://github.com/apache/airflow/issues/28751#issuecomment-1428036403

   I believe this a broader issue with google client that persists objects with `google.api_core.gapic_v1.method.DEFAULT` as a default parameter - those do not handle the deepcopy properly.
   
   On one hand this can be easily fixed by memoing this object which can be safely assumed as a singleton, on the other thing it highlights a bigger problem - the assumption of operators having a proper deepcopy support is implicit, applies to all operators and isn't enforced in any of the unittests (and would be hard to enforce properly as some operators have a complicated state). 
   
   Should we consider tackling the second part globally for all providers as a seprate bug?


-- 
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-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org