ashb commented on a change in pull request #7428: [AIRFLOW-4526] Poll k8s logs
to avoid problem with kubernetes logs --follow
URL: https://github.com/apache/airflow/pull/7428#discussion_r383248130
##########
File path: airflow/kubernetes/pod_launcher.py
##########
@@ -162,22 +166,95 @@ def base_container_is_running(self, pod: V1Pod):
return False
return status.state.running is not None
+ @tenacity.retry(
+ stop=tenacity.stop_after_attempt(3),
+ wait=tenacity.wait_exponential(),
+ reraise=True,
+ )
+ def _read_pod_log_chunk(self, pod: V1Pod, last_line: bytes) ->
HTTPResponse:
+ # The CoreV1Api doesn't support since_time even though the API does,
so we must use
Review comment:
Has anyone reported this to the upstream project? We should and put a link
to that issue here so we can remove this hack when they eventually fix it.
----------------------------------------------------------------
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]
With regards,
Apache Git Services
