[checkmk-commits] Check_MK Git: check_mk: FIX: Changed logging output " Time to process request" to be debug output

Module: check_mk Branch: master Commit: 5a69c5b644b26634c265b847ca11f5f9e930fcb3 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=5a69c5b644b266… Author: Lars Michelsen <lm(a)mathias-kettner.de> Date: Fri Jan 4 08:57:41 2013 +0100 FIX: Changed logging output "Time to process request" to be debug output --- .bugs/867 | 7 +++++-- .bugs/870 | 7 +++++-- ChangeLog | 3 +++ livestatus/src/Store.cc | 3 ++- 4 files changed, 15 insertions(+), 5 deletions(-) diff --git a/.bugs/867 b/.bugs/867 index 0f6210a..a965154 100644 --- a/.bugs/867 +++ b/.bugs/867 @@ -1,8 +1,11 @@ Title: All rules get the dict key "docu_url" - is it really wanted? Component: wato -State: open +Class: cleanup +State: done Date: 2012-10-25 15:45:59 Targetversion: future -Class: cleanup Since the comment and docu url have been implemented, the rules always get the dict added with an empty 'docu_url' key/value. + +2013-01-04 08:52:03: changed state open -> done +Has already been fixed diff --git a/.bugs/870 b/.bugs/870 index 89b9435..b1b8dd4 100644 --- a/.bugs/870 +++ b/.bugs/870 @@ -1,12 +1,15 @@ Title: livestatus.log is spammed with "Time to process request" Component: livestatus -State: open +Class: cleanup +State: done Date: 2012-10-26 12:31:04 Targetversion: future -Class: cleanup Seems on each livestatus request a log entry "2012-10-26 12:25:49 Time to process request" is sent to the livestatus logfile. This entry is not very informative, why not simply drop it? + +2013-01-04 08:56:47: changed state open -> done +Seems to be debug outout. Added if (g_debug_level > 0). diff --git a/ChangeLog b/ChangeLog index d7115e4..d0f941d 100644 --- a/ChangeLog +++ b/ChangeLog @@ -103,6 +103,9 @@ * FIX: remove event pipe on program shutdown, prevents syslog freeze * FIX: hostnames in livestatus query now being utf8 encoded + Livestatus: + * FIX: Changed logging output "Time to process request" to be debug output + 1.2.1i3: Core: * added HOST/SERVICEPROBLEMID to notification macros diff --git a/livestatus/src/Store.cc b/livestatus/src/Store.cc index 269059a..fee45ee 100644 --- a/livestatus/src/Store.cc +++ b/livestatus/src/Store.cc @@ -156,7 +156,8 @@ void Store::answerCommandRequest(const char *command) void Store::answerGetRequest(InputBuffer *input, OutputBuffer *output, const char *tablename) { output->reset(); - logger(LG_INFO, "Time to process request"); + if (g_debug_level > 0) + logger(LG_INFO, "Time to process request"); if (!tablename[0]) { output->setError(RESPONSE_CODE_INVALID_REQUEST, "Invalid GET request, missing tablename"); }