graphite-project / carbon

Carbon is one of the components of Graphite, and is responsible for receiving metrics over the network and writing them down to disk using a storage backend.
http://graphite.readthedocs.org/
Apache License 2.0
1.5k stars 490 forks source link

add config option to turn off logging of lost connections #900

Closed tbenz9 closed 3 years ago

tbenz9 commented 4 years ago

This patch adds a config option which can be added for the cache/relay/aggregator instances to not log lost connections anymore. For this the config option LOG_LISTENER_CONN_LOST has to be set to False

Reason: Reduce the amount of logging from lost connections.

I tried to copy this MR as much as possible: https://github.com/graphite-project/carbon/pull/60

deniszh commented 3 years ago

Thanks, but commented out values in config example still says "True" which can be misleading. Could you please change them too? Thanks!

tbenz9 commented 3 years ago

Thanks, but commented out values in config example still says "True" which can be misleading. Could you please change them too? Thanks!

Done, thanks for your patience!

deniszh commented 3 years ago

No worries, LGTM. Merging.