Sinsin1367 commented on a change in pull request #8734:
URL: https://github.com/apache/airflow/pull/8734#discussion_r422181509
##########
File path: airflow/providers/cncf/kubernetes/operators/kubernetes_pod.py
##########
@@ -177,6 +181,7 @@ def __init__(self, # pylint:
disable=too-many-arguments,too-many-locals
full_pod_spec: Optional[k8s.V1Pod] = None,
init_containers: Optional[List[k8s.V1Container]] = None,
log_events_on_failure: bool = False,
+ log_container_statuses_on_failure: bool = False,
Review comment:
I do think these logs are useful info and absolutely no harm in having
them on by default. But I can see why they defaulted to False not to
contaminate logs unless the task has a problem and you will turn it on then.
----------------------------------------------------------------
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:
[email protected]
