mrk-andreev commented on code in PR #43853:
URL: https://github.com/apache/airflow/pull/43853#discussion_r1844932803
##########
providers/src/airflow/providers/cncf/kubernetes/utils/pod_manager.py:
##########
@@ -565,6 +585,31 @@ def _reconcile_requested_log_containers(
self.log.error("Could not retrieve containers for the pod: %s",
pod_name)
return containers_to_log
+ def fetch_requested_init_container_logs(
Review Comment:
I believe that fetching logs for non-init containers should be done in
parallel, as we might have long-running containers with interactions where
parallel logs could be useful.
However, I suggest extracting this idea into a separate PR. :)
--
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]
