Info about 'severity' field changes is not displayed via alarm-history call

Bug #1452264 reported by Yurii Prokulevych
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
Medium
Mehdi Abaakouk
Kilo
Fix Released
Undecided
Unassigned

Bug Description

Changes to the alarm's severity are not displayed in 'ceilometer alarm-history <alarm_id> ' output:

# ceilometer alarm-history 357ce1eb-839e-4782-8676-11a7b7ef5d18
+------------------+---------------------+----------------------------------------------------+
| Type | Timestamp | Detail |
+------------------+---------------------+----------------------------------------------------+
| state transition | 2015-05-06T09:53:37 | state: insufficient data |
| state transition | 2015-05-06T09:50:37 | state: alarm |
| state transition | 2015-05-06T09:20:36 | state: ok |
| rule change | 2015-05-06T09:14:38 | enabled: True |
| rule change | 2015-05-06T07:48:13 | enabled: False |
| creation | 2015-05-06T05:56:40 | name: ThresholdAlarm-1-moderate |
| | | description: Test Alarm TC474397 moderate |
| | | type: threshold |
| | | rule: alarm_meter_1_moderate >= 3.0 during 3 x 60s |
| | | time_constraints: None |
+------------------+---------------------+----------------------------------------------------+

# ceilometer alarm-threshold-update 357ce1eb-839e-4782-8676-11a7b7ef5d18 --severity low >/dev/null 2>&1

# ceilometer alarm-history 357ce1eb-839e-4782-8676-11a7b7ef5d18
+------------------+----------------------------+----------------------------------------------------+
| Type | Timestamp | Detail |
+------------------+----------------------------+----------------------------------------------------+
| rule change | 2015-05-06T11:57:32.345000 | |
| state transition | 2015-05-06T09:53:37.125000 | state: insufficient data |
| state transition | 2015-05-06T09:50:37.021000 | state: alarm |
| state transition | 2015-05-06T09:20:36.895000 | state: ok |
| rule change | 2015-05-06T09:14:38.644000 | enabled: True |
| rule change | 2015-05-06T07:48:13.844000 | enabled: False |
| creation | 2015-05-06T05:56:40.562000 | name: ThresholdAlarm-1-moderate |
| | | description: Test Alarm TC474397 moderate |
| | | type: threshold |
| | | rule: alarm_meter_1_moderate >= 3.0 during 3 x 60s |
| | | time_constraints: None |
+------------------+----------------------------+----------------------------------------------------+

# ceilometer alarm-threshold-update 357ce1eb-839e-4782-8676-11a7b7ef5d18 --severity critical >/dev/null 2>&1

# ceilometer alarm-history 357ce1eb-839e-4782-8676-11a7b7ef5d18
+------------------+----------------------------+----------------------------------------------------+
| Type | Timestamp | Detail |
+------------------+----------------------------+----------------------------------------------------+
| rule change | 2015-05-06T11:58:47.653000 | |
| rule change | 2015-05-06T11:57:32.345000 | |
| state transition | 2015-05-06T09:53:37.125000 | state: insufficient data |
| state transition | 2015-05-06T09:50:37.021000 | state: alarm |
| state transition | 2015-05-06T09:20:36.895000 | state: ok |
| rule change | 2015-05-06T09:14:38.644000 | enabled: True |
| rule change | 2015-05-06T07:48:13.844000 | enabled: False |
| creation | 2015-05-06T05:56:40.562000 | name: ThresholdAlarm-1-moderate |
| | | description: Test Alarm TC474397 moderate |
| | | type: threshold |
| | | rule: alarm_meter_1_moderate >= 3.0 during 3 x 60s |
| | | time_constraints: None |
+------------------+----------------------------+----------------------------------------------------+

Mehdi Abaakouk (sileht)
Changed in ceilometer:
assignee: nobody → Mehdi Abaakouk (sileht)
Revision history for this message
Yurii Prokulevych (yprokule) wrote :

Ceilometer versions:
rpm -qa | grep ceilo
openstack-ceilometer-common-2015.1.0-1.el7.noarch
openstack-ceilometer-alarm-2015.1.0-1.el7.noarch
openstack-ceilometer-compute-2015.1.0-1.el7.noarch
openstack-ceilometer-collector-2015.1.0-1.el7.noarch
openstack-ceilometer-notification-2015.1.0-1.el7.noarch
openstack-ceilometer-central-2015.1.0-1.el7.noarch
python-ceilometerclient-1.0.13-1.el7.noarch
openstack-ceilometer-api-2015.1.0-1.el7.noarch
python-ceilometer-2015.1.0-1.el7.noarch

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ceilometer (master)

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

Changed in ceilometer:
status: New → In Progress
ZhiQiang Fan (aji-zqfan)
Changed in ceilometer:
importance: Undecided → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ceilometer (master)

Reviewed: https://review.openstack.org/180507
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=ecd4ee6566bf718489e85026d26f12ada1d62929
Submitter: Jenkins
Branch: master

commit ecd4ee6566bf718489e85026d26f12ada1d62929
Author: Mehdi Abaakouk <email address hidden>
Date: Wed May 6 14:19:06 2015 +0200

    api: record severity change in alarm history

    A useless change was done on the original alarm object when
    we update the alarm. That have the side effect to hide the change
    from the history.

    This change fixes that

    Change-Id: I9b3030dacf0da83bc0570ed4a6f9d460f178e2be
    Closes-bug: #1452264

Changed in ceilometer:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ceilometer (stable/kilo)

Fix proposed to branch: stable/kilo
Review: https://review.openstack.org/192568

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ceilometer (stable/kilo)

Reviewed: https://review.openstack.org/192568
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=023cfd11f385dbbc3800622ea45a0111d304f8d6
Submitter: Jenkins
Branch: stable/kilo

commit 023cfd11f385dbbc3800622ea45a0111d304f8d6
Author: Mehdi Abaakouk <email address hidden>
Date: Wed May 6 14:19:06 2015 +0200

    api: record severity change in alarm history

    A useless change was done on the original alarm object when
    we update the alarm. That have the side effect to hide the change
    from the history.

    This change fixes that

    Change-Id: I9b3030dacf0da83bc0570ed4a6f9d460f178e2be
    Closes-bug: #1452264
    (cherry picked from commit ecd4ee6566bf718489e85026d26f12ada1d62929)

tags: added: in-stable-kilo
Thierry Carrez (ttx)
Changed in ceilometer:
milestone: none → liberty-1
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in ceilometer:
milestone: liberty-1 → 5.0.0
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.