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

Local health check for helm #122

Open
TJM opened this issue Feb 8, 2022 · 4 comments
Open

Local health check for helm #122

TJM opened this issue Feb 8, 2022 · 4 comments

Comments

@TJM
Copy link

TJM commented Feb 8, 2022

Related to #121 - I have been using some terraform code to deploy consul-esm using a kubernetes_deployment to our cluster. It is "generally" working, but when I was first setting it up, I realized that we need some "health checking" for consul-esm itself. I would equate this to a "liveness-probe" and/or "readiness-probe" ... i.e. how can I check to make sure that consul-esm itself is working.

@TJM
Copy link
Author

TJM commented Mar 2, 2022

IS there some sort of internal command or http request or "file" we can inspect to see that consul-esm is working from inside the container?

@tristanmorgan
Copy link
Member

Consul-esm does have a liveliness indicator in Consul's service discovery.
image

@TJM
Copy link
Author

TJM commented Aug 29, 2022

Consul-esm does have a liveliness indicator in Consul's service discovery.

I don't know if liveness/readiness probes will connect to other containers, but then we have to deal with authentication. I was hoping for something simple and local to the consul-esm container. However, I do think that what you have provided will suffice for the "what happens if consul-esm dies" as that should turn red/unavailable. Then, of course it is something else's responsibility to monitor that and alert as necessary ;). I was looking for something that would end up in the helm chart (deployment) as a local health check. Perhaps a "defacto" /healthz HTTP GET? :-/

@eikenb
Copy link
Contributor

eikenb commented Nov 7, 2022

Converting this to a feature request and updating the title to match.

@eikenb eikenb changed the title Who watches the watcher? Local health check for helm Nov 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants