alarm-history cannot support start_timestamp and end_timestamp in query filed

Bug #1291171 reported by Liusheng
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
Medium
Liusheng

Bug Description

If I use the following command in ceilometer CLI to get history of an alarm,it will raise an error.
ceilometer alarm-history -a 2656a970-5efe-4c11-b518-24c38f549316 -q start_timestamp=2014-03-04T09:56:07.270929

The traces in log file is:
http://paste.openstack.org/show/73212/

Liusheng (liusheng)
Changed in ceilometer:
assignee: nobody → Liusheng (liusheng)
Julien Danjou (jdanjou)
Changed in ceilometer:
status: New → Triaged
importance: Undecided → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ceilometer (master)

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

Changed in ceilometer:
status: Triaged → In Progress
Revision history for this message
Mehdi Abaakouk (sileht) wrote :

The request that should be work :

ceilometer alarm-history -a 2656a970-5efe-4c11-b518-24c38f549316 -q timestamp>=2014-03-04T09:56:07.270929

Revision history for this message
Openstack Gerrit (openstack-gerrit) wrote :

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

Revision history for this message
Openstack Gerrit (openstack-gerrit) wrote :

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

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

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on ceilometer (master)

Change abandoned by liusheng (<email address hidden>) on branch: master
Review: https://review.openstack.org/92066

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

Reviewed: https://review.openstack.org/128856
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=33065c16f6b7ffb1f3595a10559b395542e7e30d
Submitter: Jenkins
Branch: master

commit 33065c16f6b7ffb1f3595a10559b395542e7e30d
Author: liu-sheng <email address hidden>
Date: Thu Oct 16 17:02:49 2014 +0800

    Standardize timestamp fields of ceilometer API

    For currently, the timestamp in query fields is not unified, this
    patch standardizes the usage of timestamp query fileds:

    - unify the time query filed in API to *timestamp* supported only.
    - unify the time related fileds of filters to start_timestamp and
      end_timestamp to simplify the process logic.
    - improve the _query_to_kwargs method to give explicit presentation
      if user specifies unsupported query fields.

    Change-Id: Ic3ade312efa89edd01d2ee5ae31d341805b11f79
    Closes-bug: #1295104
    Closes-bug: #1291171

Changed in ceilometer:
status: In Progress → Fix Committed
Eoghan Glynn (eglynn)
Changed in ceilometer:
milestone: none → kilo-1
Thierry Carrez (ttx)
Changed in ceilometer:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in ceilometer:
milestone: kilo-1 → 2015.1.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.