dimberman commented on a change in pull request #15137:
URL: https://github.com/apache/airflow/pull/15137#discussion_r606390160
##########
File path: airflow/kubernetes/pod_launcher.py
##########
@@ -98,6 +98,12 @@ def delete_pod(self, pod: V1Pod):
if e.status != 404:
raise
+ @tenacity.retry(
+ stop=tenacity.stop_after_attempt(3),
+ wait=tenacity.wait_random_exponential(),
+ reraise=True,
+ retry=tenacity.retry_if_exception_type(ApiException),
Review comment:
@SamWheating I've created an issue here
https://github.com/apache/airflow/issues/15164. Please comment on it and I will
assign it to you :).
I'd say you should attack this quickly, as without this fix we won't be able
to release this fix until the next Airflow release (and it will require an
Airflow upgrade).
That said, I'm glad to make this PR a high priority, so once it's ready I
can be fast with PR reviews to get it through sooner than later.
--
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
