[checkmk-commits] Check_MK Git: check_mk: Updated bug entries #1036, #1037, #1038, #1039

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 &lt;mk(a)mathias-kettner.de&gt; 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" &lt;peterrogers(a)email.com&gt; -Message-ID: &lt;20130607125613.34790(a)gmx.com&gt; -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: &lt;CACA1nGEzOmECJufVGOgUqZm6tAxGH+vHoXoEh7SGbx7uw+3ZwQ(a)mail.gmail.com&gt; -Subject: Fwd: [BUG]: MRPE Checks mit Performance Daten bringen Fehler -From: =?ISO-8859-1?Q?Andreas_D=F6hler?= &lt;andreas.doehler(a)gmail.com&gt; -To: Check MK Feedback &lt;feedback(a)check-mk.org&gt; - - - -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: &lt;Sascha.Pfalz(a)ivz.de&gt; -To: &lt;feedback(a)check-mk.org&gt; -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