Checkmk git commits May 2011

checkmk-commits@lists.checkmk.com

2 participants
134 discussions

Check_MK Git: check_mk: Impressive speed up of stored snmp walks

by git＠mathias-kettner.de

Module: check_mk Branch: master Commit: 2cd0a937ab582b1bc85c5637701524fda5a82276 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=2cd0a937ab582b… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Thu May 5 16:02:16 2011 +0200 Impressive speed up of stored snmp walks --- checks/if | 1 + modules/snmp.py | 81 +++++++++++++++++++++++++++++++++++++++++++++++++++++- 2 files changed, 80 insertions(+), 2 deletions(-) diff --git a/checks/if b/checks/if index 1e9ffc3..c37db49 100644 --- a/checks/if +++ b/checks/if @@ -25,6 +25,7 @@ # Boston, MA 02110-1301 USA. def inventory_if(checkname, info): + # import pprint; pprint.pprint(info) return inventory_if_common(if_convert_to_if64(info)) def check_if(item, params, info): diff --git a/modules/snmp.py b/modules/snmp.py index bec9d6b..b76a132 100644 --- a/modules/snmp.py +++ b/modules/snmp.py @@ -349,6 +349,7 @@ def check_snmp_fixed(item, targetvalue, info): return (0, "OK - %s" % (value,)) return (3, "Missing item %s in SNMP data" % item) +g_walk_cache = {} def get_stored_snmpwalk(hostname, oid): if oid.startswith("."): oid = oid[1:] @@ -361,12 +362,87 @@ def get_stored_snmpwalk(hostname, oid): dot_star = False path = snmpwalks_dir + "/" + hostname + if opt_debug: sys.stderr.write("Getting %s from %s\n" % (oid, path)) if not os.path.exists(path): raise MKGeneralException("No snmpwalk file %s\n" % path) rowinfo = [] + + use_new = True + if use_new: + # New implementation: use binary search + def to_bin_string(oid): + return tuple(map(int, oid.strip(".").split("."))) + + def compare_oids(a, b): + aa = to_bin_string(a) + bb = to_bin_string(b) + if len(aa) <= len(bb) and bb[:len(aa)] == aa: + result = 0 + else: + result = cmp(aa, bb) + return result + + if hostname in g_walk_cache: + lines = g_walk_cache[hostname] + else: + lines = file(path).readlines() + g_walk_cache[hostname] = lines + + begin = 0 + end = len(lines) + hit = None + while end - begin > 0: + current = (begin + end) / 2 + parts = lines[current].split(None, 1) + comp = parts[0] + hit = compare_oids(oid_prefix, comp) + if hit == 0: + break + elif hit == 1: # we are too low + begin = current + 1 + else: + end = current + + if hit != 0: + return [] # not found + + + def collect_until(index, direction): + rows = [] + while True: + line = lines[index] + parts = line.split(None, 1) + o = parts[0] + if o.startswith('.'): + o = o[1:] + if o == oid or o.startswith(oid_prefix + "."): + if len(parts) > 1: + value = parts[1] + if agent_simulator: + value = agent_simulator_process(value) + else: + value = "" + rows.append((o, strip_snmp_value(value))) + index += direction + if index < 0 or index >= len(lines): + break + else: + break + return rows + + + rowinfo = collect_until(current, -1) + rowinfo.reverse() + rowinfo += collect_until(current + 1, 1) + # import pprint ; pprint.pprint(rowinfo) + return rowinfo + + + + # Old implementation hot = False for line in file(path): parts = line.split(None, 1) @@ -383,7 +459,8 @@ def get_stored_snmpwalk(hostname, oid): value = "" rowinfo.append((o, strip_snmp_value(value))) # return pair of OID and value if dot_star: - return rowinfo # only return first (used by get_single_oid) + break elif hot: # end of interesting part, no point in further search - return rowinfo + break + # import pprint ; pprint.pprint(rowinfo) return rowinfo

13 years, 2 months

Check_MK Git: check_mk: cmk -C with --debug now lets exceptions through

by git＠mathias-kettner.de

Module: check_mk Branch: master Commit: a4bbfc102e3de1ceaca57ee6642d002981b32d5e URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=a4bbfc102e3de1… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Thu May 5 14:15:15 2011 +0200 cmk -C with --debug now lets exceptions through --- modules/check_mk.py | 3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/modules/check_mk.py b/modules/check_mk.py index 0066b10..a97231f 100755 --- a/modules/check_mk.py +++ b/modules/check_mk.py @@ -1986,6 +1986,8 @@ def precompile_hostchecks(): try: precompile_hostcheck(host) except Exception, e: + if opt_debug: + raise sys.stderr.write("Error precompiling checks for host %s: %s\n" % (host, e)) sys.exit(5) @@ -2080,6 +2082,7 @@ no_inventory_possible = None if not path: raise MKGeneralException("Cannot find plugin for check type %s (missing file %s/%s)\n" % \ (checktype, checks_dir, checktype)) + if path not in filenames: filenames.append(path)

13 years, 2 months

Check_MK Git: check_mk: FIX: Display limit-exceeded message also in multi site setups

by git＠mathias-kettner.de

Module: check_mk Branch: master Commit: c13bf40f36414b96dc2f590550ec61ccbe45df2a URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=c13bf40f36414b… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Wed May 4 13:14:15 2011 +0200 FIX: Display limit-exceeded message also in multi site setups --- ChangeLog | 1 + web/htdocs/htmllib.py | 4 ++-- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/ChangeLog b/ChangeLog index d672a6a..5a56adc 100644 --- a/ChangeLog +++ b/ChangeLog @@ -22,6 +22,7 @@ * find configuration file in subdirectories of conf.d also Multisite: + * FIX: Display limit-exceeded message also in multi site setups * FIX: Tactical Overview: fix unhandled host problems view * FIX: customlinks snapin: Suppressing exception when no links configured * FIX: webservice: suppress livestatus errors in multi-site setups diff --git a/web/htdocs/htmllib.py b/web/htdocs/htmllib.py index 26ee4f2..801dc80 100644 --- a/web/htdocs/htmllib.py +++ b/web/htdocs/htmllib.py @@ -500,14 +500,14 @@ class html: def check_limit(self, rows, limit): count = len(rows) - if limit != None and count == limit + 1: + if limit != None and count >= limit + 1: text = "Your query produced more then %d results. " % limit if self.var("limit", "soft") == "soft" and config.may("ignore_soft_limit"): text += '<a href="%s">Repeat query and allow more results.</a>' % self.makeuri([("limit", "hard")]) elif self.var("limit") == "hard" and config.may("ignore_hard_limit"): text += '<a href="%s">Repeat query without limit.</a>' % self.makeuri([("limit", "none")]) self.show_warning(text) - del rows[-1] + del rows[limit:] return False return True

13 years, 2 months

Check_MK Git: check_mk: Updated bug entries

by git＠mathias-kettner.de

Module: check_mk Branch: master Commit: 33e8d5cfe596ffa2fd68c8bd7fdb5c261e69f441 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=33e8d5cfe596ff… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Wed May 4 13:07:34 2011 +0200 Updated bug entries --- .bugs/264 | 62 ++++++++++++++++++++++++++++++++---------------------------- 1 files changed, 33 insertions(+), 29 deletions(-) diff --git a/.bugs/264 b/.bugs/264 index d2d0ee3..d236250 100644 --- a/.bugs/264 +++ b/.bugs/264 @@ -57,32 +57,36 @@ a partial solution: Here an Email from Thomas Klopp: -ch hab nach einem aktuellen Windows Update ein Problem mit dem Check_mk_agent.exe unter Windows gehabt. - - - -Das Problem ist, daß bei einigen Systemen (64 Bit english Windows 2k3 ) die Services nicht mehr gestartet werden können, wenn Check_mk_agent in einem Verzeichniss mit Leerzeichen installiert worden ist. Es kommt neuerdings ein Fehler, daß der installierter Service keine gültige 32 Bit Anwendung ist. - - - -Der offizielle Workarround von Microsoft ist, den Service in Anführungszeichen zu setzten, was mit dem aktuellen Agent zur Folge hat, daß die Plugins nicht mehr ausgeführt werden. Du verwendest im Aktuellen Client den Pfad der check_mk_agent aus der Services und setzt diese für Plugins etc um. - - - -Somit hat dann jeder Pfad ein Anführungszeichen am Anfang, allerdings keines am Ende. - - - -Ich hab heute einen unserer Programmierer darauf angesetzt, da ich selbst in C++ nicht so firm bin. - - - -Er hat die angehängte Änderung durchgeführt, welche einfach Prüft, ob die Verzeichnisse am Anfang ein Anführungszeichen haben und diese ignorieren. Das war die einfachste lösung. Was jetzt jedoch ein Problem war und ist, ist das Compilieren der Exe. Die unter Linux compilierte exe hat einen fehler in der Systemzeit und die unter windows Compilierte stellt uns vor dll – Probleme… J - - - -Was wir nicht mehr machen konnten ist im Service Installationsbereich die Anführungszeichen einzufügen. (Das Compilieren hat alle Zeit aufgebraucht, die ich beim Programmierer abzwacken konnte… - - - -Ein Feature Request wäre die Übernahme der Änderungen in den nächsten Innovations build und die Anführungszeichen automatisch beim Installieren des Services. +Ich hab nach einem aktuellen Windows Update ein Problem mit dem +Check_mk_agent.exe unter Windows gehabt. + +Das Problem ist, daß bei einigen Systemen (64 Bit english Windows 2k3 ) die +Services nicht mehr gestartet werden können, wenn Check_mk_agent in einem +Verzeichniss mit Leerzeichen installiert worden ist. Es kommt neuerdings +ein Fehler, daß der installierter Service keine gültige 32 Bit Anwendung ist. + +Der offizielle Workarround von Microsoft ist, den Service in Anführungszeichen +zu setzten, was mit dem aktuellen Agent zur Folge hat, daß die Plugins +nicht mehr ausgeführt werden. Du verwendest im Aktuellen Client den Pfad +der check_mk_agent aus der Services und setzt diese für Plugins etc um. + +Somit hat dann jeder Pfad ein Anführungszeichen am Anfang, allerdings keines +am Ende. + +Ich hab heute einen unserer Programmierer darauf angesetzt, da ich selbst +in C++ nicht so firm bin. + +Er hat die angehängte Änderung durchgeführt, welche einfach Prüft, +ob die Verzeichnisse am Anfang ein Anführungszeichen haben und diese +ignorieren. Das war die einfachste lösung. Was jetzt jedoch ein Problem +war und ist, ist das Compilieren der Exe. Die unter Linux compilierte exe +hat einen fehler in der Systemzeit und die unter windows Compilierte stellt +uns vor dll – Probleme… J + +Was wir nicht mehr machen konnten ist im Service Installationsbereich die +Anführungszeichen einzufügen. (Das Compilieren hat alle Zeit aufgebraucht, +die ich beim Programmierer abzwacken konnte… + +Ein Feature Request wäre die Übernahme der Änderungen in den nächsten +Innovations build und die Anführungszeichen automatisch beim Installieren +des Services.

13 years, 2 months

Check_MK Git: check_mk: Updated bug entries

by git＠mathias-kettner.de

Module: check_mk Branch: master Commit: 0345de6682547f2065dc9e136ecbab4003f0516c URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=0345de6682547f… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Wed May 4 13:06:53 2011 +0200 Updated bug entries --- .bugs/264 | 88 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ 1 files changed, 88 insertions(+), 0 deletions(-) diff --git a/.bugs/264 b/.bugs/264 new file mode 100644 index 0000000..d2d0ee3 --- /dev/null +++ b/.bugs/264 @@ -0,0 +1,88 @@ +Title: Windows agent: handle quotes in directory names +Component: checks +Benefit: 2 +State: open +Cost: 2 +Date: 2011-05-04 13:05:20 +Class: bug + +The Windows agent has Problems in W2008 64 Bit since there +are appearing nasty quotes in the directory names. Here is +a partial solution: + + // + // Author: Bernd Murr + // + // Change for Windows Update. If there are hyphens at the Beginning of Pathvariables + // remove those. Hyphens are needed when microsoft updates change the behaviour of + // Windows Services. These Change can not start the service anymore if the path + // to the service contains spaces. One has to change the Registry values and put a + // path around the service definition. These Hyphens have to be removed right here. + // + // Date: 27.04.2011 + // + + char l_current_directory[256]; + char l_agent_directory[256]; + char l_plugins_dir[256]; + char l_local_dir[256]; + + //Modify Current_directory + if ( g_current_directory[0] == '"' ) + { + strcpy(l_current_directory, (g_current_directory+1)); + strcpy(g_current_directory, l_current_directory); + } + if ( g_agent_directory[0] == '"' ) + { + strcpy(l_agent_directory, (g_agent_directory+1)); + strcpy(g_agent_directory, l_agent_directory); + } + if ( g_plugins_dir[0] == '"' ) + { + strcpy(l_plugins_dir, (g_plugins_dir+1)); + strcpy(g_plugins_dir, l_plugins_dir); + } + if ( g_local_dir[0] == '"' ) + { + strcpy(l_local_dir, (g_local_dir+1)); + strcpy(g_local_dir, l_local_dir); + } + + //Modify + // + // End of Change for Windows 7 + // + + +Here an Email from Thomas Klopp: + +ch hab nach einem aktuellen Windows Update ein Problem mit dem Check_mk_agent.exe unter Windows gehabt. + + + +Das Problem ist, daß bei einigen Systemen (64 Bit english Windows 2k3 ) die Services nicht mehr gestartet werden können, wenn Check_mk_agent in einem Verzeichniss mit Leerzeichen installiert worden ist. Es kommt neuerdings ein Fehler, daß der installierter Service keine gültige 32 Bit Anwendung ist. + + + +Der offizielle Workarround von Microsoft ist, den Service in Anführungszeichen zu setzten, was mit dem aktuellen Agent zur Folge hat, daß die Plugins nicht mehr ausgeführt werden. Du verwendest im Aktuellen Client den Pfad der check_mk_agent aus der Services und setzt diese für Plugins etc um. + + + +Somit hat dann jeder Pfad ein Anführungszeichen am Anfang, allerdings keines am Ende. + + + +Ich hab heute einen unserer Programmierer darauf angesetzt, da ich selbst in C++ nicht so firm bin. + + + +Er hat die angehängte Änderung durchgeführt, welche einfach Prüft, ob die Verzeichnisse am Anfang ein Anführungszeichen haben und diese ignorieren. Das war die einfachste lösung. Was jetzt jedoch ein Problem war und ist, ist das Compilieren der Exe. Die unter Linux compilierte exe hat einen fehler in der Systemzeit und die unter windows Compilierte stellt uns vor dll – Probleme… J + + + +Was wir nicht mehr machen konnten ist im Service Installationsbereich die Anführungszeichen einzufügen. (Das Compilieren hat alle Zeit aufgebraucht, die ich beim Programmierer abzwacken konnte… + + + +Ein Feature Request wäre die Übernahme der Änderungen in den nächsten Innovations build und die Anführungszeichen automatisch beim Installieren des Services.

13 years, 2 months

Check_MK Git: check_mk: Updated Perl-API to 0.74

by git＠mathias-kettner.de

Module: check_mk Branch: master Commit: 3f72a0fda5a281a292794300e8701908783cd5d6 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=3f72a0fda5a281… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Wed May 4 12:46:46 2011 +0200 Updated Perl-API to 0.74 --- ChangeLog | 3 + livestatus/api/perl/Changes | 96 ++++- livestatus/api/perl/MANIFEST | 32 +- livestatus/api/perl/META.yml | 55 ++- livestatus/api/perl/Makefile.PL | 36 +- livestatus/api/perl/README | 12 +- livestatus/api/perl/examples/dump.pl | 4 +- livestatus/api/perl/examples/test.pl | 143 +++++ livestatus/api/perl/inc/Module/AutoInstall.pm | 19 +- livestatus/api/perl/inc/Module/Install.pm | 218 +++++--- .../api/perl/inc/Module/Install/AutoInstall.pm | 25 +- livestatus/api/perl/inc/Module/Install/Base.pm | 11 +- livestatus/api/perl/inc/Module/Install/Can.pm | 2 +- livestatus/api/perl/inc/Module/Install/Catalyst.pm | 290 ---------- livestatus/api/perl/inc/Module/Install/Fetch.pm | 2 +- livestatus/api/perl/inc/Module/Install/Include.pm | 2 +- livestatus/api/perl/inc/Module/Install/Makefile.pm | 229 +++++++-- livestatus/api/perl/inc/Module/Install/Metadata.pm | 267 ++++++--- livestatus/api/perl/inc/Module/Install/Scripts.pm | 29 - livestatus/api/perl/inc/Module/Install/Win32.pm | 2 +- livestatus/api/perl/inc/Module/Install/WriteAll.pm | 7 +- .../MKLivestatus.pm => Monitoring/Livestatus.pm} | 589 ++++++++++++++------ .../api/perl/lib/Monitoring/Livestatus/INET.pm | 121 ++++ .../Livestatus}/MULTI.pm | 406 +++++++++----- .../MKLivestatus => Monitoring/Livestatus}/UNIX.pm | 32 +- .../api/perl/lib/Nagios/MKLivestatus/INET.pm | 100 ---- .../perl/t/01-Monitoring-Livestatus-basic_tests.t | 149 +++++ .../perl/t/01-Nagios-MKLivestatus-basic_tests.t | 101 ---- ...nals.t => 02-Monitoring-Livestatus-internals.t} | 41 +- .../t/03-Monitoring-Livestatus-MULTI-internals.t | 215 +++++++ .../t/03-Nagios-MKLivestatus-MULTI-internals.t | 171 ------ ...et.t => 20-Monitoring-Livestatus-test_socket.t} | 107 +++-- ...atus-INET.t => 21-Monitoring-Livestatus-INET.t} | 10 +- ...atus-UNIX.t => 22-Monitoring-Livestatus-UNIX.t} | 10 +- ...test.t => 30-Monitoring-Livestatus-live-test.t} | 296 ++++++---- .../api/perl/t/30-Nagios-MKLivestatus-real_test.t | 195 ------- ... => 31-Monitoring-Livestatus-MULTI-live-test.t} | 38 +- .../perl/t/32-Monitoring-Livestatus-backend-test.t | 106 ++++ .../33-Monitoring-Livestatus-test_socket_timeout.t | 74 +++ .../perl/t/34-Monitoring-Livestatus-utf8_support.t | 78 +++ .../t/35-Monitoring-Livestatus-callbacks_support.t | 53 ++ livestatus/api/perl/t/98-Pod-Coverage.t | 3 +- livestatus/api/perl/t/99-Perl-Critic.t | 3 +- 43 files changed, 2695 insertions(+), 1687 deletions(-) Diff: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commitdiff;h=3f72a0fda5…

