-
Notifications
You must be signed in to change notification settings - Fork 184
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
livenessProbe and readinessProbe for Console #73
Comments
Yea, I would think that all liveness and readiness probes should be configured out-of-the-box, as they are responsible for determining application health/readiness. That's not quite something an end user should have to know about as the app needs to expose those itself. The block of code referenced is actually visible in all deployments, not just Console. |
@amalucelli thanks for reporting. Fixed see above PR. |
So the above PR only added probes to one deployment, the Console. Other charts here, such as the Enforcers, still don't seem to have any probes... Should a separate issue be opened to address those...? The PR also implemented identical liveness and readiness probes against the |
Hi, I see this block code where I can input some probe for
liveness
andreadiness
, but I can't find any default value that I can use to determine if the console is healty or ready.Is possible to know if the console implements any of those? I tried the knowledge base but couldn't find anything useful.
aqua-helm/server/templates/web-deployment.yaml
Lines 175 to 182 in 07321c6
The text was updated successfully, but these errors were encountered: