alarm-threshold-update cannot update generic alarm attributes

Bug #1253989 reported by Eoghan Glynn
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
python-ceilometerclient
Fix Released
Medium
Eoghan Glynn

Bug Description

Generic (i.e. non-threshold-related) alarm attributes cannot be updated with the alarm-threshold-update command, for example:

  $ ceilometer --debug alarm-threshold-update -a $ALARM_ID --repeat-actions True
  INFO (connectionpool:176) Starting new HTTP connection (1): 192.168.122.73
  DEBUG (connectionpool:344) "POST /v2.0/tokens HTTP/1.1" 200 9905
  'threshold_rule'

The non-update is not properly reported, i.e. the failure is semi-silent.

This is inconsistent with the help message for the command that implies all attributes can be updated, also with the behavior of alarm-combination-update.

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

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

Changed in python-ceilometerclient:
assignee: nobody → Eoghan Glynn (eglynn)
status: New → In Progress
Eoghan Glynn (eglynn)
Changed in python-ceilometerclient:
importance: Undecided → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to python-ceilometerclient (master)

Reviewed: https://review.openstack.org/57903
Committed: http://github.com/openstack/python-ceilometerclient/commit/08b476d2c9edeff7d025038467922f3d1cda6f19
Submitter: Jenkins
Branch: master

commit 08b476d2c9edeff7d025038467922f3d1cda6f19
Author: Eoghan Glynn <email address hidden>
Date: Fri Nov 22 11:18:35 2013 +0000

    Allow alarm-threshold-update to upate generic attributes

    Fixes bug 1253989

    Previously, generic (i.e. non-threshold-related) alarm attributes could
    not be updated with the alarm-threshold-update command. An attempt to
    do so failed semi-silently when a non-existent dict key was referenced.

    Now, all alarm attributes can be updated with this command.

    Change-Id: Iba3f21de879fb853575dcec1730de7873eab8afd

Changed in python-ceilometerclient:
status: In Progress → Fix Committed
Changed in python-ceilometerclient:
milestone: none → 1.3.0
status: Fix Committed → Fix Released
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.