helm-charts icon indicating copy to clipboard operation
helm-charts copied to clipboard

Should liveness and readiness probes be disabled by default?

Open baizele opened this issue 1 year ago • 2 comments

Noting this comment, should the probes be disabled by default? I ran into an issue where my CL client couldn't connect to the EL client because EL wasn't discoverable due to its readiness probe failing.

In what scenarios is it preferable to have this enabled? Would that be after the client has fully synced?

baizele avatar Jun 08 '23 17:06 baizele

@baizele At the first deployment of the chart, it is recommended to disable the probes so that the pods do not restart, after the clients are fully synced, they can be turned back on to perform checks.

antares-sw avatar Jul 15 '23 06:07 antares-sw

Thanks @antares-sw. Would it make sense then to have the default value be disabled since most apps will need this disabled as they do their initial sync?

ebaizel avatar Jul 17 '23 13:07 ebaizel