openstack
/
cinder
Code Issues Proposed changes

Pass an empty context to the notifier 

The notifier API expects a context as a first parameter. During the port
to oslo.messaging, a bug was introduced here and the first argument in
the notifier call was removed. This patch puts it back.

Change-Id: Id2f47757ee4af8f8c39a8a1e6b11dcad163f5b7b
Closes-bug: #1346466
This commit is contained in:
Flavio Percoco 2014-08-22 18:33:07 +02:00
parent 44fc450a8b
commit dbab8843ef
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
cinder/openstack/common/log_handler.py
View File

@ -25,5 +25,6 @@ class PublishErrorsHandler(logging.Handler):
if ('cinder.openstack.common.notifier.log_notifier' in
cfg.CONF.notification_driver):
return
rpc.get_notifier('error.publisher').info('error_notification',
dict(error=record.msg))
msg = record.getMessage()
rpc.get_notifier('error.publisher').error(None, 'error_notification',
dict(error=msg))