Graylog2 / graylog-docker

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

Ensure bash passes SIGTERM to java. #174

Closed malcyon closed 3 years ago

malcyon commented 3 years ago

This ensures that Graylog shuts down properly when docker stop is run.

Fixes #173

malcyon commented 3 years ago

I have validated that after this fix, the container logs show this after shutdown:

2021-08-19 17:14:22,356 INFO : org.graylog2.system.shutdown.GracefulShutdown - Goodbye.
2021-08-19 17:14:22,361 INFO : org.graylog2.shared.messageq.localkafka.LocalKafkaMessageQueueReader - Stopping.
2021-08-19 17:14:22,360 INFO : org.graylog2.shared.initializers.JerseyService - Shutting down HTTP listener at <0.0.0.0:9000>
2021-08-19 17:14:22,366 INFO : org.graylog.shaded.kafka09.log.LogManager - Shutting down.
2021-08-19 17:14:22,371 INFO : org.graylog2.lookup.LookupDataAdapterRefreshService - Stopping 0 jobs
2021-08-19 17:14:22,392 INFO : org.glassfish.grizzly.http.server.NetworkListener - Stopped listener bound to [0.0.0.0:9000]
2021-08-19 17:14:22,398 INFO : org.graylog.shaded.kafka09.log.LogManager - Shutdown complete.
2021-08-19 17:14:22,399 INFO : org.graylog2.shared.initializers.ServiceManagerListener - Services are now stopped.

I tested this using docker-compose. When I tested using the graylog/graylog:4.1 image, stopping the container did not produce these messages.