Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Should liveness and readiness probes be disabled by default? #392

Open
baizele opened this issue Jun 8, 2023 · 2 comments
Open

Should liveness and readiness probes be disabled by default? #392

baizele opened this issue Jun 8, 2023 · 2 comments

Comments

@baizele
Copy link

baizele commented Jun 8, 2023

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?

@antares-sw
Copy link
Contributor

@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.

@ebaizel
Copy link

ebaizel commented Jul 17, 2023

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?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants