Checkmk git commits

checkmk-commits@lists.checkmk.com

1 participants
64454 discussions

Check_MK Git: check_mk: FIX: LDAP configuration: allow DNs without DC=

by Mathias Kettner

Module: check_mk Branch: master Commit: 04d5a0b1c6d34e4c4e1fcc55d3698fb12b288981 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=04d5a0b1c6d34e… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Wed May 22 16:21:04 2013 +0200 FIX: LDAP configuration: allow DNs without DC= --- ChangeLog | 7 ++++--- web/htdocs/valuespec.py | 4 ---- 2 files changed, 4 insertions(+), 7 deletions(-) diff --git a/ChangeLog b/ChangeLog index 0ebc35f..58e4ac7 100644 --- a/ChangeLog +++ b/ChangeLog @@ -155,6 +155,7 @@ WATO: * FIX: convert editing of global setting to POST. This avoid URL-too-long when defining lots of Event Console actions + * FIX: LDAP configuration: allow DNs without DC= Event Console: * FIX: fix icon in events check if host specification is by IP address @@ -288,7 +289,7 @@ user/kernel percentages. (thanks to William Baum) * FIX: UPS checks support Eaton Evolution * FIX: windows agent plugin: mssql now exits after 10 seconds - + Notifications: * FIX: fixed crash on host notification when contact had explicit services set @@ -302,12 +303,12 @@ Multisite: * FIX: Avoid duplicate "Services" button in host detail views * FIX: fix rescheduling icon for services with non-ASCII characters - * New filter for IP address of a host + * New filter for IP address of a host * Quicksearch: allow searching for complete IP addresses and IP address prefixes * Add logentry class filter to view 'Host- and Service events' - BI: + BI: * FIX: fix exception with expansion level being 'None' * FIX: speedup for single host tables joined by hostname (BI-Boxes) * FIX: avoid closing BI subtree while tree is being loaded diff --git a/web/htdocs/valuespec.py b/web/htdocs/valuespec.py index 84c0c99..a3866cb 100644 --- a/web/htdocs/valuespec.py +++ b/web/htdocs/valuespec.py @@ -2340,10 +2340,6 @@ class LDAPDistinguishedName(TextAscii): def validate_value(self, value, varprefix): TextAscii.validate_value(self, value, varprefix) - # At least one DC= must be in distinguished name - if value and 'dc=' not in value.lower(): - raise MKUserError(varprefix, _('Found no "dc=" (Domain Component).')) - # Check wether or not the given DN is below a base DN if self.enforce_suffix and value and not value.lower().endswith(self.enforce_suffix.lower()): raise MKUserError(varprefix, _('Does not ends with "%s".') % self.enforce_suffix)

11 years, 4 months

Check_MK Git: check_mk: FIX: free_ipmi: fix name of sensor cache file if hostname contains domain part

by Mathias Kettner

Module: check_mk Branch: master Commit: 753c2b255f0c740650b704e7ade46526232493d0 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=753c2b255f0c74… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Wed May 22 16:02:29 2013 +0200 FIX: free_ipmi: fix name of sensor cache file if hostname contains domain part Conflicts: ChangeLog --- ChangeLog | 4 ++++ agents/check_mk_agent.linux | 2 +- 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/ChangeLog b/ChangeLog index 43328f4..0ebc35f 100644 --- a/ChangeLog +++ b/ChangeLog @@ -146,6 +146,7 @@ * FIX: zfsget: do not assume that devices begin with /, but mountpoints * FIX: ipmi_sensors: handle two cases for DELL correctly (thanks to Sebastian Talmon) * FIX: check_dns: enable performance data + * FIX: free_ipmi: fix name of sensor cache file if hostname contains domain part Multisite: * List of views: Output the alias of a datasource instead of internal name @@ -180,6 +181,9 @@ * FIX: Fixed exception during initial page rendering in python 2.6 in special cases (Internal error: putenv() argument 2 must be string, not list) + Livestatus: + * livestatus.log: show utf-8 decoding problems only with debug logging >=2 + Notifications: * FIX: HTML mails: Handle the case where plugin argument is not set * FIX: HTML mails: remove undefinded placeholders like $GRAPH_CODE$ diff --git a/agents/check_mk_agent.linux b/agents/check_mk_agent.linux index 9c80562..335ba79 100755 --- a/agents/check_mk_agent.linux +++ b/agents/check_mk_agent.linux @@ -243,7 +243,7 @@ fi # for the first run (where the cache is created). If ipmi-sensors runs into # that timeout, it leaves and empty cache file. We skip this check forever # if we find that empty cache file. -sdrcache=/var/cache/.freeipmi/sdr-cache/sdr-cache-$(hostname).127.0.0.1 +sdrcache=/var/cache/.freeipmi/sdr-cache/sdr-cache-$(hostname | cut -d. -f1).127.0.0.1 if type ipmi-sensors >/dev/null && [ ! -e "$sdrcache" -o -s "$sdrcache" ] then echo '<<<ipmi_sensors>>>'

11 years, 4 months

Check_MK Git: check_mk: mrpe: handle long plugin output correctly

by Mathias Kettner

Module: check_mk Branch: master Commit: 7705017d0d5db5093d9bdaa1538eecac56832dfe URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=7705017d0d5db5… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Wed May 22 15:35:16 2013 +0200 mrpe: handle long plugin output correctly --- ChangeLog | 1 + checks/mrpe | 55 ++++++++++++++++++++++++++++++++++++------------------- 2 files changed, 37 insertions(+), 19 deletions(-) diff --git a/ChangeLog b/ChangeLog index 2032928..43328f4 100644 --- a/ChangeLog +++ b/ChangeLog @@ -103,6 +103,7 @@ * logwatch: Agent can now use logwatch.d/ to split config to multipe files * logwatch: Agent can now rewrite Messages * apache_status: New rule: set levels for number of remaining open slots + * mrpe: handle long plugin output correctly, including performance data Notifications: * notify.py: unique spoolfiles name no longer created with uuid diff --git a/checks/mrpe b/checks/mrpe index b0f8e12..aa1390e 100644 --- a/checks/mrpe +++ b/checks/mrpe @@ -37,7 +37,14 @@ def inventory_mrpe(info): return items +def mrpe_parse_perfdata(perfinfo): + varname, valuetxt = perfinfo.split("=", 1) + values = valuetxt.split(";") + return tuple( [varname] + values) + + def check_mrpe(item, params, info): + # This check is cluster-aware. An item might be found # more than once. In that case we use the best of the # multiple statuses. @@ -52,38 +59,48 @@ def check_mrpe(item, params, info): check_name = None if line[0] == item: state = int(line[1]) - rest = " ".join(line[2:]) - parts = rest.split("|", 1) - # replace first line break with "\\n" -> Nagios expects it like this - output = parts[0].replace("\1", "\\n", 1).replace("\1", "<br>") + # convert to original format by joining and replacing \1 back with \n + rest = " ".join(line[2:]).replace("\1", "\n") + # split into lines + lines = rest.split('\n') + # First line: OUTPUT|PERFDATA + parts = lines[0].split("|", 1) + output = [parts[0].strip()] + if len(parts) > 1: + perfdata = parts[1].strip().split() + else: + perfdata = [] - perfdata = [] - if len(parts) > 1: # found pipe symbol - perftxt = parts[1].strip() - for perfinfo in perftxt.split(" "): - try: - varname, valuetxt = perfinfo.split("=", 1) - values = valuetxt.split(";") - perfdata.append(tuple( [varname] + values) ) - except: - pass + # Further lines + now_comes_perfdata = False + for l in lines[1:]: + if now_comes_perfdata: + perfdata += l.split() + else: + parts = l.split("|", 1) + output.append(parts[0].strip()) + if len(parts) > 1: + perfdata += parts[1].strip().split() + now_comes_perfdata = True - # name of check command needed for PNP to choose the correct template - if check_name: - perfdata.append(check_name) if best_state in [ None, 2 ] \ or (state < best_state and state != 2): - best_result = state, output, perfdata + infotext = "\\n".join(output) + perf_parsed = map(mrpe_parse_perfdata, perfdata) + # name of check command needed for PNP to choose the correct template + if check_name: + perf_parsed.append(check_name) + best_result = state, "\\n".join(output), perf_parsed best_state = state + if best_state == None: return (3, "Check output not found in output of MRPE") else: return best_result - check_info["mrpe"] = { 'check_function': check_mrpe, 'inventory_function': inventory_mrpe,

11 years, 4 months

Check_MK Git: check_mk: fix trailing space

by Mathias Kettner

Module: check_mk Branch: master Commit: 0e7b4523c3175d66f73f65c7801b18a3fa4d508b URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=0e7b4523c3175d… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Wed May 22 12:33:45 2013 +0200 fix trailing space --- checks/ipmi_sensors | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/checks/ipmi_sensors b/checks/ipmi_sensors index a9c2b34..e23ffbe 100644 --- a/checks/ipmi_sensors +++ b/checks/ipmi_sensors @@ -113,7 +113,7 @@ def check_freeipmi(item, _no_params, info): or status == "Entity_Present" \ or status.endswith("is_connected") \ or status.endswith("Presence_detected") \ - or status == "battery_presence_detected" \ + or status == "battery_presence_detected" \ or status == "Drive_Presence" \ or status.startswith("Fully_Redundant"): return (0, infotext, perfdata)

