openstack
/
neutron
Code Issues Proposed changes

Merge "Downgrade callback abortable event log to debug"

This commit is contained in:
Jenkins 2017-03-21 15:41:45 +00:00 committed by Gerrit Code Review
parent a22aba547f 69d3b53fb8
commit 993725f0cc
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
neutron/callbacks/manager.py
View File

@ -155,7 +155,7 @@ class CallbacksManager(object):
{'callback': callback_id, {'callback': callback_id,
'resource': resource, 'event': event}) 'resource': resource, 'event': event})
else: else:
LOG.error(_LE("Callback %(callback)s raised %(error)s"), LOG.debug("Callback %(callback)s raised %(error)s",
{'callback': callback_id, 'error': e}) {'callback': callback_id, 'error': e})
errors.append(exceptions.NotificationError(callback_id, e)) errors.append(exceptions.NotificationError(callback_id, e))
return errors return errors

2
neutron/tests/unit/callbacks/test_manager.py
View File

@ -239,7 +239,7 @@ class CallBacksManagerTestCase(base.BaseTestCase):
self.manager._notify_loop( self.manager._notify_loop(
resources.PORT, events.PRECOMMIT_CREATE, mock.ANY) resources.PORT, events.PRECOMMIT_CREATE, mock.ANY)
self.assertFalse(_logger.exception.call_count) self.assertFalse(_logger.exception.call_count)
self.assertTrue(_logger.error.call_count) self.assertTrue(_logger.debug.call_count)
def test_object_instances_as_subscribers(self): def test_object_instances_as_subscribers(self):
"""Ensures that the manager doesn't think these are equivalent.""" """Ensures that the manager doesn't think these are equivalent."""