From 2af229206d3232b59fd52a72680614c0c4b4bb2f Mon Sep 17 00:00:00 2001 From: Sanjana Pai Date: Wed, 9 Dec 2015 12:03:26 +0530 Subject: [PATCH] Restructuring a comment in Python-Ceilometerclient The comment describes the handling of HTTPNotFound exception about alarm, but it is ambiguous, this change make it more clear. Closes-Bug:#1523866 Change-Id: Id7be71dcda6693875222435e10cba929e7c81e56 --- ceilometerclient/v2/alarms.py | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/ceilometerclient/v2/alarms.py b/ceilometerclient/v2/alarms.py index e7dfd36a..393dc094 100644 --- a/ceilometerclient/v2/alarms.py +++ b/ceilometerclient/v2/alarms.py @@ -89,11 +89,10 @@ class AlarmManager(base.Manager): return None except exc.HTTPNotFound: - # When we try to get deleted alarm HTTPNotFound occurs - # or when alarm doesn't exists this exception don't must - # go deeper because cleanUp() (method which remove all - # created things like instance, alarm, etc.) at scenario - # tests doesn't know how to process it + # When we try to get a deleted alarm, or + # when an alarm doesn't exist, HTTPNotFound exception occurs. + # Since scenario tests at the time of cleanUp() will not know + # how to handle it, we only return None. return None @classmethod