11 years, 4 months

Check_MK Git: check_mk: FIX: check_dns: enable performance data

by Mathias Kettner

Module: check_mk Branch: master Commit: 39fe4b6665247e2e27043a58e6194901bc0e50f2 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=39fe4b6665247e… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Wed May 22 12:34:04 2013 +0200 FIX: check_dns: enable performance data --- ChangeLog | 1 + checks/check_dns | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/ChangeLog b/ChangeLog index 88386a3..2032928 100644 --- a/ChangeLog +++ b/ChangeLog @@ -144,6 +144,7 @@ * FIX: aix_lvm: handle agents that output an extra header line * FIX: zfsget: do not assume that devices begin with /, but mountpoints * FIX: ipmi_sensors: handle two cases for DELL correctly (thanks to Sebastian Talmon) + * FIX: check_dns: enable performance data Multisite: * List of views: Output the alias of a datasource instead of internal name diff --git a/checks/check_dns b/checks/check_dns index 0a5b48b..0a29dfe 100644 --- a/checks/check_dns +++ b/checks/check_dns @@ -50,6 +50,6 @@ active_check_info['dns'] = { "command_line" : '$USER1$/check_dns $ARG1$', "argument_function" : check_dns_arguments, "service_description" : lambda params: "DNS %s" % params[0], - "has_perfdata" : False, + "has_perfdata" : True, }

11 years, 4 months

Check_MK Git: check_mk: FIX: ipmi_sensors: handle two cases for DELL correctly (thanks to Sebastian Talmon)

by Mathias Kettner

Module: check_mk Branch: master Commit: fc7a164843b310b88d610b822735de49d1c3c760 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=fc7a164843b310… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Wed May 22 12:10:09 2013 +0200 FIX: ipmi_sensors: handle two cases for DELL correctly (thanks to Sebastian Talmon) --- ChangeLog | 1 + checks/ipmi_sensors | 2 ++ 2 files changed, 3 insertions(+) diff --git a/ChangeLog b/ChangeLog index c15d27a..88386a3 100644 --- a/ChangeLog +++ b/ChangeLog @@ -143,6 +143,7 @@ * FIX: ipmi_sensors: ignore sensors in state [NA] (not available) * FIX: aix_lvm: handle agents that output an extra header line * FIX: zfsget: do not assume that devices begin with /, but mountpoints + * FIX: ipmi_sensors: handle two cases for DELL correctly (thanks to Sebastian Talmon) Multisite: * List of views: Output the alias of a datasource instead of internal name diff --git a/checks/ipmi_sensors b/checks/ipmi_sensors index 9d89ea9..a9c2b34 100644 --- a/checks/ipmi_sensors +++ b/checks/ipmi_sensors @@ -113,6 +113,8 @@ def check_freeipmi(item, _no_params, info): or status == "Entity_Present" \ or status.endswith("is_connected") \ or status.endswith("Presence_detected") \ + or status == "battery_presence_detected" \ + or status == "Drive_Presence" \ or status.startswith("Fully_Redundant"): return (0, infotext, perfdata) else:

11 years, 4 months

Check_MK Git: check_mk: FIX: zfsget: do not assume that devices begin with /, but mountpoints

by Mathias Kettner

Module: check_mk Branch: master Commit: ebe7d33125c2b25d785d9af6eab18e39db21208d URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=ebe7d33125c2b2… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Wed May 22 12:05:31 2013 +0200 FIX: zfsget: do not assume that devices begin with /, but mountpoints --- ChangeLog | 1 + checks/zfsget | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/ChangeLog b/ChangeLog index 1707784..c15d27a 100644 --- a/ChangeLog +++ b/ChangeLog @@ -142,6 +142,7 @@ * FIX: f5_bigip_psu: handle more than first power supply, thanks to Miro Ramza * FIX: ipmi_sensors: ignore sensors in state [NA] (not available) * FIX: aix_lvm: handle agents that output an extra header line + * FIX: zfsget: do not assume that devices begin with /, but mountpoints Multisite: * List of views: Output the alias of a datasource instead of internal name diff --git a/checks/zfsget b/checks/zfsget index cb51094..b152291 100644 --- a/checks/zfsget +++ b/checks/zfsget @@ -109,7 +109,7 @@ def zfsget_parse_df_info(entries, info): for device, kbytes, used, avail, percent, mountpoint in info: # ignore entries already contained in zfsget and also # entries for virtual filesystems (like swap) - if device.startswith("/") and mountpoint not in entries: + if mountpoint.startswith("/") and mountpoint not in entries: entry = {} total = int(kbytes) / 1024.0 entry["total"] = total

11 years, 4 months

Check_MK Git: check_mk: FIX: fix icon in events check if host specification is by IP address

by Mathias Kettner

Module: check_mk Branch: master Commit: dbbfd86db327e16339e4a51287eaca34c9789733 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=dbbfd86db327e1… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Wed May 22 11:54:22 2013 +0200 FIX: fix icon in events check if host specification is by IP address --- ChangeLog | 3 +++ mkeventd/web/plugins/icons/mkeventd.py | 4 ++-- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/ChangeLog b/ChangeLog index 7769b01..1707784 100644 --- a/ChangeLog +++ b/ChangeLog @@ -151,6 +151,9 @@ * FIX: convert editing of global setting to POST. This avoid URL-too-long when defining lots of Event Console actions + Event Console: + * FIX: fix icon in events check if host specification is by IP address + 1.2.2p1: Core: * FIX: correctly quote ! and \ in active checks for Nagios diff --git a/mkeventd/web/plugins/icons/mkeventd.py b/mkeventd/web/plugins/icons/mkeventd.py index c7027d3..01d67ad 100644 --- a/mkeventd/web/plugins/icons/mkeventd.py +++ b/mkeventd/web/plugins/icons/mkeventd.py @@ -78,11 +78,11 @@ def paint_mkeventd(what, row, tags, custom_vars): if getattr(config, 'mkeventd_distributed', False): url_prefix = site['url_prefix'] + 'check_mk/' - title = _('Events of Host %s') % (host) + title = _('Events of Host %s') % (row["host_name"]) url = 'view.py?' + htmllib.urlencode_vars([ ("view_name", "ec_events_of_monhost"), ("site", row["site"]), - ("host", host), + ("host", row["host_name"]), ]) if len(args) >= 2:

11 years, 4 months

Check_MK Git: check_mk: Added draft for livestatus proxy

by Mathias Kettner

