Checkmk git commits September 2013

checkmk-commits@lists.checkmk.com

6 participants
227 discussions

Check_MK Git: check_mk: Further speedup of agent_vsphere.fast (40% faster)

by Andreas Boesl

Module: check_mk Branch: master Commit: 1162adbc2bf3f53c076bd74f6a6edd393f40795e URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=1162adbc2bf3f5… Author: Andreas Boesl <ab(a)mathias-kettner.de> Date: Thu Sep 12 18:40:24 2013 +0200 Further speedup of agent_vsphere.fast (40% faster) now generates counter soap request dynamically --- agents/special/agent_vsphere.fast | 318 ++++++++++++++++++++++--------------- 1 file changed, 187 insertions(+), 131 deletions(-) Diff: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commitdiff;h=1162adbc2b…

11 years

Check_MK Git: check_mk: FIX: Folder Tree Snapin: make folder filter also work for remote folders

by Mathias Kettner

Module: check_mk Branch: master Commit: 4ba7915b40e3419b735b3dacc2b26ec80bda21ab URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=4ba7915b40e341… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Thu Sep 12 17:34:45 2013 +0200 FIX: Folder Tree Snapin: make folder filter also work for remote folders --- ChangeLog | 2 ++ web/plugins/views/wato.py | 2 +- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/ChangeLog b/ChangeLog index daa3f94..5873af8 100644 --- a/ChangeLog +++ b/ChangeLog @@ -357,6 +357,8 @@ * FIX: Fixed wrong default settings of view filters in localized multisite * FIX: Fixed exception when enabling sounds for views relying on e.g. alert statistics source + * FIX: Folder Tree Snapin: make folder filter also work for remote + folders that do not exist locally WATO: * FIX: fix layout of Auxiliary tags table diff --git a/web/plugins/views/wato.py b/web/plugins/views/wato.py index f53b5a3..abca215 100644 --- a/web/plugins/views/wato.py +++ b/web/plugins/views/wato.py @@ -52,7 +52,7 @@ class FilterWatoFile(Filter): def filter(self, infoname): self.check_wato_data_update() current = html.var(self.name) - if current and current in self.path_to_tree: + if current: return "Filter: host_filename ~ ^/wato/%s/\n" % current.replace("\n", "") # prevent insertions attack else: return ""

11 years

Check_MK Git: check_mk: level unknown messages are send to stdout

by Götz Golla

Module: check_mk Branch: master Commit: 48dc520c531a47ba8b2c20b31b9ff98935604577 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=48dc520c531a47… Author: Götz Golla <gg(a)mathias-kettner.de> Date: Thu Sep 12 17:31:28 2013 +0200 level unknown messages are send to stdout --- doc/treasures/active_checks/check_sql | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/doc/treasures/active_checks/check_sql b/doc/treasures/active_checks/check_sql index 0a2ce37..e97848b 100755 --- a/doc/treasures/active_checks/check_sql +++ b/doc/treasures/active_checks/check_sql @@ -75,7 +75,7 @@ try: opts, args = getopt.getopt(sys.argv[1:], short_options, long_options) except getopt.GetoptError, e: - sys.stderr.write("UNKOWN - Error in command options: %s\n" % e) + print "UNKOWN - Error in command options: %s" % e sys.exit(3) opt_debug = False @@ -116,7 +116,7 @@ try: if args: opt_sql = ' '.join(map(str,args)) else: - sys.stderr.write("UNKNOWN - no sql statement given\n") + print "UNKNOWN - no sql statement given" sys.exit(3) except Exception, e: @@ -190,10 +190,10 @@ if opt_verbose: # try: if len(rows) > 1: - sys.stderr.write("UNKNOWN - SQL statement returned more than one line\n") + print "UNKNOWN - SQL statement returned more than one line" sys.exit(3) elif len(rows) == 0: - sys.stderr.write("UNKNOWN - SQL statement returned no data\n") + print "UNKNOWN - SQL statement returned no data" sys.exit(3) row = rows[0] @@ -211,7 +211,7 @@ try: if number in [ 0 ,1 ,2 ,3 ]: state = number else: - sys.stderr.write("UNKNOWN - <%s> is not a state, and no levels given\n" % str(number)) + print "UNKNOWN - <%s> is not a state, and no levels given" % str(number) sys.exit(3) except Exception, e:

11 years

Check_MK Git: check_mk: Updated bug entries #0951

by Mathias Kettner

