wolfier commented on code in PR #39924:
URL: https://github.com/apache/airflow/pull/39924#discussion_r1621362041


##########
airflow/providers/cncf/kubernetes/executors/kubernetes_executor_utils.py:
##########
@@ -230,7 +230,6 @@ def process_status(
                 
self.kube_config.worker_pod_pending_fatal_container_state_reasons
                 and "status" in event["raw_object"]
             ):
-                self.log.info("Event: %s Pending, annotations: %s", pod_name, 
annotations_string)

Review Comment:
   I forgot to add an else statement to the for loop. When there are no fatal 
reasons, the code path should be to print out the pod pending log.



-- 
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]

Reply via email to