-
-
Notifications
You must be signed in to change notification settings - Fork 855
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 healthcheck #2047
base: master
Are you sure you want to change the base?
Add healthcheck #2047
Conversation
Add healthcheck feature on Docker image.
hi, did something happen? can you paste the logs of how the application crashed initially? |
No crash, it's only to monitore the docker container with the docker's healthcheck feature. |
ok, is it possible you can use curl instead? many OS's/dists/docker bases already have curl, so it might be nicer than wget |
tries=1 is also far too low |
Done.
Currently not in standalone with your debian version. I use apt for it.
It's a wget feature. But the retry is on the healthcheck feature: https://docs.docker.com/engine/reference/builder/#healthcheck 3 retry by default. |
This just tripped me when trying to implement the usual health check, as neither @dgtlmoon I see you force pushed to the master branch, however, the changes have never made it there. |
This PR add wget in image to allow a healthcheck feature on Docker.