-
Notifications
You must be signed in to change notification settings - Fork 575
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
Add support for log rotation #527
Add support for log rotation #527
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
First, thank you so much for your pull request!
Almost OK to me, but let me add minor comment as following:
- Currently multus keeps
vendor
directory to build multus without go modules. Could you dogo mod vendor
to update vendor package? - Could you please change
loggingFp
inlogging/logging_test.go
file?
7404bdf
to
3cd0274
Compare
Thank you for your comments - I fixed and pushed |
Pull Request Test Coverage Report for Build 137706987
💛 - Coveralls |
logging/logging.go
Outdated
MaxAge: 5, // days | ||
Compress: true, | ||
} | ||
if w == nil { | ||
fmt.Fprintf(os.Stderr, "multus logging: cannot open %s", filename) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@shahar-klein why would w
be nil here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the comment - fixed.
3cd0274
to
5e87ea8
Compare
When using debug level the log files tends to fill up quickly. Add support for log rotation using lumberjack, simple and easy to use log rotation module. Signed-off-by: Shahar Klein <sklein@nvidia.com>
5e87ea8
to
6ea65a2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
/lgtm as well |
LGTM, too! thank you for your help! Merged. |
When using debug level the log files tends to fill up quickly.
Add support for log rotation using lumberjack, simple and easy to use
log rotation module.
Signed-off-by: Shahar Klein sklein@nvidia.com