listing alarms shouldn't ordered by alarm id

[Liusheng]

Listing alarms is ordered by alarm id (an uuid), it is meaningless.
It should be ordered by creating time.

[ZhiQiang Fan (aji-zqfan)]

is there any plan to implement 'order by' feature in juno? or is just not so useful and can be an optional print argument in ceilometerclient?

[OpenStack Infra (hudson-openstack)]

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

[Nejc Saje (nejc-saje)]

Wouldn't it make more sense for the alarms to be ordered alphabetically by their name?

[Ilya Tyaptin (ityaptin)]

Issues with alarm ordering also are found in unit test class ceilometer.tests.storage.test_storage_scenarios.ComplexAlarmHistoryQueryTest[1]. In tests we use index in alarms list from alarm_conn.get_alarms() and in different backend index of alarm in this list is different.

CR for current bug does not affect this issue and I guess that order should to be fixed at storage level. As another approach - we can change unittest.

https://github.com/openstack/ceilometer/blob/master/ceilometer/tests/storage/test_storage_scenarios.py#L2598

[OpenStack Infra (hudson-openstack)]

Related fix proposed to branch: master
Review: https://review.openstack.org/128255

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Change abandoned by Samta Rangare (<email address hidden>) on branch: master
Review: https://review.openstack.org/127843
Reason: This patch is abandon.
Please refer: https://review.openstack.org/#/c/128573

[OpenStack Infra (hudson-openstack)]

Change abandoned by Samta Rangare (<email address hidden>) on branch: master
Review: https://review.openstack.org/106371
Reason: This patch is abandon.
Please refer: https://review.openstack.org/#/c/128573

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

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

commit b9d8b5832092128cb6ef6cf58e56ff8c4edf707d
Author: Samta <email address hidden>
Date: Wed Oct 15 13:37:58 2014 +0530

    Alarms listing based on "timestamp"

    Fixes are proposed for ordering alarms based
    on "timestamp" in descending order.
    Existing tests has been modified at storage
    and api without breaking their functionality.

    One test case has been provided with this patch.

    Co-Authored-By: Ilya Tyaptin <email address hidden>

    Change-Id: Ie2bb950b6d32188f451f905e2cf07e4d938df1ac
    Closes-Bug: #1287119