Module: check_mk Branch: master Commit: 170e0806ef588a75ec7113b2969cad08d96603d5 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=170e0806ef588a… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Thu Sep 12 17:28:44 2013 +0200 Updated bug entries #0951 --- .bugs/951 | 12 ++++++++++++ 1 file changed, 12 insertions(+) diff --git a/.bugs/951 b/.bugs/951 new file mode 100644 index 0000000..38ba855 --- /dev/null +++ b/.bugs/951 @@ -0,0 +1,12 @@ +Title: Folder-Tree Snapin shows foreign folders, but filter does not work +Component: multisite +State: open +Date: 2013-09-12 17:27:28 +Targetversion: 1.2.3i2 +Class: bug + +If you create a disributed monitoring then the folder tree snapin +can show folders from remote sites (without alias), which is OK. +But when you click on a folder then the filter does not work +because (i think) it does not know the folder and hence ignored +it.

11 years

Check_MK Git: check_mk: Added help text for levels in process inventory rule

by Götz Golla

Module: check_mk Branch: master Commit: 6948623fca0ae3dcc0a59c6d4ca320a4cbeb335f URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=6948623fca0ae3… Author: Götz Golla <gg(a)mathias-kettner.de> Date: Thu Sep 12 16:50:32 2013 +0200 Added help text for levels in process inventory rule --- web/plugins/wato/check_parameters.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/web/plugins/wato/check_parameters.py b/web/plugins/wato/check_parameters.py index 748d822..b367fd2 100644 --- a/web/plugins/wato/check_parameters.py +++ b/web/plugins/wato/check_parameters.py @@ -360,6 +360,8 @@ register_rule(group + '/' + subgroup_inventory, )), ('levels', Tuple( title = _('Levels'), + help = _("Please note that if you specify and also if you modify levels here, the change is activated " + "only during an inventory. Saving this rule is not enough. This is due to the nature of inventory rules."), elements = [ Integer( title = _("Critical below"),

11 years

Check_MK Git: check_mk: service description for inventory processes must not be empty since

by Götz Golla

Module: check_mk Branch: master Commit: 01cf6cec40b6e7cae217cd22096145b7e4c602b4 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=01cf6cec40b6e7… Author: Götz Golla <gg(a)mathias-kettner.de> Date: Thu Sep 12 15:11:02 2013 +0200 service description for inventory processes must not be empty since serveral rules are configurable which need to have different service descriptions. --- web/plugins/wato/check_parameters.py | 1 + 1 file changed, 1 insertion(+) diff --git a/web/plugins/wato/check_parameters.py b/web/plugins/wato/check_parameters.py index 58fe698..1596630 100644 --- a/web/plugins/wato/check_parameters.py +++ b/web/plugins/wato/check_parameters.py @@ -292,6 +292,7 @@ register_rule(group + '/' + subgroup_inventory, elements = [ ('descr', TextAscii( title = _('Service Description'), + allow_empty = False, help = _('<p>The service description may contain one or more occurances of <tt>%s</tt>. If you do this, then the pattern must be a regular ' 'expression and be prefixed with ~. For each <tt>%s</tt> in the description, the expression has to contain one "group". A group ' 'is a subexpression enclosed in brackets, for example <tt>(.*)</tt> or <tt>([a-zA-Z]+)</tt> or <tt>(...)</tt>. When the inventory finds a process '

11 years

Check_MK Git: check_mk: check_form_submit: Finished implementing checking multiple hosts ; added number of succeeded check levels

by Lars Michelsen

Module: check_mk Branch: master Commit: 2f3575bbde4cb97740d5a499d7fe6b7a737d151e URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=2f3575bbde4cb9… Author: Lars Michelsen <lm(a)mathias-kettner.de> Date: Thu Sep 12 14:35:19 2013 +0200 check_form_submit: Finished implementing checking multiple hosts ; added number of succeeded check levels --- checks/check_form_submit | 17 ++++++----- doc/treasures/active_checks/check_form_submit | 8 +++-- web/htdocs/userdb.py | 2 +- web/plugins/wato/active_checks.py | 40 ++++++++++++------------- 4 files changed, 35 insertions(+), 32 deletions(-) diff --git a/checks/check_form_submit b/checks/check_form_submit index 6a76dfa..ee69015 100644 --- a/checks/check_form_submit +++ b/checks/check_form_submit @@ -29,12 +29,7 @@ def check_form_submit_arguments(params): args = '' if "virthost" in settings: - vhost, omit_ip = settings["virthost"] - args += ' -H %s' % quote_shell_string(vhost) - if omit_ip: - args += ' -I %s' % quote_shell_string(vhost) - else: - omit_ip = False + args += ' -H %s' % quote_shell_string(settings["virthost"]) if "port" in settings: args += ' -p %d' % settings["port"] @@ -57,8 +52,14 @@ def check_form_submit_arguments(params): if "query" in settings: args += ' -q %s' % quote_shell_string(settings["query"]) - if not omit_ip: - args = " -I $HOSTADDRESS$" + args + if "num_succeeded" in settings: + args += ' -n %s' % ','.join(map(str, settings["num_succeeded"])) + + if not settings.get('hosts'): + args += " -I $HOSTADDRESS$" + else: + for address in settings['hosts']: + args += " -I %s" % quote_shell_string(address) return args diff --git a/doc/treasures/active_checks/check_form_submit b/doc/treasures/active_checks/check_form_submit index dcd3858..cb69f80 100755 --- a/doc/treasures/active_checks/check_form_submit +++ b/doc/treasures/active_checks/check_form_submit @@ -48,7 +48,7 @@ # # ./check_form_submit -I localhost -u /event -q '_origtarget=wato.py&_username=omdadmin&_password=omd' -e 'WATO' -import sys, getopt, urllib2, urllib, re +import sys, getopt, urllib2, urllib, re, socket def usage(): sys.stderr.write(""" @@ -200,6 +200,8 @@ def open_url(url, method = 'GET', data = None): new_state(2, 'Unable to open %s: [%d] %s' % (url, e.code, e.reason)) except urllib2.URLError, e: new_state(2, 'Unable to open %s: %s' % (url, e.reason)) + except socket.timeout, e: + new_state(2, 'Unable to open %s: %s' % (url, e)) real_url = fd.geturl() code = fd.getcode() content = fd.read() @@ -324,10 +326,10 @@ try: # use the worst state as summary state sum_state = max_state - elif num_crit is not None and len(failed) >= num_crit: + elif num_crit is not None and len(success) <= num_crit: sum_state = 2 - elif num_warn is not None and len(failed) >= num_warn: + elif num_warn is not None and len(success) <= num_warn: sum_state = 1 txt = '%d succeeded, %d failed' % (len(success), len(failed)) diff --git a/web/htdocs/userdb.py b/web/htdocs/userdb.py index 2e9b55e..13281b9 100644 --- a/web/htdocs/userdb.py +++ b/web/htdocs/userdb.py @@ -302,7 +302,7 @@ def load_users(lock = False): id = d # read special values from own files - for val, conv_func in [ ('num_failed', int), ('last_seen', float) ]: + for val, conv_func in [ ('num_failed', saveint), ('last_seen', savefloat) ]: if id in result: try: result[id][val] = conv_func(file(dir + d + '/' + val + '.mk').read().strip()) diff --git a/web/plugins/wato/active_checks.py b/web/plugins/wato/active_checks.py index f8dfe06..449693a 100644 --- a/web/plugins/wato/active_checks.py +++ b/web/plugins/wato/active_checks.py @@ -928,27 +928,20 @@ register_rule(group, Dictionary( title = _("Check the URL"), elements = [ - ("virthost", Tuple( + ("hosts", ListOfStrings( + title = _('Check specific host(s)'), + help = _('By default, if you do not specify any host addresses here, ' + 'the host address of the host this service is assigned to will ' + 'be used. But by specifying one or several host addresses here, ' + 'it is possible to let the check monitor one or multiple hosts.') + )), + ("virthost", TextAscii( title = _("Virtual host"), - elements = [ - TextAscii( - title = _("Name of the virtual host"), - help = _("Set this in order to specify the name of the " - "virtual host for the query (using HTTP/1.1). When you " - "leave this empty, then the IP address of the host " - "will be used instead."), - allow_empty = False - ), - Checkbox( - label = _("Omit specifying an IP address"), - help = _("Usually Check_MK will nail this check to the " - "IP address of the host it is attached to. With this " - "option you can have the check use the name of the " - "virtual host instead and do a dynamic DNS lookup."), - true_label = _("omit IP address"), - false_label = _("specify IP address"), - ), - ] + help = _("Set this in order to specify the name of the " + "virtual host for the query (using HTTP/1.1). When you " + "leave this empty, then the IP address of the host " + "will be used instead."), + allow_empty = False, )), ("uri", TextAscii( title = _("URI to fetch (default is <tt>/</tt>)"), @@ -990,6 +983,13 @@ register_rule(group, "is URL-encoded (for example \"key1=val1&key2=val2\")."), size = 40, )), + ("num_succeeded", Tuple( + title = _("Multiple Hosts: Number of successful results"), + elements = [ + Integer(title = _("Warning if equal or below")), + Integer(title = _("Critical if equal or below")), + ] + )), ] ), ]

11 years

Check_MK Git: check_mk: cisco_wlc:

by Andreas Boesl

Module: check_mk Branch: master Commit: 97ee4e048d1ddf66593d901b59a85368305a5dfb URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=97ee4e048d1ddf… Author: Andreas Boesl <ab(a)mathias-kettner.de> Date: Thu Sep 12 13:30:29 2013 +0200 cisco_wlc: new rule group cisco_wlc: allows to configure handling of missing AP --- ChangeLog | 2 ++ checkman/cisco_wlc | 18 +++++++++++++++++ checks/cisco_wlc | 24 +++++++++++++++-------- web/plugins/wato/check_parameters.py | 36 ++++++++++++++++++++++++++++++++++ 4 files changed, 72 insertions(+), 8 deletions(-) diff --git a/ChangeLog b/ChangeLog index f8dac3e..daa3f94 100644 --- a/ChangeLog +++ b/ChangeLog @@ -86,6 +86,7 @@ * ADD: etherbox: new check for etherbox (messpc) sensors. currently supported: temperature, humidity, switch contact and smoke sensors * cisco_wlc_client: now supports low/high warn and crit levels + * cisco_wlc: now supports configuration options for missing AP Notifications: @@ -185,6 +186,7 @@ * ADD: Rule to check state of physical switch (currently used by etherbox check) * FIX: Cisco ruleset "Cisco WLC WiFi client connections" has been generalized to "WLC WiFi client connections" + * ADD: new ruleset cisco_wlc: Allows to configure handling of missing AP Notifications: * Flexible Notification can now filter service levels diff --git a/checkman/cisco_wlc b/checkman/cisco_wlc index 9fdb0c6..300270f 100644 --- a/checkman/cisco_wlc +++ b/checkman/cisco_wlc @@ -11,3 +11,21 @@ description: inventory: One check for each online accesspoint is created + +examples: + checkgroup_parameters.setdefault('cisco_wlc', []) + + checkgroup_parameters['cisco_wlc'] = [ + ( {'ap_name': [('HOMEOFFICE', 0), ('FALLBACK', 2)], 'ap_model': [('K7-B30329', 3)]}, [], ALL_HOSTS, ALL_SERVICES ), + ] + checkgroup_parameters['cisco_wlc'] + + +[parameters] +parameters(dict): with the keys "ap_model" and "ap_name" + + The values of in this dict are list of tuples. Each tuple contain + eiter (ap_name(string), action(int)) or (ap_model(string), action(int)), + whereas the ap_name is the human readable name of the AP and the ap_model is the model number + The action represent the outcome when this model is missing and can be + set from 0 to 3 (OK, WARN, CRIT, UNKNOWN) + diff --git a/checks/cisco_wlc b/checks/cisco_wlc index c0bdbde..77208a3 100644 --- a/checks/cisco_wlc +++ b/checks/cisco_wlc @@ -24,27 +24,22 @@ # to the Free Software Foundation, Inc., 51 Franklin St, Fifth Floor, # Boston, MA 02110-1301 USA. -# Example output from agent: -# Put here the example output from your TCP-Based agent. If the -# check is SNMP-Based, then remove this section - - # States: # 1: OK, 2: Crit, 3: Warn def cisco_wlc_convert(info): data = [] for i in range(0, len(info[0])): - data.append((info[0][i][1], info[1][i][1], info[2][i][1] )) + data.append((info[0][i][1], info[1][i][1], info[2][i][1], info[3][i][1] )) return data def inventory_cisco_wlc(info): data = cisco_wlc_convert(info) return [ (x[0], None) for x in data ] -def check_cisco_wlc(item, _no_params, info): +def check_cisco_wlc(item, params, info): data = cisco_wlc_convert(info) - for name, mac, state in data: + for name, mac, state, model in data: if name == item: state = saveint(state) if state == 1: @@ -53,17 +48,30 @@ def check_cisco_wlc(item, _no_params, info): return 1, "Accesspoint state Warning" if state == 2: return 2, "State Critical" + + # Special treatment if this device is missing + if params: + for name, mac, state, model in data: + for ap_name, ap_state in params.get("ap_name", []): + if name.startswith(ap_name): + return ap_state, "Accesspoint not found (State set to %s by rule)" % nagios_state_names[ap_state] + for ap_model, ap_state in params.get("ap_model", []): + if mode.startswith(ap_model): + return ap_state, "Accesspoint not found (State set to %s by rule)" % nagios_state_names[ap_state] + return 2, "Accesspoint not found" check_info["cisco_wlc"] = { "check_function" : check_cisco_wlc, "inventory_function" : inventory_cisco_wlc, + "group" : "cisco_wlc", "service_description" : "AP %s", "has_perfdata" : False, "snmp_scan_function" : lambda oid: oid('.1.3.6.1.2.1.1.2.0') in [".1.3.6.1.4.1.9.1.1069"], "snmp_info" : [( ".1.3.6.1.4.1.14179.2.2.1.1.3", [ OID_END, '' ]), ( ".1.3.6.1.4.1.14179.2.2.1.1.1", [ OID_END, '' ]), ( ".1.3.6.1.4.1.14179.2.2.1.1.6", [ OID_END, '' ]), + ( ".1.3.6.1.4.1.14179.2.2.1.1.16", [ OID_END, '' ]), ], } diff --git a/web/plugins/wato/check_parameters.py b/web/plugins/wato/check_parameters.py index 58fe698..7e05a33 100644 --- a/web/plugins/wato/check_parameters.py +++ b/web/plugins/wato/check_parameters.py @@ -1120,6 +1120,42 @@ register_check_parameters( ) register_check_parameters( + subgroup_networking, + "cisco_wlc", + _("Cisco WLAN AP"), + Dictionary( + help = _("Here you can set which alert type is set when the given " + "access point is missing (might be powered off). The access point " + "can be specified by the AP name or the AP model"), + elements = [ + ( "ap_name", + ListOf( + Tuple( + elements = [ + TextAscii(title = _("AP name")), + MonitoringState( title=_("State when missing"), default_value = 2) + ] + ), + title = _("Access point name"), + add_label = _("Add name")) + ), + ( "ap_model", + ListOf( + Tuple( + elements = [ + TextAscii(title = _("AP model")), + MonitoringState( title=_("State when missing"), default_value = 2) + ] + ), + title = _("Access point device id"), + add_label = _("Add id")) + ) + ] + ), + TextAscii(title = _("Access Point")), + "first", +) +register_check_parameters( subgroup_networking, "tcp_conn_stats", ("TCP connection stats (LINUX / UNIX)"),

11 years

Check_MK Git: check_mk: initial git version of sql check

by Götz Golla

Module: check_mk Branch: master Commit: a222fd571dc99432b048842cd80bb1c1351fc821 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=a222fd571dc994… Author: Götz Golla <gg(a)mathias-kettner.de> Date: Thu Sep 12 13:07:46 2013 +0200 initial git version of sql check --- doc/treasures/active_checks/check_sql | 226 +++++++++++++++++++++++++++++++++ 1 file changed, 226 insertions(+) diff --git a/doc/treasures/active_checks/check_sql b/doc/treasures/active_checks/check_sql new file mode 100755 index 0000000..0a2ce37 --- /dev/null +++ b/doc/treasures/active_checks/check_sql @@ -0,0 +1,226 @@ +#!/usr/bin/python +# -*- encoding: utf-8; py-indent-offset: 4 -*- +# +------------------------------------------------------------------+ +# | ____ _ _ __ __ _ __ | +# | / ___| |__ ___ ___| | __ | \/ | |/ / | +# | | | | '_ \ / _ \/ __| |/ / | |\/| | ' / | +# | | |___| | | | __/ (__| < | | | | . \ | +# | \____|_| |_|\___|\___|_|\_\___|_| |_|_|\_\ | +# | | +# | Copyright Mathias Kettner 2013 mk(a)mathias-kettner.de | +# +------------------------------------------------------------------+ +# +# This file is part of Check_MK. +# The official homepage is at http://mathias-kettner.de/check_mk. +# +# check_mk is free software; you can redistribute it and/or modify it +# under the terms of the GNU General Public License as published by +# the Free Software Foundation in version 2. check_mk is distributed +# in the hope that it will be useful, but WITHOUT ANY WARRANTY; with- +# out even the implied warranty of MERCHANTABILITY or FITNESS FOR A +# PARTICULAR PURPOSE. See the GNU General Public License for more de- +# ails. You should have received a copy of the GNU General Public +# License along with GNU Make; see the file COPYING. If not, write +# to the Free Software Foundation, Inc., 51 Franklin St, Fifth Floor, +# Boston, MA 02110-1301 USA. + + +import sys, getopt + +def usage(): + sys.stderr.write("""Check_MK SQL Test + +USAGE: check-sql [OPTIONS] SQL-Statement + check-sql -h + +ARGUMENTS: + SQL-Statement Valid SQL-Statement for the selected database + The statement must return at least a number and a string, + plus optional performance data. + +OPTIONS: + -h, --help Show this help message and exit + -u USER, --user USER Username for database access + -p PASS, --password PASS Password for database access + -d DBMS, --dbms DBMS Name of the database management system. + Default is 'postgres', other valid values are + 'mysql' and 'oracle' + -H HOST , --hostname HOST Hostname or IP-Address where the database lives. Default is '127.0.0.1' + -n NAME, --name NAME Name of the database on the DBMS + -w RANGE, --warning RANGE lower and upper level for the warning state + -c RANGE, --critical RANGE lower and upper level for the critical state + --debug Debug mode: let Python exceptions come through + -v, --verbose Verbose mode: print sql statement and levels + +""") + +def saverfloat(f): + if f == "": + return None + else: + return float(f) + + + +# +# defining and collection input parameters +# +short_options = 'hu:p:d:n:H:w:c:v' +long_options = [ + 'help', 'user=', 'password=', 'dbms=', 'name=', 'warning=', 'critical=', 'hostname=', + 'debug', 'verbose' +] + +try: + opts, args = getopt.getopt(sys.argv[1:], short_options, long_options) + +except getopt.GetoptError, e: + sys.stderr.write("UNKOWN - Error in command options: %s\n" % e) + sys.exit(3) + +opt_debug = False +opt_verbose = False +opt_dbms = "postgres" +opt_name = None +opt_user = opt_dbms +opt_password = None +opt_hostname = "127.0.0.1" +opt_sql = None +warn = [ None, None ] +crit = [ None, None ] + +try: + for o,a in opts: + if o in [ '--debug' ]: + opt_debug = True + elif o in [ '-v', '--verbose' ]: + opt_verbose = True + elif o in [ '-d', '--dbms' ]: + opt_dbms = a + elif o in [ '-n', '--name' ]: + opt_name = a + elif o in [ '-u', '--user' ]: + opt_user = a + elif o in [ '-p', '--password' ]: + opt_password = a + elif o in [ '-H', '--hostname' ]: + opt_hostname = a + elif o in [ '-w', '--warning' ]: + warn = map(saverfloat,a.split(":",2)) + elif o in [ '-c', '--critical' ]: + crit = map(saverfloat,a.split(":",2)) + elif o in [ '-h', '--help' ]: + usage() + sys.exit(0) + + if args: + opt_sql = ' '.join(map(str,args)) + else: + sys.stderr.write("UNKNOWN - no sql statement given\n") + sys.exit(3) + +except Exception, e: + if opt_debug: + raise + print "UNKNOWN - Error while processing input: %s" % e + sys.exit(3) + + +# +# database connection +# +try: + + if opt_dbms == "postgres": + import psycopg2 # postgres driver + db = psycopg2 + db_connection = db.connect(host=opt_hostname, database=opt_name, \ + user=opt_user, password=opt_password ) + + elif opt_dbms == "mysql": + import MySQLdb # mysql driver + db = MySQLdb + db_connection = db.connect(host=opt_hostname, db=opt_name, \ + user=opt_user, passwd=opt_password ) + + elif opt_dbms == "odbc": + import pyodbc # odbc driver + db = odbc + db_connection = db.connect(DRIVER=opt_dbms, SERVER=opt_hostname, DATABASE=opt_name, \ + UID=opt_user, PWD=opt_password, CHARSET='UTF8' ) + + else: + print "UNKNOWN - dbms not found" + sys.exit(3) + + cursor = db_connection.cursor() + +except Exception, e: + if len(e.args) > 1: + print "UNKNOWN - Error while connecting to database: [%d] %s" % (e.args[0], e.args[1]) + else: + print "UNKNOWN - Error while connecting to database: %s" % e + sys.exit(3) + +# +# excecuting the sql statement +# +try: + if opt_verbose: + sys.stderr.write("\nSQL Statement:\n %s" % opt_sql) + cursor.execute(opt_sql) + rows = cursor.fetchmany(2) + cursor.close() + db_connection.close() + +except Exception, e: + if opt_debug: + raise + if len(e.args) > 1: + print "UNKNOWN - Error while executing SQL statement: [%d] %s" % (e.args[0], e.args[1]) + else: + print "UNKNOWN - Error while executing SQL statement: %s" % e + sys.exit(3) + +if opt_verbose: + sys.stderr.write("\nSQL Result:\n %s\n\n" % rows) + +# +# processing of result +# +try: + if len(rows) > 1: + sys.stderr.write("UNKNOWN - SQL statement returned more than one line\n") + sys.exit(3) + elif len(rows) == 0: + sys.stderr.write("UNKNOWN - SQL statement returned no data\n") + sys.exit(3) + + row = rows[0] + number = float(row[0]) + text = str(row[1]) + state = 0 + if warn != [ None, None] or crit != [ None, None ]: + if (warn[0] != None and warn[0] > number) or \ + (warn[1] != None and warn[1] < number): + state = 1 + if (crit[0] != None and crit[0] > number) or \ + (crit[1] != None and crit[1] < number): + state = 2 + else: + if number in [ 0 ,1 ,2 ,3 ]: + state = number + else: + sys.stderr.write("UNKNOWN - <%s> is not a state, and no levels given\n" % str(number)) + sys.exit(3) + +except Exception, e: + if opt_debug: + raise + print "UNKNOWN - Error while processing result of SQL statement: %s" % e + sys.exit(3) + +result = ["OK", "WARN", "CRIT", "UNKNOWN" ][state] +print "%s - %s %s" % (result, number, text) +sys.exit(state) +

11 years

Check_MK Git: check_mk: cisco_wlc_clients:

by Andreas Boesl

Module: check_mk Branch: master Commit: 25de1be817f5b58bbad9e4d3f01e9bfda35470bc URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=25de1be817f5b5… Author: Andreas Boesl <ab(a)mathias-kettner.de> Date: Thu Sep 12 13:01:57 2013 +0200 cisco_wlc_clients: Parameters increased to (crit_low, crit_warn, high_warn, high_crit) not compatible to older versions! (check first introduced 3 weeks ago) Add missing check group for cisco_wlc_client Fixed unused rule group cisco_wlc_clients -> wlc_clients Updated manpage Added pnp-template --- ChangeLog | 3 ++ checkman/cisco_wlc_clients | 6 ++-- checks/cisco_wlc_clients | 21 +++++++++----- pnp-templates/check_mk-cisco_wlc_clients.php | 39 ++++++++++++++++++++++++++ web/plugins/wato/check_parameters.py | 12 ++++---- 5 files changed, 67 insertions(+), 14 deletions(-) diff --git a/ChangeLog b/ChangeLog index efbd743..f8dac3e 100644 --- a/ChangeLog +++ b/ChangeLog @@ -85,6 +85,7 @@ is now possible using WATO * ADD: etherbox: new check for etherbox (messpc) sensors. currently supported: temperature, humidity, switch contact and smoke sensors + * cisco_wlc_client: now supports low/high warn and crit levels Notifications: @@ -182,6 +183,8 @@ * FIX: automation snmp scan now adhere rules for shoddy snmp devices which have no sys description * ADD: Rule to check state of physical switch (currently used by etherbox check) + * FIX: Cisco ruleset "Cisco WLC WiFi client connections" has been generalized to + "WLC WiFi client connections" Notifications: * Flexible Notification can now filter service levels diff --git a/checkman/cisco_wlc_clients b/checkman/cisco_wlc_clients index 4e05f6c..4ae5418 100644 --- a/checkman/cisco_wlc_clients +++ b/checkman/cisco_wlc_clients @@ -19,5 +19,7 @@ inventory: One check for each wifi network, and one summary check will be created [parameters] -warn(int): Optional warning level for the number of clients -crit(int): Critical level for the number of clients +crit_low(int): Critical level for minimum number of clients +warn_low(int): Warning level for minimum number of clients +warn_high(int): Warning level for maximum number of clients +crit_high(int): Critical level for maximum number of clients diff --git a/checks/cisco_wlc_clients b/checks/cisco_wlc_clients index 01fd893..dd4d72c 100644 --- a/checks/cisco_wlc_clients +++ b/checks/cisco_wlc_clients @@ -29,10 +29,9 @@ def inventory_cisco_wlc_clients(info): if len(info) > 0: - return [ ('Summary', (None, None) ) ] + [ (x[0], (None, None)) for x in info ] + return [ ('Summary', None ) ] + [ (x[0], None) for x in info ] def check_cisco_wlc_clients(item, params, info): - warn, crit = params state = 0 found = False for line in info: @@ -51,11 +50,18 @@ def check_cisco_wlc_clients(item, params, info): break if found: - if crit and num_of_clients > crit: - state = 2 - elif warn and num_of_clients > warn: - state = 1 - perf = [ ("clients", num_of_clients, warn, crit ) ] + if params: + crit_low, warn_low, warn_high, crit_high = params + if num_of_clients < crit_low: + state = 2 + elif num_of_clients < warn_low: + state = 1 + elif num_of_clients > crit_high: + state = 2 + elif num_of_clients > warn_high: + state = 1 + + perf = [ ("clients", num_of_clients, None, None) ] return state, "%d connections (%s)" % (num_of_clients, interface), perf return 3, "WiFi not found" @@ -64,6 +70,7 @@ check_info["cisco_wlc_clients"] = { "check_function" : check_cisco_wlc_clients, "inventory_function" : inventory_cisco_wlc_clients, "service_description" : "Clients", + "group" : "wlc_clients", "has_perfdata" : True, "snmp_scan_function" : lambda oid: oid('.1.3.6.1.2.1.1.2.0') in [".1.3.6.1.4.1.9.1.1069"], "snmp_info" : ( ".1.3.6.1.4.1.14179.2.1.1.1", [ 2, 42, 38 ]), diff --git a/pnp-templates/check_mk-cisco_wlc_clients.php b/pnp-templates/check_mk-cisco_wlc_clients.php new file mode 100644 index 0000000..9c0c848 --- /dev/null +++ b/pnp-templates/check_mk-cisco_wlc_clients.php @@ -0,0 +1,39 @@ +<?php +# +------------------------------------------------------------------+ +# | ____ _ _ __ __ _ __ | +# | / ___| |__ ___ ___| | __ | \/ | |/ / | +# | | | | '_ \ / _ \/ __| |/ / | |\/| | ' / | +# | | |___| | | | __/ (__| < | | | | . \ | +# | \____|_| |_|\___|\___|_|\_\___|_| |_|_|\_\ | +# | | +# | Copyright Mathias Kettner 2013 mk(a)mathias-kettner.de | +# +------------------------------------------------------------------+ +# +# This file is part of Check_MK. +# The official homepage is at http://mathias-kettner.de/check_mk. +# +# check_mk is free software; you can redistribute it and/or modify it +# under the terms of the GNU General Public License as published by +# the Free Software Foundation in version 2. check_mk is distributed +# in the hope that it will be useful, but WITHOUT ANY WARRANTY; with- +# out even the implied warranty of MERCHANTABILITY or FITNESS FOR A +# PARTICULAR PURPOSE. See the GNU General Public License for more de- +# ails. You should have received a copy of the GNU General Public +# License along with GNU Make; see the file COPYING. If not, write +# to the Free Software Foundation, Inc., 51 Franklin St, Fifth Floor, +# Boston, MA 02110-1301 USA. + + +$opt[1] = "--vertical-label \"Connections\" --title \"$servicedesc\" "; + +$def[1] = "DEF:var1=$RRDFILE[1]:$DS[1]:MAX "; +$def[1] .= "LINE2:var1#2080ff:\"Connections\:\" "; +$def[1] .= "GPRINT:var1:LAST:\"%5.1lf%%\" "; +$def[1] .= "GPRINT:var1:AVERAGE:\"(Avg\: %5.1lf%%,\" "; +$def[1] .= "GPRINT:var1:MIN:\"Min\: %5.1lf%%,\" "; +$def[1] .= "GPRINT:var1:MAX:\"Max\: %5.1lf%%)\" "; +$def[1] .= "HRULE:$CRIT[1]#FFFF00 "; +$def[1] .= "HRULE:$WARN[1]#FF0000 "; +$def[1] .= "HRULE:$MIN[1]#FFFF00 "; +$def[1] .= "HRULE:$MAX[1]#FF0000 "; +?> diff --git a/web/plugins/wato/check_parameters.py b/web/plugins/wato/check_parameters.py index 8d1a6fc..58fe698 100644 --- a/web/plugins/wato/check_parameters.py +++ b/web/plugins/wato/check_parameters.py @@ -1103,14 +1103,16 @@ register_check_parameters( register_check_parameters( subgroup_networking, - "cisco_wlc_clients", - _("Cisco WLC WiFi client connections"), + "wlc_clients", + _("WLC WiFi client connections"), Tuple( title = _("Number of connections"), - help = _("Number of connections for a WiFi "), + help = _("Number of connections for a WiFi"), elements = [ - Integer(title = _("Warning if above"), label = _("connections")), - Integer(title = _("Critical if above"), label = _("connections")) + Integer(title = _("Critical if below"), unit=_("connections")), + Integer(title = _("Warning if below"), unit=_("connections")), + Integer(title = _("Warning if above"), unit=_("connections")), + Integer(title = _("Critical if above"), unit=_("connections")), ] ), TextAscii( title = _("Name of Wifi")),

11 years

← Newer
1
...
13
14
15
16
17
18
19
...
23
Older →

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

Checkmk git commits September 2013