alarm-notifier log action splits reason reason string

[Eoghan Glynn]

The 'log://' action split the reason string into a list of separate characters, e.g.

2013-09-19 13:19:36.116 30090 INFO ceilometer.alarm.notifier.log [req-82e0361b-5e03-477e-8b6c-47914de5bdce None None] Notifying alarm 73f90998-05ba-4db6-9337-9a63c37a05d1 from ok to alarm with action SplitResult(scheme=u'log', netloc=u'', path=u'', query='', fragment='') because [u'T', u'r', u'a', u'n', u's', u'i', u't', u'i', u'o', u'n', u' ', u't', u'o', u' ', u'a', u'l', u'a', u'r', u'm', u' ', u'd', u'u', u'e', u' ', u't', u'o', u' ', u'1', u' ', u's', u'a', u'm', u'p', u'l', u'e', u's', u' ', u'o', u'u', u't', u's', u'i', u'd', u'e', u' ', u't', u'h', u'r', u'e', u's', u'h', u'o', u'l', u'd', u',', u' ', u'm', u'o', u's', u't', u' ', u'r', u'e', u'c', u'e', u'n', u't', u':', u' ', u'1', u'3', u'.', u'8', u'1', u'3', u'5', u'5', u'9', u'3', u'2', u'2']

This results from the reason being passed as ceilometer.openstack.common.gettextutils.Message, as opposed to its string representation.

Whereas the log message should look more like:

2013-09-19 13:32:14.081 30090 INFO ceilometer.alarm.notifier.log [req-281c2b58-5ffe-48d8-a354-3c631291a68e None None] Notifying alarm 73f90998-05ba-4db6-9337-9a63c37a05d1 from ok to alarm with action SplitResult(scheme=u'log', netloc=u'', path=u'', query='', fragment='') because Transition to alarm due to 1 samples outside threshold, most recent: 14.5666666667

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/47379

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/47379
Committed: http://github.com/openstack/ceilometer/commit/f8796ee380d8bdbd2340a8c808cc2b708fc20abc
Submitter: Jenkins
Branch: master

commit f8796ee380d8bdbd2340a8c808cc2b708fc20abc
Author: Eoghan Glynn <email address hidden>
Date: Thu Sep 19 14:47:11 2013 +0100

    Notify with string representation of alarm reason

    Fixes bug 1227668

    Ensure string representation of alarm reason is passed to the
    alarm notifier over RPC (as opposed to a gettextutils.Message
    instance).

    Change-Id: I6a840789f83068ee5721cb139b06cf0071240152