openstack/aodh
Code Issues Proposed changes

Merge "Fix notify method signature on LogAlarmNotifier"

Browse Source
This commit is contained in:
Jenkins 2013-08-01 06:28:36 +00:00 committed by Gerrit Code Review
commit f6e8b8a613
2 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
ceilometer/alarm/notifier/log.py
View File

@ -26,6 +26,7 @@ LOG = log.getLogger(__name__)
class LogAlarmNotifier(notifier.AlarmNotifier):
"Log alarm notifier."""
@staticmethod
def notify(action, alarm, state, reason):
LOG.info("Notifying alarm %s in state %s with action %s because %s",
alarm, state, action, reason)

3
ceilometer/alarm/notifier/rest.py
View File

@ -52,7 +52,8 @@ cfg.CONF.register_opts(REST_NOTIFIER_OPTS, group="alarm")
class RestAlarmNotifier(notifier.AlarmNotifier):
"""Rest alarm notifier."""
def notify(self, action, alarm, state, reason):
@staticmethod
def notify(action, alarm, state, reason):
LOG.info("Notifying alarm %s in state %s with action %s because %s",
alarm, state, action, reason)
body = {'state': state, 'reason': reason}