mrk-andreev commented on code in PR #43853:
URL: https://github.com/apache/airflow/pull/43853#discussion_r1843027989
##########
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:
For init containers, we only need to wait until the init container starts.
After that, we can begin fetching logs. There may be cases where the first
container successfully starts and returns logs, while the next one has not
started yet. In such cases, we can start fetching logs as soon as possible.
I’m not sure why we have await_pod_start, but for init containers, this
logic isn't necessary. That’s why we have a separate flow.
--
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]
