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

Suggestion: Add HEALTHCHECK? #93

Closed
fdcastel opened this issue Nov 7, 2017 · 1 comment
Closed

Suggestion: Add HEALTHCHECK? #93

fdcastel opened this issue Nov 7, 2017 · 1 comment

Comments

@fdcastel
Copy link

fdcastel commented Nov 7, 2017

There are any plans to implement HEALTHCHECK in this image?

@tianon
Copy link
Member

tianon commented Nov 8, 2017

Not at present; see docker-library/cassandra#76 (comment)

I do not feel that generalized healthchecks on the official images are really that useful.

  • users will have their own idea of what is "healthy"
  • it does not actually test that the service is listening to connections outside of localhost (see https://github.com/docker-library/healthcheck for some examples that do more than what's proposed here, including attempting to check whether the service is listening remotely)
    • some of the Official Images even purposely start in a localhost only mode for database initialization and then kill and start the main service with full network availability
  • after upgrading their images, current users will have extra unexpected load on their systems for healthchecks they don't necessarily need/want and may be unaware of

@tianon tianon closed this as completed Apr 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants