Checkmk git commits August 2013

checkmk-commits@lists.checkmk.com

6 participants
167 discussions

Check_MK Git: check_mk: df.include: always return 3 values

by Mathias Kettner

Module: check_mk Branch: master Commit: 1ef8b7966d1f0c41d41f925a4049babeb6c7aed5 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=1ef8b7966d1f0c… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Wed Aug 28 12:33:40 2013 +0200 df.include: always return 3 values --- checks/df.include | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/checks/df.include b/checks/df.include index 8928194..28d67a2 100644 --- a/checks/df.include +++ b/checks/df.include @@ -238,7 +238,7 @@ def df_check_filesystem_list(item, params, fslist): def df_check_filesystem_single(hostname, mountpoint, size_mb, avail_mb, params): if size_mb == 0: - return (1, "size of filesystem is 0 MB") + return (1, "size of filesystem is 0 MB", []) used_mb = size_mb - avail_mb used_perc = 100.0 * (float(used_mb) / size_mb)

10 years, 10 months

Check_MK Git: check_mk: Updated bug entries #1036, #1037, #1038, #1039

by Mathias Kettner

Module: check_mk Branch: master Commit: ea77cc4e5995a6993da126bb155a668270be761b URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=ea77cc4e5995a6… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Wed Aug 28 12:37:04 2013 +0200 Updated bug entries #1036, #1037, #1038, #1039 --- .bugs/1036 | 13 +++++-------- .bugs/1037 | 25 ++++++++----------------- .bugs/1038 | 52 +++++++++++++++++++++------------------------------- .bugs/1039 | 43 ++++++++++++------------------------------- 4 files changed, 46 insertions(+), 87 deletions(-) diff --git a/.bugs/1036 b/.bugs/1036 index 43329c7..7d30140 100644 --- a/.bugs/1036 +++ b/.bugs/1036 @@ -1,9 +1,9 @@ Title: df_check_filesystem_single returns a 2-tuple when check_esx_vsphere_datastores expects 3 Component: checks -State: open +Class: bug +State: done Date: 2013-08-27 10:50:42 Targetversion: 1.2.3 -Class: bug came in via feedback@ // mail is still available at feedback/gitbug folder @@ -26,12 +26,9 @@ def df_check_filesystem_single(hostname, mountpoint, size_mb, avail_mb, params): def check_esx_vsphere_datastores(item, params, info): -… - -… - -… state, infotext, perfdata = df_check_filesystem_single( - g_hostname, item, size_mb, avail_mb, params) + +2013-08-28 12:33:57: changed state open -> done +Fixed. diff --git a/.bugs/1037 b/.bugs/1037 index ff1bce5..4c9999f 100644 --- a/.bugs/1037 +++ b/.bugs/1037 @@ -3,27 +3,18 @@ Component: checks Class: bug State: open Date: 2013-08-27 10:58:29 -Targetversion: 1.2.3 - -came in via feedback@ // original mail still available in feedback/gitbug folder - - -Date: Fri, 07 Jun 2013 08:56:13 -0400 -From: "Peter Rogers" <peterrogers(a)email.com> -Message-ID: <20130607125613.34790(a)gmx.com> -MIME-Version: 1.0 -Subject: check_mk_agent.exe unable to restart after a crash -To: feedback(a)check-mk.org - +Targetversion: 1.2.3i2 Host = Windows 7 CMK Agent = Agent version 1.2.2p1 -I have found a problem where the check_mk_agent.exe is unable to restart after a crash because the logstate.txt file becomes currupt. +I have found a problem where the check_mk_agent.exe is unable to restart +after a crash because the logstate.txt file becomes currupt. Attached is an example currupt logstate.txt file. -To work around the problem I remove the logstate.txt and then start the check_mk_agent.exe successfully. +To work around the problem I remove the logstate.txt and then start the +check_mk_agent.exe successfully. hint by bs: the attached logstate.txt is 0 bytes @@ -46,7 +37,7 @@ If I edit the file in Notepad++ I see one line of null entries, and converted to 00000000000000000000000000000000 0000000000000000000000 -I have had a number of crashes on this server as the Windows machine is running on a VM server which has had problems, but the currupt logstate.txt file is always 123k in size and looks the same when edited. +I have had a number of crashes on this server as the Windows machine is +running on a VM server which has had problems, but the currupt logstate.txt +file is always 123k in size and looks the same when edited. -Regards, -Peter Rogers diff --git a/.bugs/1038 b/.bugs/1038 index 99e9d80..34f2057 100644 --- a/.bugs/1038 +++ b/.bugs/1038 @@ -2,24 +2,12 @@ Title: problem with mrpe if lable has white space characters Component: checks State: open Date: 2013-08-27 12:12:33 -Targetversion: 1.2.3 +Targetversion: future Class: bug -came in via feedback@ // original mail still available in feedback/gitbug folder - - -Date: Wed, 26 Jun 2013 15:03:06 +0200 -Message-ID: <CACA1nGEzOmECJufVGOgUqZm6tAxGH+vHoXoEh7SGbx7uw+3ZwQ(a)mail.gmail.com> -Subject: Fwd: [BUG]: MRPE Checks mit Performance Daten bringen Fehler -From: =?ISO-8859-1?Q?Andreas_D=F6hler?= <andreas.doehler(a)gmail.com> -To: Check MK Feedback <feedback(a)check-mk.org> - - - -Hallo zusammen, - -habe heute eine seltsame Sache mit einigen MRPE Checks gehabt. -Falls die Checks Performance Daten liefern und das Label der Performance Daten Leerzeichen enthält so wir der Check mit einer Exception abgebrochen. +habe heute eine seltsame Sache mit einigen MRPE Checks gehabt. Falls die +Checks Performance Daten liefern und das Label der Performance Daten +Leerzeichen enthält so wir der Check mit einer Exception abgebrochen. Traceback (most recent call last): File "/usr/share/check_mk/modules/check_mk.py", line 5442, in <module> @@ -34,23 +22,25 @@ Traceback (most recent call last): varname, valuetxt = perfinfo.split("=", 1) ValueError: need more than 1 value to unpack -Bei genauerer Untersuchung hab ich festgestellt das dieses Verhalten seit dem Umbau des MRPE am 22.05. (Commit 7705017d0d5db5093d9bdaa1538eecac56832dfe) auftritt. -Das Problem besteht aus zwei Punkten. -Einmal diese Zeile -perfdata = parts[1].strip().split() -Hierbei werden nicht nur vorn und hinten die Leerzeichen abgeschnitten sondern auch der String vom Performancedaten Label zerschnitten. -Beispiel: -'Cluster Groups on This Node'=0;0;0 Perfdaten noch ok -["'Cluster", 'Groups', 'on', 'This', "Node'=0;0;0"] nach der "Behandlung" +Bei genauerer Untersuchung hab ich festgestellt das dieses Verhalten seit dem +Umbau des MRPE am 22.05. (Commit 7705017d0d5db5093d9bdaa1538eecac56832dfe) +auftritt. Das Problem besteht aus zwei Punkten. Einmal diese Zeile perfdata += parts[1].strip().split() Hierbei werden nicht nur vorn und hinten die +Leerzeichen abgeschnitten sondern auch der String vom Performancedaten Label +zerschnitten. Beispiel: 'Cluster Groups on This Node'=0;0;0 Perfdaten noch ok +["'Cluster", 'Groups', 'on', 'This', "Node'=0;0;0"] nach der "Behandlung" Der zweite Punkt tritt dann auf wenn ich den String hier im ganzen lasse. -'Cluster Groups on This Node'=0;0;0 -und es nur einen Perfomance Daten Wert gibt dann wird perfdata nicht zur Liste und bringt einen Fehler dann beim Aufruf der map Funktion in Zeile 90 -Deshalb hab ich extra noch vor der ersten perfdata Zuweisung in Zeile 70 eine Zeile eingefügt mit perfdata = [] und Zeile 70 auf perfdata += parts[1].strip() geändert. -Danach funktionieren wieder alle meine MRPE Checks mit Performance Daten ohne Fehler. - -Einen MRPE Check mit mehreren Werten hatte ich leider nicht an der Hand zum testen. -Damit ist der Abschnitt ab # Further lines noch offen ob das alles so klappt. :) +'Cluster Groups on This Node'=0;0;0 und es nur einen Perfomance Daten Wert +gibt dann wird perfdata nicht zur Liste und bringt einen Fehler dann beim +Aufruf der map Funktion in Zeile 90 Deshalb hab ich extra noch vor der ersten +perfdata Zuweisung in Zeile 70 eine Zeile eingefügt mit perfdata = [] und +Zeile 70 auf perfdata += parts[1].strip() geändert. Danach funktionieren +wieder alle meine MRPE Checks mit Performance Daten ohne Fehler. + +Einen MRPE Check mit mehreren Werten hatte ich leider nicht an der Hand +zum testen. Damit ist der Abschnitt ab # Further lines noch offen ob das +alles so klappt. :) Gruß Andreas diff --git a/.bugs/1039 b/.bugs/1039 index da19013..b8da79a 100644 --- a/.bugs/1039 +++ b/.bugs/1039 @@ -2,24 +2,17 @@ Title: livestatus bug in OutputFormat: json Component: livestatus State: open Date: 2013-08-27 16:02:02 -Targetversion: 1.2.3 +Targetversion: 1.2.3i2 Class: bug -came in via feedback@ // mail still available in feedback/gitbug folder - - -From: <Sascha.Pfalz(a)ivz.de> -To: <feedback(a)check-mk.org> -Subject: [Bug]: JSON Data falsch -Thread-Topic: [Bug]: JSON Data falsch -Thread-Index: Ac6IZny063o7fFKiSS2eLhEqKFzKaA== -Date: Wed, 24 Jul 2013 12:08:20 +0000 - Hi, -Erst mal möchte ich danke sagen für das super Plugin, das hat mir viel Arbeit erspart und macht den Einsatz von unseren Nagios und SMBox Systemen etwas leichter +Erst mal möchte ich danke sagen für das super Plugin, das hat mir viel +Arbeit erspart und macht den Einsatz von unseren Nagios und SMBox Systemen +etwas leichter -Allerdings scheint sich ein Bug in die V1.2.2p2 eingeschlichen zu haben, und zwar geht es um das Auslesen von Kommentaren bei folgendem Kommando: +Allerdings scheint sich ein Bug in die V1.2.2p2 eingeschlichen zu haben, +und zwar geht es um das Auslesen von Kommentaren bei folgendem Kommando: $CMD = "GET services\n"; $CMD.= "Columns: host_name comments_with_info\n"; @@ -40,7 +33,9 @@ Der Host localhost hat 11 Services definiert, wobei einer einen Kommentar defini ["localhost",[]], ["localhost",[[705,"nagios","Testing",]]]] -Und das ist leider nicht korrekt, das letzte ",]" ist falsch! Diese Daten lassen sich leider nicht via json_decode() in PHP dekodieren. Wandel ich das Ganze aber wie folgt ab: +Und das ist leider nicht korrekt, das letzte ",]" ist falsch! Diese Daten +lassen sich leider nicht via json_decode() in PHP dekodieren. Wandel ich +das Ganze aber wie folgt ab: [["localhost",[]], ["localhost",[]], @@ -59,24 +54,10 @@ Und das ist leider nicht korrekt, das letzte ",]" ist falsch! Diese Daten lassen Dann funktioniert das auch sofort wieder. -Ich habe leider aktuell nicht so viel Zeit, um einen Patch zu liefern, deswegen dieser etwas chaotische Bugreport, ich hoffe aber, das man das soweit nachvollziehen kann. +Ich habe leider aktuell nicht so viel Zeit, um einen Patch zu liefern, +deswegen dieser etwas chaotische Bugreport, ich hoffe aber, das man das +soweit nachvollziehen kann. In der 1.1.10p3 war dieser Bug definitiv nicht drin. Getestet habe ich mit den Nagios Versionen 3.2.1 und 3.5.0, bei beiden tritt diese falsche JSON Kodierung auf. - -Nochmals vielen Dank für das Plugin! - -Viele Grüße, --- -Sascha Pfalz -Oracle Developer -Archiv- und Produktionssysteme - -Tel. +49(0)30 97993-86318 -sascha.pfalz"at"ivz.de - -Informations-Verarbeitungs-Zentrum -Gemeinschaftseinrichtung von -DW - Deutschlandradio - MDR - NDR - RB - RBB - SR - WDR -Masurenallee 8-14 14057 Berlin

