Checkmk git commits

checkmk-commits@lists.checkmk.com

1 participants
64139 discussions

Check_MK Git: check_mk: Fixed syntax error in manpage of akcp_sensor_temp

by git＠mathias-kettner.de

Module: check_mk Branch: master Commit: 815d75c003a0b25e55f5627a8595dbbea8b9b575 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=815d75c003a0b2… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Sun Oct 24 19:17:26 2010 +0200 Fixed syntax error in manpage of akcp_sensor_temp --- LIESMICH.zutun | 2 -- checkman/akcp_sensor_temp | 2 +- 2 files changed, 1 insertions(+), 3 deletions(-) diff --git a/LIESMICH.zutun b/LIESMICH.zutun index 9b25838..f5527da 100644 --- a/LIESMICH.zutun +++ b/LIESMICH.zutun @@ -12,8 +12,6 @@ BUGS beheben bis => 1.1.8 -------------------------------------------------------------------------------- CHECK_MK: -akcp_sensor_temp -> Man-Page ist kaputt - doc/ aufräumen und komplett 1:1 installieren. Clusters mit IP-Adressen: Der Host-Check soll die IP-Adresse verwenden, diff --git a/checkman/akcp_sensor_temp b/checkman/akcp_sensor_temp index e985a57..5e7f7a8 100644 --- a/checkman/akcp_sensor_temp +++ b/checkman/akcp_sensor_temp @@ -33,4 +33,4 @@ critical high (int): upper border for critical level [configuration] akcp_sensor_humidity (int, int, int, int): default levels for lower critical lower warning -and upper warning, upper critical. Preset to ( 35, 40, 60, 65 ). + and upper warning, upper critical. Preset to ( 35, 40, 60, 65 ).

13 years, 11 months

Check_MK Git: check_mk: Multisite: allow non-ASCII character in downtimes and comments

by git＠mathias-kettner.de

Module: check_mk Branch: master Commit: 86ad0ce641650e8c66b06325ecb12823b13f8071 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=86ad0ce641650e… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Sun Oct 24 18:52:16 2010 +0200 Multisite: allow non-ASCII character in downtimes and comments --- ChangeLog | 1 + LIESMICH.zutun | 29 ++++------------------------- web/htdocs/htmllib.py | 6 ++++++ web/htdocs/views.py | 22 ++++++++++------------ 4 files changed, 21 insertions(+), 37 deletions(-) diff --git a/ChangeLog b/ChangeLog index 10ebf51..d1f8bbb 100644 --- a/ChangeLog +++ b/ChangeLog @@ -27,6 +27,7 @@ size of external command buffer * Fix links for items containing + (e.g. service descriptionen including spaces) + * Allow non-ASCII character in downtimes and comments 1.1.8b3: Core, Setup, etc.: diff --git a/LIESMICH.zutun b/LIESMICH.zutun index 5135fab..1ede43a 100644 --- a/LIESMICH.zutun +++ b/LIESMICH.zutun @@ -1,18 +1,12 @@ -------------------------------------------------------------------------------- UMLAUTPROBLEME (alle gemeinsam angehen) -------------------------------------------------------------------------------- -Eingabe von Downtimes: Auch hier Umlaute abfangen wie bei Comments - Noch ein Problem mit Umlauten: Ein Drucker sendet per SNMP Latin1 kodierte Wert, z.B. Resttonerbehälter. Daraus baut Check_MK eine Servicedesription, die in nagios/conf.d/check_mk_objects.cfg ein Latin1-Zeichen enthält. Hier sollte unbedingt UTF-8 verwendet werden. Nur: woher kennen wir die Kodierung vom Drucker?? -Sieht so aus, dass das Eurozeichen von Livestatus nicht korrekt kodiert -wird. Wenn man commant_data=... und ein Euro drinnen hat, verschwindet -es bei der Ausgabe mit python/json. CSV geht. - -------------------------------------------------------------------------------- BUGS beheben bis => 1.1.8 -------------------------------------------------------------------------------- @@ -32,28 +26,13 @@ doc/ aufräumen und komplett 1:1 installieren. Clusters mit IP-Adressen: Der Host-Check soll die IP-Adresse verwenden, wenn sie vorhanden ist. -MULTISITE: Hallo zusammen, mir ist bei meinen Tests mit der Multisite -GUI aufgefallen, dass keine Comments zu Services oder Hosts dargestellt -werden. Comments zu Downtimes, werden hingegen ohne Probleme dargestellt. -Unter der Default-View "Other - > Comments" wird ebenfalls kein Host /Service -dargestellt, obwohl Comments hinterlegt sind, welche in der Classic GUI auch -dargestellt werden. Beim Versuch eine eigene View zu erstellen, ist mir -dann auch aufgefallen, dass die Datasource Comments nicht zur Auswahl steht. -Getestet habe ich dies mit den Versionen: 1.1.6.p1 und 1.1.8.b3 Hat jemand -schon ähnliche Erfahrungen gesammelt? Vielleicht ist es ja auch nur ein -Bedienerfehler meinerseits. - -Wie siehts denn mit dem utf-8 Problem aus. Kommt das noch in die 1.1.8 rein, bzw, raus :-) -Zum Ausprobieren reicht es per Nagios oder Thruk eine Downtime "€" zu setzen. Mit Multisite kann man sie zwar -nicht setzen, aber er zeigt sie auch falsch an. -Btw, beim Comment setzen kommt: -Please only use ASCII characters in your comment. -Beim Versuch eine Downtime zu setzen: -Internal error: 'ascii' codec can't decode byte 0xe2 in position 0: ordinal not in range(128) (Retry with debug mode) - -------------------------------------------------------------------------------- BUGS beheben ab 1.1.9i1 -------------------------------------------------------------------------------- +Livestatus: bestimmte UTF-8 Zeichen werden nicht korrekt kodiert, z.B. "€". +Um das umzustellen, muss man wahrscheinlich einfach UTF-8 generell +annehmen. + Wenn ein Counter Wrap passiert und der check selbst die Exception nicht abfängt, dann wird das Ergebnis nicht an Nagios weitergeleitet. Der Service wird auch nicht in die aggregierten Services eines Summary Hosts diff --git a/web/htdocs/htmllib.py b/web/htdocs/htmllib.py index 54fba99..57ba466 100644 --- a/web/htdocs/htmllib.py +++ b/web/htdocs/htmllib.py @@ -66,6 +66,7 @@ class uriinfo: def attrencode(value): return cgi.escape(str(value), True) +# This function returns a str object, never unicode! def urlencode_vars(vars): output = "" for varname, value in vars: @@ -83,6 +84,8 @@ def urlencode_vars(vars): return output def urlencode(value): + if type(value) == unicode: + value = value.encode("utf-8") ret = "" for c in value: if c == " ": @@ -418,6 +421,9 @@ class html: def var(self, varname, deflt = None): return self.req.vars.get(varname, deflt) + def var_utf8(self, varname, deflt = None): + return unicode(self.req.vars.get(varname, deflt), "utf-8") + def set_var(self, varname, value): self.req.vars[varname] = value diff --git a/web/htdocs/views.py b/web/htdocs/views.py index ccd2e02..37a7be0 100644 --- a/web/htdocs/views.py +++ b/web/htdocs/views.py @@ -983,7 +983,9 @@ def show_view(view, show_heading = False, show_buttons = True): if html.do_actions() and html.transaction_valid(): # submit button pressed, no reload try: html.write("<tr class=form><td class=whiteborder colspan=%d>" % colspan) - has_done_actions = do_actions(datasource["infos"][0], rows, html.makeuri([])) + # Create URI with all actions variables removed + backurl = html.makeuri([]) + has_done_actions = do_actions(datasource["infos"][0], rows, backurl) html.write("</td></tr>") except MKUserError, e: html.show_error(e.message) @@ -1511,25 +1513,19 @@ def nagios_host_service_action_command(what, dataset): title = "manually set check results to %s for" % statename elif html.var("_acknowledge") and config.may("action.acknowledge"): - comment = html.var("_ack_comment") + comment = html.var_utf8("_ack_comment") if not comment: raise MKUserError("_ack_comment", "You need to supply a comment.") - try: - command = "ACKNOWLEDGE_" + cmdtag + "_PROBLEM;%s;2;1;0;%s" % \ + command = "ACKNOWLEDGE_" + cmdtag + "_PROBLEM;%s;2;1;0;%s" % \ (spec, html.req.user) + (";%s" % comment) - except: - raise MKUserError("_ack_comment", "Please only use ASCII characters in your comment.") title = "acknowledge the problems of" elif html.var("_add_comment") and config.may("action.addcomment"): - comment = html.var("_comment") + comment = html.var_utf8("_comment") if not comment: raise MKUserError("_comment", "You need to supply a comment.") - try: - command = "ADD_" + cmdtag + "_COMMENT;%s;1;%s" % \ + command = "ADD_" + cmdtag + "_COMMENT;%s;1;%s" % \ (spec, html.req.user) + (";%s" % comment) - except: - raise MKUserError("_ack_comment", "Please only use ASCII characters in your comment.") title = "add a comment to" elif html.var("_remove_ack") and config.may("action.acknowledge"): @@ -1591,7 +1587,7 @@ def nagios_host_service_action_command(what, dataset): raise MKUserError(None, "Sorry. This command is not implemented.") if down_to: - comment = html.var("_down_comment") + comment = html.var_utf8("_down_comment") if not comment: raise MKUserError("_down_comment", "You need to supply a comment for your downtime.") command = (("SCHEDULE_" + cmdtag + "_DOWNTIME;%s;" % spec) \ @@ -1616,6 +1612,8 @@ def do_actions(what, rows, backurl): for row in rows: title, nagios_commands = nagios_action_command(what, row) for command in nagios_commands: + if type(command) == unicode: + command = command.encode("utf-8") html.live.command(command, row["site"]) count += 1

13 years, 11 months

Check_MK Git: check_mk: Updated internal doku

by git＠mathias-kettner.de

Module: check_mk Branch: master Commit: 8c026a319ecaa7bf5f62c25b57003c25ddf4c7d6 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=8c026a319ecaa7… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Sun Oct 24 17:59:52 2010 +0200 Updated internal doku --- LIESMICH.zutun | 23 ++++++----------------- 1 files changed, 6 insertions(+), 17 deletions(-) diff --git a/LIESMICH.zutun b/LIESMICH.zutun index 8fdf0ca..5135fab 100644 --- a/LIESMICH.zutun +++ b/LIESMICH.zutun @@ -18,12 +18,6 @@ BUGS beheben bis => 1.1.8 -------------------------------------------------------------------------------- CHECK_MK: -Wenn ein Counter Wrap passiert und der check selbst die Exception -nicht abfängt, dann wird das Ergebnis nicht an Nagios weitergeleitet. -Der Service wird auch nicht in die aggregierten Services eines Summary Hosts -aufgenommen. Das kann dazu führen, dass die Anzahl der Services in einem -Summary-Host variabel ist. - inventory_processes: wenn man hier einen Benuter fest angibt, dann wird der nicht übernommen, sondern None fest kodiert. @@ -57,20 +51,15 @@ Please only use ASCII characters in your comment. Beim Versuch eine Downtime zu setzen: Internal error: 'ascii' codec can't decode byte 0xe2 in position 0: ordinal not in range(128) (Retry with debug mode) -I'm running check_mk version 1.1.8b2 on Slackware linux. -I just added a new host to check_mk to monitor, which added a service called 'CUPS Queue LQ-300+'. -Works fine, but in the check_mk page a link is generated that doesn't work because the + sign isn't URL encoded. -So instead of making it: -<a href="http://nagios/check_mk/view.py?view_name=service&site=&service=CUPS%20Queue…">CUPS Queue LQ-300+</a> -It creates: -<a href="http://nagios/check_mk/view.py?view_name=service&site=&service=CUPS%20Queue…"> -(note the + vs %2B) - - - -------------------------------------------------------------------------------- BUGS beheben ab 1.1.9i1 -------------------------------------------------------------------------------- +Wenn ein Counter Wrap passiert und der check selbst die Exception +nicht abfängt, dann wird das Ergebnis nicht an Nagios weitergeleitet. +Der Service wird auch nicht in die aggregierten Services eines Summary Hosts +aufgenommen. Das kann dazu führen, dass die Anzahl der Services in einem +Summary-Host variabel ist. + Cluster-Inventur: Wenn ein Service auf der Node sitzt und man die Regeln zu Clustered services ändert, entfernt ein -II das nicht von der node

13 years, 11 months

Check_MK Git: check_mk: Multisite: Fix links to texts containing +

by git＠mathias-kettner.de

Module: check_mk Branch: master Commit: dec280fc5ea934d24a6f8ad73250f3fc7e0b02df URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=dec280fc5ea934… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Sun Oct 24 18:00:33 2010 +0200 Multisite: Fix links to texts containing + --- ChangeLog | 2 ++ web/htdocs/htmllib.py | 2 +- 2 files changed, 3 insertions(+), 1 deletions(-) diff --git a/ChangeLog b/ChangeLog index 5c3c38b..10ebf51 100644 --- a/ChangeLog +++ b/ChangeLog @@ -25,6 +25,8 @@ Multisite: * Snapin Performance: show external command per second and usage and size of external command buffer + * Fix links for items containing + (e.g. service descriptionen including + spaces) 1.1.8b3: Core, Setup, etc.: diff --git a/web/htdocs/htmllib.py b/web/htdocs/htmllib.py index 82fc403..54fba99 100644 --- a/web/htdocs/htmllib.py +++ b/web/htdocs/htmllib.py @@ -79,7 +79,7 @@ def urlencode_vars(vars): output += varname output += "=" - output += value + output += urlencode(value) return output def urlencode(value):

13 years, 11 months

Check_MK Git: check_mk: Updated internal doku

by git＠mathias-kettner.de

Module: check_mk Branch: master Commit: 2e318332d9e8ac6671dfdd1a569db5dcdb77e3c1 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=2e318332d9e8ac… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Sun Oct 24 10:25:44 2010 +0200 Updated internal doku --- LIESMICH.zutun | 19 +++++++++++++++++++ 1 files changed, 19 insertions(+), 0 deletions(-) diff --git a/LIESMICH.zutun b/LIESMICH.zutun index 641b152..8fdf0ca 100644 --- a/LIESMICH.zutun +++ b/LIESMICH.zutun @@ -49,6 +49,24 @@ Getestet habe ich dies mit den Versionen: 1.1.6.p1 und 1.1.8.b3 Hat jemand schon ähnliche Erfahrungen gesammelt? Vielleicht ist es ja auch nur ein Bedienerfehler meinerseits. +Wie siehts denn mit dem utf-8 Problem aus. Kommt das noch in die 1.1.8 rein, bzw, raus :-) +Zum Ausprobieren reicht es per Nagios oder Thruk eine Downtime "€" zu setzen. Mit Multisite kann man sie zwar +nicht setzen, aber er zeigt sie auch falsch an. +Btw, beim Comment setzen kommt: +Please only use ASCII characters in your comment. +Beim Versuch eine Downtime zu setzen: +Internal error: 'ascii' codec can't decode byte 0xe2 in position 0: ordinal not in range(128) (Retry with debug mode) + +I'm running check_mk version 1.1.8b2 on Slackware linux. +I just added a new host to check_mk to monitor, which added a service called 'CUPS Queue LQ-300+'. +Works fine, but in the check_mk page a link is generated that doesn't work because the + sign isn't URL encoded. +So instead of making it: +<a href="http://nagios/check_mk/view.py?view_name=service&site=&service=CUPS%20Queue…">CUPS Queue LQ-300+</a> +It creates: +<a href="http://nagios/check_mk/view.py?view_name=service&site=&service=CUPS%20Queue…"> +(note the + vs %2B) + + -------------------------------------------------------------------------------- BUGS beheben ab 1.1.9i1 @@ -99,6 +117,7 @@ getrennte Checks für Memory and Pagefile. --snmpwalk: Braucht man das translaten noch? Ich arbeite doch jetzt eh nur noch ohne MIB-Dateien. Das würde das ganze deutlich vereinfachen! + ------------------------------------------------------------------------------- ab 1.1.9i1: --------------------------------------------------------------------------------

13 years, 11 months

Check_MK Git: check_mk: setup: turn off Python debugging

by git＠mathias-kettner.de

