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

[BUG] Trace logging is on by default #191

Closed
jdmarshall opened this issue Jan 28, 2022 · 5 comments
Closed

[BUG] Trace logging is on by default #191

jdmarshall opened this issue Jan 28, 2022 · 5 comments
Labels

Comments

@jdmarshall
Copy link

jdmarshall commented Jan 28, 2022

Please open bug only for Docker component here, if you want to do so for Graphite, please use graphite-web repo

Describe the bug
Nobody needs to see every time a client connects to graphite is opened and closed. This is trace-level information and should be off by default

To Reproduce
Run docker-graphite-statsd, and watch the screen scroll and scroll.

Expected behavior
Error logging to console, and not much else.

Environment (please complete the following information):
Straight out of the box.

Additional context
I'm trying to run this in docker-compose, and I can't read any logs from my apps because graphite is scrolling the window every fifteen seconds:

28/01/2022 01:22:55 :: [listener] MetricLineReceiver connection with 127.0.0.1:46950 established

@jdmarshall jdmarshall added the bug label Jan 28, 2022
@deniszh
Copy link
Member

deniszh commented Jan 28, 2022

Hi @jdmarshall ,
I'll test if LOG_LISTENER_CONN_SUCCESS=False fixing issue, will change default then in next release.

@jdmarshall
Copy link
Author

It did not for me. I think the conversation elsewhere is that the problem is in the config file, and that the python code is not evaluating environment variables during initialization.

@deniszh
Copy link
Member

deniszh commented Jan 28, 2022 via email

@deniszh
Copy link
Member

deniszh commented Feb 6, 2022

Fixed in 1.1.8-6, image will be ready in couple of hours (as automation build it).

@deniszh deniszh closed this as completed Feb 6, 2022
@deniszh
Copy link
Member

deniszh commented Feb 6, 2022

Correction - in 1.1.8-7

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants