potiuk commented on code in PR #33107:
URL: https://github.com/apache/airflow/pull/33107#discussion_r1285231392
##########
chart/values.yaml:
##########
@@ -721,6 +721,14 @@ scheduler:
failureThreshold: 5
periodSeconds: 60
command: ~
+
+ # Wait for at most 10 minutes (60*10s) for the scheduler container to
startup.
+ # livenessProbe kicks in after the startup
+ startupProbe:
+ failureThreshold: 60
Review Comment:
I thin the defaults are pretty long. If there is an error, the user in
default configuraiton will have to waut for 10 minutes before they find out
something is wrong.
Minute as a default should be quite enough IMHO
--
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]
