baryluk commented on a change in pull request #17649:
URL: https://github.com/apache/airflow/pull/17649#discussion_r690765281
##########
File path: airflow/providers/cncf/kubernetes/utils/pod_launcher.py
##########
@@ -217,7 +223,7 @@ 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)
+ @tenacity.retry(stop=tenacity.stop_after_attempt(4),
wait=tenacity.wait_exponential(), reraise=True)
Review comment:
I can revert that part for now. It does help a little, but with the
retry that will be now attempted if needed in the outer loop (as long as the
pod is alive) it indeed seems not necessary.
--
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]
