From 3ef3572c7052e5c88f00cd73d0e43305caf6c465 Mon Sep 17 00:00:00 2001 From: Pradeep Kilambi Date: Tue, 22 Apr 2014 09:25:22 -0700 Subject: [PATCH] Remove conversion debug message Removing unnecessary debug message confusing the user to think the error is fatal. Instead just let it pass silently to avoid confusion. The debug message wasnt adding much value anyway. Closes-Bug: #1306280 Change-Id: I03420311f0a6ec77d68b750afce3c4110ccffecf --- ceilometer/api/controllers/v2.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/ceilometer/api/controllers/v2.py b/ceilometer/api/controllers/v2.py index ea543acf2..f30ce98e4 100644 --- a/ceilometer/api/controllers/v2.py +++ b/ceilometer/api/controllers/v2.py @@ -242,9 +242,9 @@ class Query(_Base): try: converted_value = ast.literal_eval(self.value) except (ValueError, SyntaxError): - msg = _('Failed to convert the metadata value %s' - ' automatically') % (self.value) - LOG.debug(msg) + # Unable to convert the metadata value automatically + # let it default to self.value + pass else: if type not in self._supported_types: # Types must be explicitly declared so the @@ -254,7 +254,7 @@ class Query(_Base): raise TypeError() converted_value = self._type_converters[type](self.value) except ValueError: - msg = _('Failed to convert the value %(value)s' + msg = _('Unable to convert the value %(value)s' ' to the expected data type %(type)s.') % \ {'value': self.value, 'type': type} raise ClientSideError(msg)