13 years, 2 months

Check_MK Git: check_mk: Updated bug entries

by git＠mathias-kettner.de

Module: check_mk Branch: master Commit: 791cd60feb2f2092b727e58f3bbe8c06ae15a524 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=791cd60feb2f20… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Wed May 4 12:43:11 2011 +0200 Updated bug entries --- .bugs/263 | 13 +++++++++++++ 1 files changed, 13 insertions(+), 0 deletions(-) diff --git a/.bugs/263 b/.bugs/263 new file mode 100644 index 0000000..a5c76f2 --- /dev/null +++ b/.bugs/263 @@ -0,0 +1,13 @@ +Title: Perf-O-Meter should not honor command arguments +Component: multisite +Benefit: 1 +State: open +Cost: 1 +Date: 2011-05-04 12:41:53 +Class: cleanup + +When creating perfometers for custom commands (such as legacy +checks) it would be better, if the Perf-O-Meter would not +honor the complete check command but drop everything after the +first !. check_http!www.hirni.de!123 should display the Perf-O-Meter +of check_http, without the !...

13 years, 2 months

Check_MK Git: check_mk: FIX: Tactical Overview: fix unhandled host problems view

by git＠mathias-kettner.de

Module: check_mk Branch: master Commit: ba6e8c62ac0613709f47000db82f216b34d3366a URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=ba6e8c62ac0613… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Wed May 4 12:35:55 2011 +0200 FIX: Tactical Overview: fix unhandled host problems view --- ChangeLog | 1 + web/plugins/views/builtin.py | 4 +++- 2 files changed, 4 insertions(+), 1 deletions(-) diff --git a/ChangeLog b/ChangeLog index 04fd8f0..1560dc1 100644 --- a/ChangeLog +++ b/ChangeLog @@ -22,6 +22,7 @@ * find configuration file in subdirectories of conf.d also Multisite: + * FIX: Tactical Overview: fix unhandled host problems view * FIX: customlinks snapin: Suppressing exception when no links configured * FIX: webservice: suppress livestatus errors in multi-site setups * FIX: install missing example icons in web/htdocs/images/icons diff --git a/web/plugins/views/builtin.py b/web/plugins/views/builtin.py index 87ecaae..9fff76e 100644 --- a/web/plugins/views/builtin.py +++ b/web/plugins/views/builtin.py @@ -736,6 +736,7 @@ multisite_builtin_views = { ('hst1', 'on'), ('hst2', 'on'), ('hstp', ''), + ('is_host_acknowledged', '-1'), ('host', ''), ('is_summary_host', '0'), ('opthostgroup', '')], @@ -759,7 +760,8 @@ multisite_builtin_views = { 'show_filters': ['host_in_notification_period', 'hoststate', 'hostregex', - 'opthostgroup'], + 'opthostgroup', + 'host_acknowledged'], 'sorters': [], 'title': 'Host problems', 'topic': 'Problems'},

13 years, 2 months

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

by git＠mathias-kettner.de

Module: check_mk Branch: master Commit: c4ba0e6a9793023f6dccb9117b7e7073e27b0e57 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=c4ba0e6a979302… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Wed May 4 12:37:54 2011 +0200 Merge branch 'master' of ssh://mathias-kettner.de/check_mk ---

13 years, 2 months

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

by git＠mathias-kettner.de

Module: check_mk Branch: master Commit: 25db8df3f41605156ee2cb663ea8ea28940416f5 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=25db8df3f41605… Author: Florian Heigl <fh(a)mathias-kettner.de> Date: Wed May 4 08:18:17 2011 +0200 Merge branch 'master' of ssh://mathias-kettner.de/check_mk ---

13 years, 2 months

← Newer
1
...
8
9
10
11
12
13
14
Older →

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

Checkmk git commits May 2011