From 8f415054110857cb2dfcc0328caa8fbd138248df Mon Sep 17 00:00:00 2001 From: crucial7 Date: Fri, 11 Sep 2015 13:24:05 -0400 Subject: [PATCH] If statement wasn't checking for None correctly, which made failures "pass" when they shouldn't have. --- cabot/cabotapp/models.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cabot/cabotapp/models.py b/cabot/cabotapp/models.py index 1ae2f22..3f3cb3b 100644 --- a/cabot/cabotapp/models.py +++ b/cabot/cabotapp/models.py @@ -626,7 +626,7 @@ class GraphiteStatusCheck(StatusCheck): raise Exception(u'Check type %s not supported' % self.check_type) - if failure_value: + if not failure_value is None: failures.append(failure_value) failed = True