alarm-evaluator severity attribute null value causes error in >= juno

Bug #1539570 reported by Edward Hope-Morley
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Aodh
Invalid
Undecided
Unassigned
Ceilometer
Won't Fix
Undecided
Unassigned

Bug Description

If I create a threshold alarm in Icehouse without providing a value for --severity everything works fine until I upgrade to any newer release (i.e. Juno or above) at which point the alarm-evaluator throws the following error:

2016-01-29 12:33:27.966 10419 TRACE ceilometer.alarm.service Traceback (most recent call last):
2016-01-29 12:33:27.966 10419 TRACE ceilometer.alarm.service File "/usr/lib/python2.7/dist-packages/ceilometer/alarm/service.py", line 93, in _evaluate_assigned_alarms
2016-01-29 12:33:27.966 10419 TRACE ceilometer.alarm.service alarms = self._assigned_alarms()
2016-01-29 12:33:27.966 10419 TRACE ceilometer.alarm.service File "/usr/lib/python2.7/dist-packages/ceilometer/alarm/service.py", line 151, in _assigned_alarms
2016-01-29 12:33:27.966 10419 TRACE ceilometer.alarm.service 'value': True}])
2016-01-29 12:33:27.966 10419 TRACE ceilometer.alarm.service File "/usr/lib/python2.7/dist-packages/ceilometerclient/v2/alarms.py", line 77, in list
2016-01-29 12:33:27.966 10419 TRACE ceilometer.alarm.service return self._list(options.build_url(self._path(), q))
2016-01-29 12:33:27.966 10419 TRACE ceilometer.alarm.service File "/usr/lib/python2.7/dist-packages/ceilometerclient/common/base.py", line 68, in _list
2016-01-29 12:33:27.966 10419 TRACE ceilometer.alarm.service resp = self.api.get(url)
2016-01-29 12:33:27.966 10419 TRACE ceilometer.alarm.service File "/usr/lib/python2.7/dist-packages/ceilometerclient/openstack/common/apiclient/client.py", line 355, in get
2016-01-29 12:33:27.966 10419 TRACE ceilometer.alarm.service return self.client_request("GET", url, **kwargs)
2016-01-29 12:33:27.966 10419 TRACE ceilometer.alarm.service File "/usr/lib/python2.7/dist-packages/ceilometerclient/openstack/common/apiclient/client.py", line 345, in client_request
2016-01-29 12:33:27.966 10419 TRACE ceilometer.alarm.service self, method, url, **kwargs)
2016-01-29 12:33:27.966 10419 TRACE ceilometer.alarm.service File "/usr/lib/python2.7/dist-packages/ceilometerclient/openstack/common/apiclient/client.py", line 261, in client_request
2016-01-29 12:33:27.966 10419 TRACE ceilometer.alarm.service method, self.concat_url(endpoint, url), **kwargs)
2016-01-29 12:33:27.966 10419 TRACE ceilometer.alarm.service File "/usr/lib/python2.7/dist-packages/ceilometerclient/openstack/common/apiclient/client.py", line 201, in request
2016-01-29 12:33:27.966 10419 TRACE ceilometer.alarm.service raise exceptions.from_response(resp, method, url)
2016-01-29 12:33:27.966 10419 TRACE ceilometer.alarm.service BadRequest: Invalid input for field/attribute severity. Value: 'None'. Value should be one of: moderate, critical, low (HTTP 400) (Request-ID: req-deab771d-fede-4cdc-962d-d7043b046022)

Presumably we could easily assume a default of low severity if a value of None is discovered thus allowing upgrades to continure working.

Revision history for this message
Edward Hope-Morley (hopem) wrote :

I've added both ceilometer and aodh since the alarm subsystem has moved to its own project for Mitaka but we'd like to see this fix backported to older versions of Openstack.

no longer affects: aodh (Ubuntu)
Zi Lian Ji (jizilian)
Changed in aodh:
assignee: nobody → Zi Lian Ji (jizilian)
Changed in ceilometer:
assignee: nobody → Zi Lian Ji (jizilian)
gordon chung (chungg)
Changed in aodh:
status: New → Triaged
Changed in ceilometer:
status: New → Triaged
Changed in aodh:
assignee: Zi Lian Ji (jizilian) → nobody
Changed in ceilometer:
assignee: Zi Lian Ji (jizilian) → nobody
Revision history for this message
luwei (luwei) wrote :

I create alarms via aodh API instead of ceilometer API from Liberty, and I did NOT meet this bug. If I didn't give a specific value of severity, the default is "low". See in aodh/api/controllers/v2/alarms.py.

Revision history for this message
ZhiQiang Fan (aji-zqfan) wrote :

I think juno already reach its end of life, are we going to support it still?

ZhiQiang Fan (aji-zqfan)
Changed in ceilometer:
status: Triaged → Invalid
status: Invalid → Triaged
Revision history for this message
gordon chung (chungg) wrote :

does not exist in ceilometer

Changed in ceilometer:
status: Triaged → Won't Fix
Revision history for this message
gordon chung (chungg) wrote :

this is referencing old relesaes, last comments seem to point to non-issue. feel free to repoen

Changed in aodh:
status: Triaged → Invalid
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.