-
Notifications
You must be signed in to change notification settings - Fork 265
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
Clean up logging #40
Comments
@mreiferson - we are looking to implement this (high-prio) feature for stackato and send a pull request. i see that this issue is already assigned to you. do you plan to work on your own, or should we collaborate? let me know also if you want me to work on it instead. specifically, i want to have log levels (debug, info, warn, error) that can be overridden with command line arguments or signals. could possibly build on top of what i already have. |
+1 on signals - it's what we use in Skynet for changing log levels. |
Yea, I did start working on this, I'll push my code up. I like the idea of using signals, too, I'll add that. I am interested in some feedback as to what is useful information to log in the default (info) case. |
that's excellent. i'm looking forward to your changes.
|
sounds reasonable, I'll take a pass and we can discuss over code :) |
Logging is too verbose. Fix logging to make it consistent, possibly add log levels for more verbosity when necessary.
Option: Change log level of running doozerd instances with signals.
The text was updated successfully, but these errors were encountered: