Skip to content
This repository has been archived by the owner on Jan 15, 2021. It is now read-only.

do not assume ascending order #101

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
do not assume ascending order
do not assume warning and critical levels are in ascending order, eg. check_oplog:  {'warning': 36, 'critical': 24}
charnier authored Feb 20, 2019

Verified

This commit was signed with the committer’s verified signature.
ImgBotApp Imgbot
commit f35b59104d28c817ee04c3e440c3678ef92587b6
19 changes: 12 additions & 7 deletions nagios/bin/pmp-check-mongo.py
Original file line number Diff line number Diff line change
@@ -357,14 +357,19 @@ def connect(self, connectTimeout=5000):

def check_levels(self, check_result, warning_level, critical_level, message):
self.close()
if check_result < warning_level:
return "ok", message
elif check_result > critical_level:
return "critical", message
elif check_result > warning_level and check_result < critical_level:
return "warning", message
if warning_level == critical_level:
return "unknown", "Warning and critical levels must be different"
if warning_level < critical_level:
if check_result < warning_level:
return "ok", message
if check_result < critical_level:
return "warning", message
else:
return "unknown", "Unable to parse %s into a result" % check_result
if warning_level < check_result:
return "ok", message
if critical_level < check_result:
return "warning", message
return "critical", message

def check_connect(self, args, warning_level, critical_level):
warning_level = warning_level or self.get_default('check_connect', 'warning')