You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@pulsar.apache.org by GitBox <gi...@apache.org> on 2022/01/31 17:19:51 UTC

[GitHub] [pulsar-helm-chart] lhotari commented on pull request #220: Increase Zookeeper probe timeouts

lhotari commented on pull request #220:
URL: https://github.com/apache/pulsar-helm-chart/pull/220#issuecomment-1026018475


   > > 5 seconds seems to be too short a probe timeout on a system with low resources, such as in CI
   > 
   > Were you seeing CI errors because of the low value? What are the consequences of changing this default? It seems to me that it'd let an unresponsive ZK receive traffic for 25 seconds longer, but zk might have some other mechanism that intercedes first. If it does increase the time to recovery, I am not sure that we should change the default for the chart. Can we update a CI `values.yaml` instead?
   
   There hasn't been a timeout for k8s <1.20 before #214 . It's better to set a safe default than have a very tight timeout.


-- 
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: dev-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org