Checkmk git commits May 2013

checkmk-commits@lists.checkmk.com

3 participants
111 discussions

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, 1 month

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, 1 month

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, 1 month

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, 1 month

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, 1 month

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, 1 month

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, 1 month

Check_MK Git: check_mk: Remove Notification Command user settings, please use flexible notifications instead

by Mathias Kettner

Module: check_mk Branch: master Commit: f99bf3a65b4bf4de1ee135847705d22d0d2d7755 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=f99bf3a65b4bf4… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Mon May 20 23:01:20 2013 +0200 Remove Notification Command user settings, please use flexible notifications instead --- ChangeLog | 1 + web/htdocs/wato.py | 11 ----------- 2 files changed, 1 insertion(+), 11 deletions(-) diff --git a/ChangeLog b/ChangeLog index 28f415f..3b1fd9c 100644 --- a/ChangeLog +++ b/ChangeLog @@ -121,6 +121,7 @@ * Configuration of an alternativ host check command * Inventory: Display link symbol for ps ruleset * FIX: Rulesets: correct display of rules within subfolders + * Remove Notification Command user settings, please use flexible notifications instead 1.2.2p2: diff --git a/web/htdocs/wato.py b/web/htdocs/wato.py index eb82b7a..ac41331 100644 --- a/web/htdocs/wato.py +++ b/web/htdocs/wato.py @@ -8084,9 +8084,6 @@ def mode_edit_user(phase): for what, opts in [ ( "host", "durfs"), ("service", "wucrfs") ]: new_user[what + "_notification_options"] = "".join( [ opt for opt in opts if html.get_checkbox(what + "_" + opt) ]) - # FIXME: Validate notification commands. Do they really exist? - # FIXME(2): This is deprecated anyway. Remove in future. - new_user[what + "_notification_commands"] = html.var(what + "_notification_commands") value = vs_notification_method.from_html_vars("notification_method") vs_notification_method.validate_value(value, "notification_method") @@ -8307,14 +8304,6 @@ def mode_edit_user(phase): forms.section(_("Notification Method")) vs_notification_method.render_input("notification_method", user.get("notification_method")) - # Notification commands (deprecated) - forms.section(_("Notification Command for Hosts")) - html.text_input("host_notification_commands", user.get("host_notification_commands", "check-mk-notify")) - html.help(_("Use this Nagios command for sending host notifications.")) - forms.section(_("Notification Command for Services")) - html.text_input("service_notification_commands", user.get("service_notification_commands", "check-mk-notify")) - html.help(_("Use this Nagios command for sending service notifications.")) - forms.header(_("Personal Settings"), isopen = False) select_language(user.get('language', '')) for name, attr in userdb.get_user_attributes():

11 years, 1 month

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

by Mathias Kettner

Module: check_mk Branch: master Commit: 67e67f08b410045e342cdfdead1a3db5e9038966 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=67e67f08b41004… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Mon May 20 23:01:25 2013 +0200 Merge branch 'master' of ssh://mathias-kettner.de/check_mk ---

11 years, 1 month

Check_MK Git: check_mk: EC: more intelligent TP caching

by Mathias Kettner

Module: check_mk Branch: master Commit: 326447012547dfe950186313fef59c84f4f89277 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=326447012547df… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Fri May 17 13:15:18 2013 +0200 EC: more intelligent TP caching --- mkeventd/bin/mkeventd | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/mkeventd/bin/mkeventd b/mkeventd/bin/mkeventd index 67289e8..718db13 100755 --- a/mkeventd/bin/mkeventd +++ b/mkeventd/bin/mkeventd @@ -259,7 +259,7 @@ g_last_timeperiod_update = 0 def update_timeperiods(): global g_timeperiods, g_last_timeperiod_update - if g_timeperiods != None and time.time() < g_last_config_reload + 60: + if g_timeperiods != None and int(time.time()) / 60 == g_last_timeperiod_update: return # only update once a minute log("Updating timeperiod information") @@ -274,7 +274,7 @@ def update_timeperiods(): for tpname, alias, isin in table: new_timeperiods[tpname] = (alias, isin == '1' and True or False) g_timeperiods = new_timeperiods - g_last_timeperiod_update = time.time() + g_last_timeperiod_update = int(time.time()) / 60 except Exception, e: log("Cannot update timeperiod information: %s" % e) if opt_debug:

11 years, 1 month

← Newer
1
2
3
4
5
6
7
8
...
12
Older →

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

Checkmk git commits May 2013