-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
add /health_check for load balancers #1857
Comments
I'm not quite sure what this means - is it related to tmpnb or jupyterhub, perhaps? |
Thanks! we have our own docker spawner which is hard to integrate with tmpnb. So I run notebook directly and need a /health_check. |
I'm still not sure what you're after. What information would be in the response of |
Are you interest in a /health_check REST endpoint that a load balancer can hit to get the status of the notebook server? |
I +1 a I personally like what Elasticsearch does: https://www.elastic.co/guide/en/elasticsearch/reference/current/cluster-health.html - but in general a return code of 200 would be good enough in most cases. |
would be really useful |
Do we have any updates on this? It would be really useful to have a |
Any update regarding this issue? |
For a quick-and-dirty solution, the |
I've tried that with the latest version and I get 404. Having a plain, dumb URL that always returns code 200 would be really useful. |
You can hit cf. notebook/notebook/base/handlers.py Lines 851 to 855 in 6fe9755
|
That does it. Folks, we have a solution. A good endpoint to check the health of a Jupyter Notebook instance is simply
|
Thanks for the tip @afshin - closing. |
No description provided.
The text was updated successfully, but these errors were encountered: