Graylog2 / graylog-docker

Official Graylog Docker image
https://hub.docker.com/r/graylog/graylog/
Apache License 2.0
372 stars 134 forks source link

multiple fixes for health check and added feature #107

Closed jalogisch closed 4 years ago

jalogisch commented 4 years ago

This added the feature to auto set master in a nomad cluster - similar to the kubernetes one - this will also be documented. CLOSE: https://github.com/Graylog2/graylog-docker/issues/103

It will fix two bugs with the docker health check. The first was introduced because the protocol was not removed when the http_publish_uri was set in a configuration file, but if set via environment variable it was cleaned. At last the protocol was added again and if it was set via configuration the result was a double protocol FIX: https://github.com/Graylog2/graylog-docker/issues/102

According to the issue ( https://github.com/Graylog2/graylog-docker/issues/101 ) when the container runs in docker swarm it runs into the problem that the http_publish_uri might not be reachable for the container. What should not happen, but could. To fix this I introduced a second that if the first fails to localhost what can be seen as a fix for many similar situation. If that also fails the healtch_check will fail anyway. FIX: https://github.com/Graylog2/graylog-docker/issues/101