can't mix ceilometer and cwlite alarms in one stack

Bug #1375111 reported by Angus Salkeld
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
OpenStack Heat
Fix Released
Medium
Angus Salkeld

Bug Description

From: https://ask.openstack.org/en/question/47690/create-a-highly-available-auto-scaling-group/?answer=49223#post-id-49223

It seems like heat-engine trying to parse ceilometer cpu_alarm_high/low as a cloudwatch alarm. Using each feature alone - autoscaling with ceilometer and restart with cloudwatch, worked fine and never gave this error.

 2014-09-17 12:21:39.146 11734 TRACE heat.openstack.common.loopingcall self.check_stack_watches(sid)
 2014-09-17 12:21:39.146 11734 TRACE heat.openstack.common.loopingcall File "/usr/lib/python2.7/site-packages/heat/engine/service.py", line 229, in check_stack_watches
 2014-09-17 12:21:39.146 11734 TRACE heat.openstack.common.loopingcall actions = rule.evaluate()
 2014-09-17 12:21:39.146 11734 TRACE heat.openstack.common.loopingcall File "/usr/lib/python2.7/site-packages/heat/engine/watchrule.py", line 234, in evaluate
 2014-09-17 12:21:39.146 11734 TRACE heat.openstack.common.loopingcall return self.run_rule()
 2014-09-17 12:21:39.146 11734 TRACE heat.openstack.common.loopingcall File "/usr/lib/python2.7/site-packages/heat/engine/watchrule.py", line 241, in run_rule
 2014-09-17 12:21:39.146 11734 TRACE heat.openstack.common.loopingcall new_state = self.get_alarm_state()
 2014-09-17 12:21:39.146 11734 TRACE heat.openstack.common.loopingcall File "/usr/lib/python2.7/site-packages/heat/engine/watchrule.py", line 224, in get_alarm_state
 2014-09-17 12:21:39.146 11734 TRACE heat.openstack.common.loopingcall fn = getattr(self, 'do_%s' % self.rule['Statistic'])
 2014-09-17 12:21:39.146 11734 TRACE heat.openstack.common.loopingcall KeyError: 'Statistic'
 2014-09-17 12:21:39.146 11734 TRACE heat.openstack.common.loopingcall

Angus Salkeld (asalkeld)
Changed in heat:
status: New → Triaged
importance: Undecided → Medium
assignee: nobody → Angus Salkeld (asalkeld)
milestone: none → next
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to heat (master)

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

Changed in heat:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to heat (master)

Reviewed: https://review.openstack.org/124661
Committed: https://git.openstack.org/cgit/openstack/heat/commit/?id=c9301bb81dbe29e7e3c692b56f530794909b3edf
Submitter: Jenkins
Branch: master

commit c9301bb81dbe29e7e3c692b56f530794909b3edf
Author: Angus Salkeld <email address hidden>
Date: Mon Sep 29 14:37:31 2014 +1000

    Don't evaluate ceilometer watchrules

    If you have both Ceilometer and cwlite alarms in one stack (a little odd)
    we try to evaluate Ceilometer alarms in the periodic task.

    Change-Id: Ia75bf1e5cfa3e2ac31242eafd5fa96d757786ada
    Closes-bug: #1375111

Changed in heat:
status: In Progress → Fix Committed
Angus Salkeld (asalkeld)
Changed in heat:
milestone: next → 2014.2.2
Chuck Short (zulcss)
Changed in heat:
status: Fix Committed → Fix Released
status: Fix Released → Fix Committed
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.