resource-list seems not work with postgresql

[Mehdi Abaakouk]

Hi,

I have created some ceilometerclient very simple test in tempest https://review.openstack.org/#/c/49700/

And ceilometer resource-list won't work with postgres backend:

Failed test:
http://logs.openstack.org/00/49700/4/check/check-tempest-devstack-vm-postgres-full/43a8857/console.html#_2013-10-18_08_03_50_004

Ceilometer API error on this test:
http://logs.openstack.org/00/49700/4/check/check-tempest-devstack-vm-postgres-full/43a8857/logs/screen-ceilometer-api.txt.gz

Cheers,

[Thomas Maddox (thomas-maddox)]

Looks to be because of three non-aggregate values being selected from an aggregated subquery:

(SELECT
        max(meter.id) AS max_id,
        anon_2.resource_id AS resource_id,
        anon_2.max_ts AS max_ts,
        anon_2.min_ts AS min_ts
    FROM meter,
        (SELECT
            meter.resource_id AS resource_id,
            max(meter.timestamp) AS max_ts,
            min(meter.timestamp) AS min_ts
        FROM meter
        GROUP BY meter.resource_id) AS anon_2
    WHERE
        meter.resource_id = anon_2.resource_id AND
        meter.timestamp = anon_2.max_ts
    GROUP BY meter.resource_id)

The logs were complaining about anon_2.resource_id; I'd wager the other 2 will cause trouble as well.

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/52966
Committed: http://github.com/openstack/ceilometer/commit/0a98159bc9c727a89bd8c15347ac380a21acaa59
Submitter: Jenkins
Branch: master

commit 0a98159bc9c727a89bd8c15347ac380a21acaa59
Author: Thomas Maddox <email address hidden>
Date: Mon Oct 21 15:55:49 2013 +0000

    Fix for get_resources with postgresql

    Add max_ts and min_ts to GROUP BY in sub-query, since they need to be aggregated to SELECT them.

    Closes-Bug: #1241526
    Change-Id: Ifdd2bc661b5da31bd40d1c3fa1fc442d7417399f