Module: check_mk Branch: master Commit: 919a4e24af71e2e832bda6bcf19e3bc51980adf3 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=919a4e24af71e2… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Tue May 21 15:36:33 2013 +0200 Added draft for livestatus proxy --- doc/drafts/LIESMICH.livestatusproxy | 128 +++++++++++++++++++++++++++++++++++ 1 file changed, 128 insertions(+) diff --git a/doc/drafts/LIESMICH.livestatusproxy b/doc/drafts/LIESMICH.livestatusproxy new file mode 100644 index 0000000..bd8ea5c --- /dev/null +++ b/doc/drafts/LIESMICH.livestatusproxy @@ -0,0 +1,128 @@ +Daemon könnte heißen: liveproxyd "Livestatus Proxy Daemon" + +Schritte der Implementierung: + +1. Grundgerüst Daemon mit --help und daemonize(). Dazu + die Option -g, damit er im Vordergrund läuft und + --debug, was die Exceptions durchkommen lässt. Dazu + getopt. Auch ein var/log/livepd.log. Bei -g wird + statt in log auf stdout geschrieben (siehe mkeventd) + + Vorlage: mkeventd + + Vorerst läuft as nur in OMD. Daher ist keine Konfiguration + von Pfaden, etc. notwendig. + + +2. Laden der Konfiguration und Anlegen der globalen Datenstruktur + g_sites. Pro Site ein Eintrag mit dynamischen Daten (Zustand + und Channels) und Konfigurationsdaten. Dazu ein globales + Lock. Zugriff dann immer mit with lock_sites: Testkonfiguration + von Hand schreiben. + + +3. Eine Klasse, die einen Thread implementiert für eine Site. + Diese sorgt dafür, dass die konfigurierte Anzahl von + Verbindungen vorhanden ist. Und macht einfach in einer Schleife + ein TCP connect auf das Ziel. Und trägt neue Connections in + den Pool ein (lock beachten). Wenn all Verbindungen da sind, + kann er warten. + + --> Jetzt kann man schon testen, ob die Verbindungen aufgebaut + werden. + + --> Zum Testen z.B. eine Wartezeit von 5 Sekunden in die Schleife + bauen. + + +4. Hauptprozess: Erzeugen der UNIX-Sockets pro Site in + tmp/run/liveproxy/$SITE und auf listen() gehen. + +5. Hauptschleife: mit einem select() auf allen diesen Sockets + warten. Wenn eine Verbindung kommt, diese annehmen und + in einem Clientpool speichern. Clientpool: + + [ { + "site" : "muc", + "wait" : "query"/"response"/"channel", + "since" : wartet seit. + "channel" -> Zeigt auf Kanaldict im Falle "response" + "socket" -> Socket-Objekt, dass von accept() geliefert wurde + ... + } + ... + ] + + --> Das kann man jetzt wieder testen. + +6. select() erweitern, so dass es jetzt auch auf die bestehenden + Client-Verbindungen wartet. In der Schleife, jetzt zusätzlich + auch Queries lesen und Testeweise ausgeben. + + --> Testen. + +7. Hauptschleife so erweitern, dass das select() nicht länger + als z.B. 0.1 sec wartet. Danach immer alle Sites durchgehen + und Heartbeat abwickeln. z.B. in einer eigenen Funktion + + Heartbeat: Alle Sites durchgehen. Pro Site gucken: Ist + ein Heartbeat ausstehend? Ja? Wartezeit prüfen. Wenn zu + lange, dann Site als tot markieren. Alle Verbindungen + abrechen. + + Wenn kein Heartbeat aussteht: Ist kein Kanal frei, dann + sollte die Site noch auf tot stehen. Dann einfach warten. + Wenn ein Kanal frei ist, ein Heartbeat senden und dessen + Zeitstempel im Kanal vermerken. + + +8. select() auf die Kanäle erweitern, die im Zustand "busy" + oder "heartbeat" stehen. Wenn so ein Kanal lesbar wird, + dann kann es aktuell nur "heartbeat" sein. Diesen + einsammeln und die Zeit des letzten Heartbeat vermerken. + +9. Jetzt von Punkt 6 das so erweitern, dass wenn Client-Anfrage + reinkommt. Wenn ein Kanal für die Zielsite frei ist, dann + Anfrage vom Client lesen und in den Kanal schreiben. Kanal + markiern. --> TESTEN. Jetzt muss eine Anfrage auf dem Zielsystem + eingehen. + + Wenn die Site tot ist, dann Anfrage auch einlesen, und selbst + mit einem Fehler-Header beantworten. + + Wenn Site lebt, aber kein Kanal frei ist: + Fall 1: erster Versuch: setzen Zeitstempel beim dict der + Client-Verbindung ("since" etdc.) + Fall 2: "since" Feld bereits gesetzt. Timeout prüfen. + Der Wert steht in g_sites[site]["timeout"]. Wenn überschritten, + dann Query lesen und mit Fehler beantworten "All %d channels busy, timeout waiting for free channel." + +10. 8 erweitern: Wenn kanal lesbar wird, der im Zustand "busy" + ist, dann die Anfrage auslesen. Nur ein read(), damit wir + nicht blockieren! Wenn die Anfrage nicht komplett ist, setzen + speichern wir das bereits gelesene im Kanal unter der Variable + "partial" ein. Und machen vorerst nix weiter. + Wenn bereits ein "partial" vorhanden war, hängen wir uns dran. + Wenn jetzt alle Daten da sind, senden wir das Ergebnis zum + Client. Zustände von Client und Kanal entsprechend setzen. + +11. Signal zum Beenden implemtieren und versuchen, sauber runterzufahren. + +12. Startskript schreiben. + +13. omd-Hook schreiben, mit dem man den Proxy aktivieren kann. Der proxy + muss dann auch in etc/check_mk/multisite.d/liveproxy.mk die Zeile + liveproxy = True schreiben. + +14. WATO-Site-Konfiguration: Wenn proxy aktiviert per omd config + (config.liveproxy), dann zusätzliche Option: + [X] Use Livestatus Proxy + Number of channels: [ 10] + Wait for free channel: [ 5] sec + Heartbeat interval: [ 3] sec + Heartbeat timeout: [ 50]% over typical response time + Maximum Response time: [120] sec + + Beim Schreiben der sites.mk auch die liveproxy.mk ein etc/check_mk + erzeugen. Dabei 'omd restart liveproxy' aufrufen. +

11 years, 4 months

Check_MK Git: check_mk: Merge branch 'master' of ssh://mathias-kettner.de/ check_mk

by Mathias Kettner

Module: check_mk Branch: master Commit: 4d46e86b22d255acadd058648d96cd0d727c82bd URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=4d46e86b22d255… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Tue May 21 15:36:41 2013 +0200 Merge branch 'master' of ssh://mathias-kettner.de/check_mk ---

11 years, 4 months

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

Checkmk git commits