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
[dev.icinga.com #11419] Config parser should not log names of included files by default #4046
Comments
Updated by mfriedrich on 2016-03-21 08:04:21 +00:00
|
Updated by mfriedrich on 2016-03-21 08:10:56 +00:00
Hi Tom, we already have an issue for that :) Kind regards, |
Updated by tgelf on 2016-03-21 08:18:51 +00:00
Hi Michael, I'm perfectly aware of #10717, but please re-evaluate this issue. It targets another problem, I'm talking about systems running with hundreds or even thousands valid and non-expired comments or downtimes. Erroneous ones like those in #10717 make it worse, but this problem wouldn't change once that is fixed. Thanks, |
Updated by mfriedrich on 2016-03-21 19:44:12 +00:00
|
Updated by mfriedrich on 2016-03-21 19:46:28 +00:00
Discussed it offline. The problem is with a lot of comment/downtime objects put into separate files, causing a long list of 'compiling file' entries. Though that's the default output of the config compiler - it might be a discussion point to filter that away somehow for the API packages startup.log file. |
Updated by tgelf on 2016-03-21 20:00:22 +00:00 Hi Michi, if it was only for the API I wouldn't care. I could perfectly filter away that noise before storing to the Director DB. But such massive log outputs might influence a lot of other components:
We probably do not know where all this data is going to finish. To me that's a good reason to generate as less noise as possible. Several thousand lines for a reload seems a little bit too much to me. This way the related (startup) log data written to disk is larger than the real comment/downtime payload. Cheers, |
Updated by mfriedrich on 2016-07-05 13:38:35 +00:00
One thing which feels much more comfortable - hide loading libraries and including each file into the "notice" log level. Adding an additional "loading config files" log entry to let the user know when it starts. With the change in #12098 you'll only have certain warnings for unreadable files, but won't have to visually grep between "Information" and "Warning".
|
Updated by mfriedrich on 2016-07-05 13:45:10 +00:00
Applied in changeset 915ebe1. |
Updated by tgelf on 2016-07-05 14:03:16 +00:00 @dnsmichi: I know that you are used to modify subjects to obtain a good-looking changelog, allowing you to benefit from the "Include in Changelog" flag. However, please consider opening a dedicated ticket for that purpose when modifying the subject of a user-generated issue to something completely different than initially requested. When someone doesn't get what he wants that's something he has to live with. But when history overview suggests that he has been in some way responsible for a change he didn't ask for, he could remain ... well ... not amused. |
Updated by mfriedrich on 2016-08-08 11:21:00 +00:00
|
Updated by gbeutner on 2016-08-22 11:48:32 +00:00
|
This issue has been migrated from Redmine: https://dev.icinga.com/issues/11419
Created by tgelf on 2016-03-19 22:28:23 +00:00
Assignee: mfriedrich
Status: Resolved (closed on 2016-07-05 13:45:10 +00:00)
Target Version: 2.5.0
Last Update: 2016-08-22 11:48:32 +00:00 (in Redmine)
System description:
Problem: "information/ConfigCompiler: Compiling config file: ..." lines make up more than half a megabyte of data at each restart. And I didn't even add those nasty "warning/ConfigObject: Ignoring config object ... due to errors" lines to that number.
We should either find a more efficient way to store that kind of data (read: one single file for all of them) or get rid of those log lines.
Cheers,
Thomas
Changesets
2016-07-05 13:43:48 +00:00 by mfriedrich 915ebe1
The text was updated successfully, but these errors were encountered: