openstack/aodh
Code Issues Proposed changes

Fix notify method signature on LogAlarmNotifier

Browse Source 
Otherwise notification on alarms with the 'log://' style
of action will fail due to the additional self arg.

Change-Id: Iae0dbad2232a27180091f9bc4c1f69430f48b5ea
This commit is contained in:
Eoghan Glynn 2013-07-30 13:42:49 +00:00
parent c525699e9a
commit 1d9ef34006
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}