ephraimbuddy commented on code in PR #40183:
URL: https://github.com/apache/airflow/pull/40183#discussion_r1637720232
##########
airflow/providers/cncf/kubernetes/executors/kubernetes_executor_utils.py:
##########
@@ -269,21 +271,6 @@ def process_status(
(pod_name, namespace, TaskInstanceState.FAILED, annotations,
resource_version)
)
elif status == "Succeeded":
- # We get multiple events once the pod hits a terminal state, and
we only want to
- # send it along to the scheduler once.
- # If our event type is DELETED, we have the POD_EXECUTOR_DONE_KEY,
or the pod has
- # a deletion timestamp, we've already seen the initial Succeeded
event and sent it
- # along to the scheduler.
- if (
Review Comment:
Yeah, found out this is still needed.
--
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-unsubscr...@airflow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
