ceilometer alarm-history should throw error when alarm id is invalid
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
python-ceilometerclient |
In Progress
|
Undecided
|
Abhishek Talwar |
Bug Description
when I execute 'ceilometer alarm-histoy' command with invalid alarm-id '123' error is not thrown whereas when I use the same
alarm id with 'ceilometer alarm-show ' command an error is thrown.
$ ceilometer alarm-list
+------
| Alarm ID | Name | State | Severity | Enabled | Continuous | Alarm condition | Time constraints |
+------
| 20369539-
+------
$ ceilometer alarm-history 123
+------
| Type | Timestamp | Detail |
+------
+------
but if we execute alarm-show command with id "123"
dev@vyom:~$ ceilometer alarm-show 123
Alarm 123 not found (HTTP 404) (Request-ID: req-a72c4a50-
I think appropriate error should be thrown when invalid alarm ID is provided as an argument to 'ceilometer alarm-history ' command .
Changed in python-ceilometerclient: | |
assignee: | nobody → sonu (sonu-bhumca11) |
Changed in python-ceilometerclient: | |
assignee: | sonu (sonu-bhumca11) → Abhishek Talwar (abhishek-talwar) |
if an alarm is created but never change it state after some time which longer than alarm history ttl, then the alarm exists but have no alarm history
I think just return an empty result is acceptable