Module: check_mk
Branch: master
Commit: 4c1f2890eb220b937af897f7501c3e7602a28789
URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=4c1f2890eb220b…
Author: Lars Michelsen <lm(a)mathias-kettner.de>
Date: Tue Nov 16 08:10:47 2010 +0100
Fixed some global vars after reports of validate_checks
---
checks/blade_bays | 4 ++--
checks/heartbeat_crm | 5 ++---
doc/helpers/validate_checks | 2 +-
3 files changed, 5 insertions(+), 6 deletions(-)
diff --git a/checks/blade_bays b/checks/blade_bays
index 80413da..672f423 100644
--- a/checks/blade_bays
+++ b/checks/blade_bays
@@ -24,7 +24,7 @@
# to the Free Software Foundation, Inc., 51 Franklin St, Fifth Floor,
# Boston, MA 02110-1301 USA.
-blade_bay_module_state = {
+blade_bays_module_state = {
0: 'standby',
1: 'on',
2: 'notPresent',
@@ -42,7 +42,7 @@ def check_blade_bays(item, params, info):
type = line[2]
if state == 1:
return (0, "OK - State %s (Type: %s, ID: %s)" %
- (blade_bay_module_state.get(state, 'Unhandled'), type, line[3]))
+ (blade_bays_module_state.get(state, 'Unhandled'), type, line[3]))
elif state == 2:
return (1, "WARNING - Not present")
elif state == 0:
diff --git a/checks/heartbeat_crm b/checks/heartbeat_crm
index 7e74547..566b731 100644
--- a/checks/heartbeat_crm
+++ b/checks/heartbeat_crm
@@ -44,8 +44,7 @@
# resource_slapmaster (ocf::heartbeat:OpenLDAP): Started mwp
# resource_slapslave (ocf::heartbeat:OpenLDAP): Started smwp
-from datetime import datetime, timedelta
-import time
+import time, datetime
# Nails down the DC to the node which is the DC during inventory. The check
# will report CRITICAL when another node becomes the DC during later checks.
@@ -98,7 +97,7 @@ def check_heartbeat_crm(item, params, info):
# Check the freshness of the crm_mon output and terminate with CRITICAL
# when too old information are found
- dt = int(datetime(*time.strptime(lastUpdated, '%a %b %d %H:%M:%S %Y')[0:5]).strftime("%s"))
+ dt = int(datetime.datetime(*time.strptime(lastUpdated, '%a %b %d %H:%M:%S %Y')[0:5]).strftime("%s"))
now = time.time()
delta = now - dt
if delta > param_max_age:
diff --git a/doc/helpers/validate_checks b/doc/helpers/validate_checks
index 5850bd0..0d2ab6c 100755
--- a/doc/helpers/validate_checks
+++ b/doc/helpers/validate_checks
@@ -110,7 +110,7 @@ def usage():
print "No real help available... The only option is -v|--verbose."
sys.exit(0)
-ignored_variables = []
+ignored_variables = [ 'datetime', ]
# Load all checks and record global var definitions
# Also read the man pages
Module: check_mk
Branch: master
Commit: cb660c13fa81d3ec4056bb5d9a832b678db803c3
URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=cb660c13fa81d3…
Author: Mathias Kettner <mk(a)mathias-kettner.de>
Date: Sun Nov 14 09:36:26 2010 +0100
if/if64: allow to ignore target speed
---
ChangeLog | 1 +
checkman/if | 3 ++-
checks/if | 7 +++----
checks/if64 | 2 +-
4 files changed, 7 insertions(+), 6 deletions(-)
diff --git a/ChangeLog b/ChangeLog
index 7c18255..7c08d82 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -42,6 +42,7 @@
Checks & Agents:
* megaraid_pdisks: Using the real enclosure number as check item now
* if/if64: Added expected interface speed to warning output
+ * if/if64: Allow to ignore speed setting (set target speed to None)
* wut_webtherm: handle more variants of WuT Webtherms (thanks to Lefty)
* cisco_fan: Does not inventorize 'notPresent' sensors anymore. Improved output
* cisco_power: Not using power source as threshold anymore. Improved output
diff --git a/checkman/if b/checkman/if
index 271b9ee..9e04cbf 100644
--- a/checkman/if
+++ b/checkman/if
@@ -42,7 +42,8 @@ err_warn (float): The percentage of errors + discards at which a warning should
triggered. Default is {0.01} percent (one of 10.000).
err_crit (float): The percentage of errors + discards at which a critical state is
reached. Default is {0.1} percent (one of 1.000).
-targetspeed (int): The target speed in bits per second of the interface.
+targetspeed (int): The target speed in bits per second of the interface. You can set this to {None} if you
+ do not want to check for a certain speed setting.
bw_warn (float): Optional: The in/out rate to raise a WARNING state. Defaults to {None}.
bw_crit (float): Optional: The in/out rate to raise a CRITICAL state. Defaults to {None}.
allowed_operstates (tuple): Optional: A list of allowed operstates. All other states
diff --git a/checks/if b/checks/if
index e08c247..fe903bc 100644
--- a/checks/if
+++ b/checks/if
@@ -108,12 +108,11 @@ def check_if(item, params, info):
# Check speed
speed = saveint(ifSpeed)
+ infotext += get_nic_speed_human_readable(speed)
bandwidth = speed / 8.0 # in Bytes / sec
- if speed != targetspeed:
- infotext += "%s (wrong speed! expected: %s)" % (speed, get_nic_speed_human_readable(targetspeed))
+ if not targetspeed is None and speed != targetspeed:
+ infotext += " (wrong speed! expected: %s)" % get_nic_speed_human_readable(targetspeed)
state = 1
- else:
- infotext += get_bytes_human_readable(speed, 1000)+'it/s'
# Performance counters
this_time = time.time()
diff --git a/checks/if64 b/checks/if64
index 897787b..72be2eb 100644
--- a/checks/if64
+++ b/checks/if64
@@ -122,7 +122,7 @@ def check_if64(item, params, info):
speed = saveint(ifSpeed)
infotext += get_nic_speed_human_readable(speed)
bandwidth = speed / 8.0 # in Bytes / sec
- if speed != targetspeed:
+ if not targetspeed is None and speed != targetspeed:
infotext += " (wrong speed! expected: %s)" % get_nic_speed_human_readable(targetspeed)
state = 1