Heat template doesn't support "time_constraints" property in OS::Ceilometer::Alarm resource

Bug #1447942 reported by Yee Li
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Heat
Fix Released
Medium
Rico Lin

Bug Description

As https://review.openstack.org/#/c/75391/ and https://blueprints.launchpad.net/ceilometer/+spec/time-constrained-alarms described, alarm already support time constraints property. But "OS::Ceilometer::Alarm" resource in heat is not support yet. So we couldn't create an alarm constrained by time-of-day or day-of-week by launching stack in heat.

Yee Li (yee-li)
Changed in heat:
assignee: nobody → Yee Li (yee-li)
Changed in heat:
status: New → Triaged
importance: Undecided → Medium
Revision history for this message
Kent Wang (k.wang) wrote :

Hi Yee are you still working on this bug?

Yee Li (yee-li)
Changed in heat:
assignee: Yee Li (yee-li) → nobody
Rico Lin (rico-lin)
Changed in heat:
assignee: nobody → Rico Lin (rico-lin)
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/201498

Changed in heat:
status: Triaged → In Progress
Revision history for this message
Yee Li (yee-li) wrote :

The property time_constraints of resource 'OS::Ceilometer::Alarm' is like this:
    "time_constraints": [
        {
            "description": "nightly build every night at 23h for 3 hours",
            "duration": 10800,
            "name": "SampleConstraint",
            "start": "0 23 * * *",
            "timezone": "Europe/Ljubljana"
        }
    ],

whether we need to validate duration, name, start and timezone?

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

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

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

commit 943ccd2937210eb5c661384193947397acac5d22
Author: ricolin <email address hidden>
Date: Thu Jul 23 14:25:29 2015 +0800

    add new constraints to ceilometer

    Add timezone and cron_expression constraints.
    this is required by time_constraints in Ceilometer alarm resources.
    Change-Id: I2b348d9eb0f1d1ebf1407dbaeeb975dfa86b40d5
    Partial-Bug: #1447942

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

commit e24aef1a91464f86489695645ff92805f499837e
Author: ricolin <email address hidden>
Date: Thu Jul 23 14:28:05 2015 +0800

    add time_constraints property in OS::Ceilometer::Alarm

    Alarm already support time constraints property,
    but "OS::Ceilometer::Alarm" resource in heat is not support yet,
    so we couldn't create an alarm constrained by time-of-day or
    day-of-week by launching stack in heat.
    Change-Id: I4d05a6bf98ca7d616c36398fdfe91dff20559f3a
    Partial-Bug: #1447942

Rico Lin (rico-lin)
Changed in heat:
status: In Progress → Fix Committed
Changed in heat:
milestone: none → liberty-3
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in heat:
milestone: liberty-3 → 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.