10 years, 10 months

Check_MK Git: check_mk: FIX: esx_vsphere_datastore: handle case with 0MB disks correctly

by Mathias Kettner

Module: check_mk Branch: master Commit: 9f08c6f9b6d8ca391131233ff79d44f6cf47b936 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=9f08c6f9b6d8ca… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Wed Aug 28 12:31:57 2013 +0200 FIX: esx_vsphere_datastore: handle case with 0MB disks correctly --- checks/esx_vsphere_datastores | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/checks/esx_vsphere_datastores b/checks/esx_vsphere_datastores index 8dec7cb..f09cd3a 100644 --- a/checks/esx_vsphere_datastores +++ b/checks/esx_vsphere_datastores @@ -64,8 +64,13 @@ def check_esx_vsphere_datastores(item, params, info): size_mb = store["capacity"] / 1024.0 / 1024.0 avail_mb = store["freeSpace"] / 1024.0 / 1024.0 - state, infotext, perfdata = df_check_filesystem_single( - g_hostname, item, size_mb, avail_mb, params) + df_info = df_check_filesystem_single(g_hostname, item, size_mb, avail_mb, params) + state = df_info[0] + infotext = df_info[1] + if len(df_info) >= 3: + perfdata = df_info[2] + else: + perfdata = [] if 'uncommitted' in store: uncommitted_mb = store["uncommitted"] / 1024.0 / 1024.0

10 years, 10 months

Check_MK Git: check_mk: Revert "FIX: esx_vsphere_datastore:

by Mathias Kettner

handle case with 0MB disks correctly" Message-ID: <521dd2d2.CDiTdcvZWbdQuagd%mk(a)mathias-kettner.de> User-Agent: Heirloom mailx 12.4 7/29/08 MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Module: check_mk Branch: master Commit: d5631b6d25312337644c4bf1c237ce224fe8d79f URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=d5631b6d253123… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Wed Aug 28 12:33:19 2013 +0200 Revert "FIX: esx_vsphere_datastore: handle case with 0MB disks correctly" This reverts commit 9f08c6f9b6d8ca391131233ff79d44f6cf47b936. --- checks/esx_vsphere_datastores | 9 ++------- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/checks/esx_vsphere_datastores b/checks/esx_vsphere_datastores index f09cd3a..8dec7cb 100644 --- a/checks/esx_vsphere_datastores +++ b/checks/esx_vsphere_datastores @@ -64,13 +64,8 @@ def check_esx_vsphere_datastores(item, params, info): size_mb = store["capacity"] / 1024.0 / 1024.0 avail_mb = store["freeSpace"] / 1024.0 / 1024.0 - df_info = df_check_filesystem_single(g_hostname, item, size_mb, avail_mb, params) - state = df_info[0] - infotext = df_info[1] - if len(df_info) >= 3: - perfdata = df_info[2] - else: - perfdata = [] + state, infotext, perfdata = df_check_filesystem_single( + g_hostname, item, size_mb, avail_mb, params) if 'uncommitted' in store: uncommitted_mb = store["uncommitted"] / 1024.0 / 1024.0

10 years, 10 months

Check_MK Git: check_mk: Updated bug entries #1027, #1028, #0902, #1032, #0926, #1030, #0829, #0958, #0959, #0960, #0962, #0965, #0966, #0967, #0969, #0970, #0971, #0972, #0973, #0975, #0976, #0977, #0998, #0884, #0830, #0889

by Mathias Kettner

Module: check_mk Branch: master Commit: 661da52ce37eb490d16e018f1845d533e2eeda79 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=661da52ce37eb4… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Wed Aug 28 12:31:49 2013 +0200 Updated bug entries #1027, #1028, #0902, #1032, #0926, #1030, #0829, #0958, #0959, #0960, #0962, #0965, #0966, #0967, #0969, #0970, #0971, #0972, #0973, #0975, #0976, #0977, #0998, #0884, #0830, #0889 --- .bugs/1027 | 32 ++++++++------------------- .bugs/1028 | 70 ++++++++-------------------------------------------------- .bugs/1030 | 18 +++++---------- .bugs/1032 | 19 ++++------------ .bugs/829 | 2 +- .bugs/830 | 2 +- .bugs/884 | 2 +- .bugs/889 | 2 +- .bugs/902 | 2 +- .bugs/926 | 4 +++- .bugs/958 | 7 ++++-- .bugs/959 | 2 +- .bugs/960 | 37 +++++-------------------------- .bugs/962 | 7 ++++-- .bugs/965 | 39 ++++++++------------------------ .bugs/966 | 14 +----------- .bugs/967 | 26 +++++----------------- .bugs/969 | 36 +++++------------------------- .bugs/970 | 22 ++++++------------ .bugs/971 | 18 +-------------- .bugs/972 | 38 +++++-------------------------- .bugs/973 | 20 +---------------- .bugs/975 | 67 +++++++++++++------------------------------------------ .bugs/976 | 61 +++++++++++++++++--------------------------------- .bugs/977 | 2 +- .bugs/998 | 4 +++- .bugs/config | 1 + 27 files changed, 126 insertions(+), 428 deletions(-) Diff: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commitdiff;h=661da52ce3…

10 years, 10 months

Check_MK Git: check_mk: FIX: hr_mem: handle virtual memory correct on some devices

by Mathias Kettner

Module: check_mk Branch: master Commit: 5e41130dc7d6371f603551d5f064fe94dc32b030 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=5e41130dc7d637… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Wed Aug 28 12:25:37 2013 +0200 FIX: hr_mem: handle virtual memory correct on some devices --- ChangeLog | 1 + checks/hr_mem | 4 ++-- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/ChangeLog b/ChangeLog index 4b3701e..d4382f9 100644 --- a/ChangeLog +++ b/ChangeLog @@ -72,6 +72,7 @@ * cisco_wlc_clients: New check for the nummber of clients in a wlc wifi * df: Negative integer levels for MB left on a device * win_printers: Monitoring of printer queue on a windows printserver + * FIX: hr_mem: handle virtual memory correct on some devices Notifications: * notify.py: Matching service level: Use the hosts service level if a diff --git a/checks/hr_mem b/checks/hr_mem index ddfb601..7c37e4c 100644 --- a/checks/hr_mem +++ b/checks/hr_mem @@ -37,7 +37,7 @@ def inventory_hr_mem(info): if saveint(hrsize) > 0: # some device have zero (broken) values return [(None, "memused_default_levels")] -def check_hr_mem(item, params, info): +def check_hr_mem(_no_item, params, info): usage = {} for hrtype, hrdescr, hrunits, hrsize, hrused in info: if hrtype in [ ".1.3.6.1.2.1.25.2.1.2", @@ -47,7 +47,7 @@ def check_hr_mem(item, params, info): # We use only the first entry of each type. We have # seen devices (pfSense), that have lots of additional # entries that are not useful. - if hrtype not in usage: + if hrtype not in usage and hrdescr != "Virtual memory": usage[hrtype] = (size, used) # Account for cached memory (this works at least for systems using

10 years, 10 months

Check_MK Git: check_mk: Added missing help text

by Lars Michelsen

