diff options
author | Tom Ritter <tom@ritter.vg> | 2016-12-28 14:22:11 -0500 |
---|---|---|
committer | Tom Ritter <tom@ritter.vg> | 2016-12-28 14:22:11 -0500 |
commit | 18aabef729c950dbed52163908ffeeb48e0e56a9 (patch) | |
tree | e4526246ccaf8c5f72d0870e1231d0337d03d0ef | |
parent | bbd009871af97d942365317b0ce577f19c95637d (diff) |
Check for 200 status code and set server timeout to 10
-rwxr-xr-x | jobs/HTTPServerChecker.py | 11 | ||||
-rwxr-xr-x | jobs/TCPServerChecker.py | 2 |
2 files changed, 8 insertions, 5 deletions
diff --git a/jobs/HTTPServerChecker.py b/jobs/HTTPServerChecker.py index e590486..9336588 100755 --- a/jobs/HTTPServerChecker.py +++ b/jobs/HTTPServerChecker.py @@ -28,12 +28,15 @@ class HTTPServerChecker(JobSpawner.JobSpawner): return self.failureNotificationFrequency
def execute(self):
try:
- requests.get(self.url)
- return True
+ i = requests.get(self.url)
+ if i.status_code != 200:
+ self.failuremsg = "Error hitting server " + self.url + " (Code: " + str(i.status_code) + ")"
+ else:
+ return True
except:
self.failuremsg = "Could not hit server " + self.url
- logging.warn(self.failuremsg)
- return False
+ logging.warn(self.failuremsg)
+ return False
def onFailure(self):
return self.sendEmail(self.failuremsg, "")
def onStateChangeSuccess(self):
diff --git a/jobs/TCPServerChecker.py b/jobs/TCPServerChecker.py index 78ae1ac..4b5460c 100755 --- a/jobs/TCPServerChecker.py +++ b/jobs/TCPServerChecker.py @@ -31,7 +31,7 @@ class TCPServerChecker(JobSpawner.JobSpawner): def execute(self):
try:
s = socket.socket(socket.AF_INET, socket.SOCK_STREAM)
- s.settimeout(5)
+ s.settimeout(10)
s.connect((self.ip, self.port))
s.close()
return True
|