-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Prevent disabling existing loggers. #902
Prevent disabling existing loggers. #902
Conversation
Why is it a problem if the configuration is read again? Are you configuring additional loggers via code after gunicorn starts up? Why are they not simply configured again by your code in the new worker? Is it a problem related to interacting with |
To clarify here is some piece of my development.ini file:
I just use accesslog and errorlog names to enable their messages in log. Then I'm running If I'm using I mean only accesslog and errorlog messages from gunicorn's loggers. |
@benoitc we have a timeline for next release? I'll try to prioritize the issues I can make time for. |
@arthur-a are you using the latest gunicorn? Gunicorn used to require --log-config with paster but now it should use the .ini file automatically. I will try to test your issue with HUP. You can also add sections for There may be a problem here, and things are too surprising. But the right fix might be somewhere else... |
I think I believe that it should not be necessary to add |
Today I'm pulling latest version from master branch and nothing's changed. |
With and without |
I'm actually about to address this in a new PR. |
Rather than using fileConfig twice and risking different defaults and behavior around disable_existing_loggers (ref #902), simply set the default logging config file to be the paster config file if it has a logger section and let glogging set up the rest.
Okay. See #941 for a new PR. |
Rather than using fileConfig twice and risking different defaults and behavior around disable_existing_loggers (ref #902), simply set the default logging config file to be the paster config file if it has a logger section and let glogging set up the rest.
In pasterapp, when we are sending HUP signal to master process gunicorn's existing loggers will be disabled.