Module: check_mk Branch: master Commit: 3adb1db8219d0653605008382038f0873a8790f5 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=3adb1db8219d06… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Sat Oct 23 12:07:42 2010 +0200 setup: turn off Python debugging --- ChangeLog | 1 + scripts/setup.sh | 2 +- 2 files changed, 2 insertions(+), 1 deletions(-) diff --git a/ChangeLog b/ChangeLog index cdf086f..5c3c38b 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,6 @@ 1.1.8: Core, Setup, etc.: + * setup.sh: turn off Python debugging * Cleaned up documentation directory Checks & Agents: diff --git a/scripts/setup.sh b/scripts/setup.sh index 52be942..f30154c 100755 --- a/scripts/setup.sh +++ b/scripts/setup.sh @@ -680,7 +680,7 @@ do <Directory $web_dir/htdocs> AddHandler mod_python .py PythonHandler index - PythonDebug On + PythonDebug Off DirectoryIndex index.py # Need Nagios authentification. Please edit the

13 years, 11 months

Check_MK Git: check_mk: Test commit

by lm＠mathias-kettner.de

Module: check_mk Branch: master Commit: c9497e72315fcbd8e6be196c424361c4962731d6 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=c9497e72315fcb… Author: Lars Michelsen <lm(a)mathias-kettner.de> Date: Fri Oct 22 11:45:55 2010 +0200 Test commit --- LIESMICH.zutun | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/LIESMICH.zutun b/LIESMICH.zutun index f8b6884..4ab9d9b 100644 --- a/LIESMICH.zutun +++ b/LIESMICH.zutun @@ -49,6 +49,7 @@ Bedienerfehler meinerseits. -------------------------------------------------------------------------------- BUGS beheben ab 1.1.9i1 -------------------------------------------------------------------------------- + Livestatus: wenn man mehrere StatsGroupBy macht, wird der Speicher von den alten nicht freigegeben (ist momentan eh nicht erlaubt. Fehlermeldung ausgeben). In neuer Version sollte man irgendwann mehrere Group-Header erlauben.

13 years, 11 months

Check_MK Git: check_mk: Test commit

by lm＠mathias-kettner.de

Module: check_mk Branch: master Commit: e3fcb085bb619619ac31af01b17d47a520bbd6a4 URL: http://git.mathias-kettner.de/git/?a=commit;h=e3fcb085bb619619ac31af01b17d4… Author: Lars Michelsen <lm(a)mathias-kettner.de> Date: Fri Oct 22 11:43:05 2010 +0200 Test commit --- LIESMICH.zutun | 1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/LIESMICH.zutun b/LIESMICH.zutun index 4ab9d9b..f8b6884 100644 --- a/LIESMICH.zutun +++ b/LIESMICH.zutun @@ -49,7 +49,6 @@ Bedienerfehler meinerseits. -------------------------------------------------------------------------------- BUGS beheben ab 1.1.9i1 -------------------------------------------------------------------------------- - Livestatus: wenn man mehrere StatsGroupBy macht, wird der Speicher von den alten nicht freigegeben (ist momentan eh nicht erlaubt. Fehlermeldung ausgeben). In neuer Version sollte man irgendwann mehrere Group-Header erlauben.

13 years, 11 months

Check_MK Git: check_mk: Test commit

by lm＠mathias-kettner.de

Module: check_mk Branch: master Commit: f7573e017ff6d4c4624d5560f248998cee1c47e5 URL: http://git.mathias-kettner.de/git/?p=check_mk.git/revisions/f7573e017ff6d4c… Author: Lars Michelsen <lm(a)mathias-kettner.de> Date: Fri Oct 22 11:36:15 2010 +0200 Test commit --- LIESMICH.zutun | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/LIESMICH.zutun b/LIESMICH.zutun index f8b6884..4ab9d9b 100644 --- a/LIESMICH.zutun +++ b/LIESMICH.zutun @@ -49,6 +49,7 @@ Bedienerfehler meinerseits. -------------------------------------------------------------------------------- BUGS beheben ab 1.1.9i1 -------------------------------------------------------------------------------- + Livestatus: wenn man mehrere StatsGroupBy macht, wird der Speicher von den alten nicht freigegeben (ist momentan eh nicht erlaubt. Fehlermeldung ausgeben). In neuer Version sollte man irgendwann mehrere Group-Header erlauben.

13 years, 11 months

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

Checkmk git commits