-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
[Ingest Manager] Logging level defaults to info #18657
[Ingest Manager] Logging level defaults to info #18657
Conversation
Pinging @elastic/ingest-management (Team:Ingest Management) |
💚 Build SucceededExpand to view the summary
Build stats
Test stats 🧪
|
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.
Looks good.
Is there a manual way to set it to debug? @ph I think we need in ingest-manager a way to change this config option |
it's configurable at start we would need to create some kind of wrapper over logger which will reinstantiate it to enable log level change, as for now loggers we use does not allow on the fly level change. |
[Ingest Manager] Logging level defaults to info (elastic#18657)
[Ingest Manager] Logging level defaults to info (elastic#18657)
This reverts commit 74476a2.
What does this PR do?
Changes default log level to info
Why is it important?
To save CPU cycles with default configuration. when using default beats are a bit chatty which makes monitoring filebeat resource heavy on CPU
Checklist
CHANGELOG.next.asciidoc
orCHANGELOG-developer.next.asciidoc
.Fixes: #18351