Module: check_mk Branch: master Commit: a34a4e1161a4a22cbd9ed76c675fb92ec414c6c7 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=a34a4e1161a4a2… Author: Lars Michelsen <lm(a)mathias-kettner.de> Date: Wed Aug 28 09:58:01 2013 +0200 Added missing help text --- mkeventd/web/plugins/wato/mkeventd.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/mkeventd/web/plugins/wato/mkeventd.py b/mkeventd/web/plugins/wato/mkeventd.py index e2a8477..8f56414 100644 --- a/mkeventd/web/plugins/wato/mkeventd.py +++ b/mkeventd/web/plugins/wato/mkeventd.py @@ -258,6 +258,9 @@ vs_mkeventd_rule = Dictionary( Checkbox( title = _("Automatic Deletion"), label = _("Delete event immediately after the actions"), + help = _("Incoming messages might trigger actions (when configured above), " + "afterwards only an entry in the event history will be left. There " + "will be no \"open event\" to be handled by the administrators."), ) ), ( "count",

10 years, 10 months

Check_MK Git: check_mk: FIX: Fixed first toggle of flags in global settings when default is set to True

by Lars Michelsen

Module: check_mk Branch: master Commit: 7d81876c26166b7353e2fc66c11406bb30bba846 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=7d81876c26166b… Author: Lars Michelsen <lm(a)mathias-kettner.de> Date: Wed Aug 28 10:08:30 2013 +0200 FIX: Fixed first toggle of flags in global settings when default is set to True Conflicts: web/htdocs/wato.py --- ChangeLog | 1 + web/htdocs/wato.py | 5 +++-- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/ChangeLog b/ChangeLog index 88a058e..4b3701e 100644 --- a/ChangeLog +++ b/ChangeLog @@ -321,6 +321,7 @@ * FIX: automation users can now be deleted again (bug was introduced in 1.2.2p1) * FIX: fix logwatch pattern analyzer message "The host xyz is not managed by WATO." after direct access via snapin + * FIX: Fixed first toggle of flags in global settings when default is set to True Event Console: * FIX: apply rewriting of application/hostname also when cancelling events diff --git a/web/htdocs/wato.py b/web/htdocs/wato.py index 1cfdd54..fcb7b90 100644 --- a/web/htdocs/wato.py +++ b/web/htdocs/wato.py @@ -5621,7 +5621,7 @@ def mode_globalvars(phase): action = html.var("_action") if varname: domain, valuespec, need_restart, allow_reset, in_global_settings = g_configvars[varname] - def_value = default_values.get(varname, valuespec.canonical_value()) + def_value = default_values.get(varname, valuespec.default_value()) if action == "reset" and not isinstance(valuespec, Checkbox): c = wato_confirm( @@ -5631,8 +5631,9 @@ def mode_globalvars(phase): (varname, valuespec.value_to_text(def_value))) else: if not html.check_transaction(): - return + return c = True # no confirmation for direct toggle + if c: # if action == "reset": # del current_settings[varname]

10 years, 10 months

Check_MK Git: check_mk: Updated bug entries #1037, #1039

by Bernd Stroessenreuther

Module: check_mk Branch: master Commit: 368ecaf55b25fbaafde8e1ea80299ddb84c655dc URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=368ecaf55b25fb… Author: Bernd Stroessenreuther <bs(a)mathias-kettner.de> Date: Tue Aug 27 16:06:15 2013 +0200 Updated bug entries #1037, #1039 --- .bugs/1037 | 25 +++++++++++++++++- .bugs/1039 | 82 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 106 insertions(+), 1 deletion(-) diff --git a/.bugs/1037 b/.bugs/1037 index 1061495..ff1bce5 100644 --- a/.bugs/1037 +++ b/.bugs/1037 @@ -1,9 +1,9 @@ Title: check_mk_agent.exe unable to restart after a crash: logstate.txt 0 bytes Component: checks +Class: bug State: open Date: 2013-08-27 10:58:29 Targetversion: 1.2.3 -Class: bug came in via feedback@ // original mail still available in feedback/gitbug folder @@ -27,3 +27,26 @@ To work around the problem I remove the logstate.txt and then start the check_mk hint by bs: the attached logstate.txt is 0 bytes + +2013-08-27 13:19:06: changed state open -> open +more info from Peter Rogers: + +Hi Bernd, + +Thank you for your reply, the currupt file is not empty (123k in size) so I have reattached a copy of the file. + +If I edit the file in Notepad++ I see one line of null entries, and converted to HEX I get the following: + +00000000000000000000000000000000 +00000000000000000000000000000000 +00000000000000000000000000000000 +00000000000000000000000000000000 +00000000000000000000000000000000 +00000000000000000000000000000000 +00000000000000000000000000000000 +0000000000000000000000 + +I have had a number of crashes on this server as the Windows machine is running on a VM server which has had problems, but the currupt logstate.txt file is always 123k in size and looks the same when edited. + +Regards, +Peter Rogers diff --git a/.bugs/1039 b/.bugs/1039 new file mode 100644 index 0000000..da19013 --- /dev/null +++ b/.bugs/1039 @@ -0,0 +1,82 @@ +Title: livestatus bug in OutputFormat: json +Component: livestatus +State: open +Date: 2013-08-27 16:02:02 +Targetversion: 1.2.3 +Class: bug + +came in via feedback@ // mail still available in feedback/gitbug folder + + +From: <Sascha.Pfalz(a)ivz.de> +To: <feedback(a)check-mk.org> +Subject: [Bug]: JSON Data falsch +Thread-Topic: [Bug]: JSON Data falsch +Thread-Index: Ac6IZny063o7fFKiSS2eLhEqKFzKaA== +Date: Wed, 24 Jul 2013 12:08:20 +0000 + +Hi, + +Erst mal möchte ich danke sagen für das super Plugin, das hat mir viel Arbeit erspart und macht den Einsatz von unseren Nagios und SMBox Systemen etwas leichter + +Allerdings scheint sich ein Bug in die V1.2.2p2 eingeschlichen zu haben, und zwar geht es um das Auslesen von Kommentaren bei folgendem Kommando: + +$CMD = "GET services\n"; +$CMD.= "Columns: host_name comments_with_info\n"; +$CMD.= "Filter: host_name = localhost\n"; +$CMD.= "OutputFormat: json\n"; + +Der Host localhost hat 11 Services definiert, wobei einer einen Kommentar definiert hat. Dabei erhalte ich folgende Ausgabe: + +[["localhost",[]], +["localhost",[]], +["localhost",[]], +["localhost",[]], +["localhost",[]], +["localhost",[]], +["localhost",[]], +["localhost",[]], +["localhost",[]], +["localhost",[]], +["localhost",[[705,"nagios","Testing",]]]] + +Und das ist leider nicht korrekt, das letzte ",]" ist falsch! Diese Daten lassen sich leider nicht via json_decode() in PHP dekodieren. Wandel ich das Ganze aber wie folgt ab: + +[["localhost",[]], +["localhost",[]], +["localhost",[]], +["localhost",[]], +["localhost",[]], +["localhost",[]], +["localhost",[]], +["localhost",[]], +["localhost",[]], +["localhost",[]], +["localhost",[[705,"nagios","Testing"]]]] + +-------------------------------------^^^ + + +Dann funktioniert das auch sofort wieder. + +Ich habe leider aktuell nicht so viel Zeit, um einen Patch zu liefern, deswegen dieser etwas chaotische Bugreport, ich hoffe aber, das man das soweit nachvollziehen kann. + +In der 1.1.10p3 war dieser Bug definitiv nicht drin. + +Getestet habe ich mit den Nagios Versionen 3.2.1 und 3.5.0, bei beiden tritt diese falsche JSON Kodierung auf. + +Nochmals vielen Dank für das Plugin! + +Viele Grüße, +-- +Sascha Pfalz +Oracle Developer +Archiv- und Produktionssysteme + +Tel. +49(0)30 97993-86318 +sascha.pfalz"at"ivz.de + +Informations-Verarbeitungs-Zentrum +Gemeinschaftseinrichtung von +DW - Deutschlandradio - MDR - NDR - RB - RBB - SR - WDR +Masurenallee 8-14 14057 Berlin

10 years, 10 months

Check_MK Git: check_mk: Improved handling of multisite authentication in check_bi_local script

by Lars Michelsen

Module: check_mk Branch: master Commit: 21d4308a49e9ab6052f80ab2230f3bbe524d9703 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=21d4308a49e9ab… Author: Lars Michelsen <lm(a)mathias-kettner.de> Date: Tue Aug 27 14:48:17 2013 +0200 Improved handling of multisite authentication in check_bi_local script --- doc/treasures/check_bi_local.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/treasures/check_bi_local.py b/doc/treasures/check_bi_local.py index cd60af7..304a68a 100755 --- a/doc/treasures/check_bi_local.py +++ b/doc/treasures/check_bi_local.py @@ -32,7 +32,7 @@ # 3. Add a correct url_prefix (OMD site and slash) # user and password with read access to Multisite. -url_prefix = "event/" # non-OMD installations +url_prefix = "" # non-OMD installations # url_prefix = "mysite/" # with OMD site name # Authentication credentials

10 years, 10 months

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

Checkmk git commits August 2013