XD-DENG commented on code in PR #27611:
URL: https://github.com/apache/airflow/pull/27611#discussion_r1019730009
##########
airflow/executors/kubernetes_executor.py:
##########
@@ -646,6 +649,14 @@ def sync(self) -> None:
json.loads(e.body)["message"],
)
self.task_queue.put(task)
+ except PodMutationHookException as e:
+ key, _, _, _ = task
+ self.log.warning(
+ 'Pod Mutation Hook failed for the task %s.
Re-queueing. Details: %s',
Review Comment:
I already thought of this: it can literally be any type of error, so
unfortunately there is no way for us to differentiate more granularly.
But yes I'm sold that we may not want to re-queue here. Let me update
accordingly.
--
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]
