diff options
| author | Bruce Ashfield <bruce.ashfield@windriver.com> | 2014-01-08 00:50:15 -0500 |
|---|---|---|
| committer | Bruce Ashfield <bruce.ashfield@windriver.com> | 2014-01-15 00:33:53 -0500 |
| commit | b45880a1a11007476446c7e2f53b0fee43c43453 (patch) | |
| tree | 160739eceae26c926c0b93ebaf5a039788f025ee /meta-openstack/recipes-devtools/python/python-ceilometer/0001-Fix-for-get_resources-with-postgresql.patch | |
| parent | 9ee16ea7922a1777e95cd5ecdf577c6d5935bbd2 (diff) | |
| download | meta-cloud-services-b45880a1a11007476446c7e2f53b0fee43c43453.tar.gz | |
ceilometer: postgresql fixes
Cherry picking two ceilometer master changes to address postgresql database
issues:
https://bugs.launchpad.net/ceilometer/+bug/1241526
https://review.openstack.org/#/c/49456/
Signed-off-by: Bruce Ashfield <bruce.ashfield@windriver.com>
Diffstat (limited to 'meta-openstack/recipes-devtools/python/python-ceilometer/0001-Fix-for-get_resources-with-postgresql.patch')
| -rw-r--r-- | meta-openstack/recipes-devtools/python/python-ceilometer/0001-Fix-for-get_resources-with-postgresql.patch | 36 |
1 files changed, 36 insertions, 0 deletions
diff --git a/meta-openstack/recipes-devtools/python/python-ceilometer/0001-Fix-for-get_resources-with-postgresql.patch b/meta-openstack/recipes-devtools/python/python-ceilometer/0001-Fix-for-get_resources-with-postgresql.patch new file mode 100644 index 0000000..52cffb3 --- /dev/null +++ b/meta-openstack/recipes-devtools/python/python-ceilometer/0001-Fix-for-get_resources-with-postgresql.patch | |||
| @@ -0,0 +1,36 @@ | |||
| 1 | From f28a381b58516018ca35cdf7b4e2879a5bcac6ad Mon Sep 17 00:00:00 2001 | ||
| 2 | From: Thomas Maddox <thomas.maddox@rackspace.com> | ||
| 3 | Date: Mon, 21 Oct 2013 15:55:49 +0000 | ||
| 4 | Subject: [PATCH 1/2] Fix for get_resources with postgresql | ||
| 5 | |||
| 6 | Add max_ts and min_ts to GROUP BY in sub-query, since they need to be aggregated to SELECT them. | ||
| 7 | |||
| 8 | Closes-Bug: #1241526 | ||
| 9 | Change-Id: Ifdd2bc661b5da31bd40d1c3fa1fc442d7417399f | ||
| 10 | (cherry picked from commit 0a98159bc9c727a89bd8c15347ac380a21acaa59) | ||
| 11 | |||
| 12 | Signed-off-by: Bruce Ashfield <bruce.ashfield@windriver.com> | ||
| 13 | --- | ||
| 14 | ceilometer/storage/impl_sqlalchemy.py | 6 +++++- | ||
| 15 | 1 file changed, 5 insertions(+), 1 deletion(-) | ||
| 16 | |||
| 17 | diff --git a/ceilometer/storage/impl_sqlalchemy.py b/ceilometer/storage/impl_sqlalchemy.py | ||
| 18 | index a6e7d307e407..546c0c0e6553 100644 | ||
| 19 | --- a/ceilometer/storage/impl_sqlalchemy.py | ||
| 20 | +++ b/ceilometer/storage/impl_sqlalchemy.py | ||
| 21 | @@ -361,7 +361,11 @@ class Connection(base.Connection): | ||
| 22 | ).filter( | ||
| 23 | Meter.resource_id == ts_subquery.c.resource_id, | ||
| 24 | Meter.timestamp == ts_subquery.c.max_ts | ||
| 25 | - ).group_by(Meter.resource_id).subquery() | ||
| 26 | + ).group_by( | ||
| 27 | + ts_subquery.c.resource_id, | ||
| 28 | + ts_subquery.c.max_ts, | ||
| 29 | + ts_subquery.c.min_ts | ||
| 30 | + ).subquery() | ||
| 31 | |||
| 32 | query = session.query( | ||
| 33 | Meter, | ||
| 34 | -- | ||
| 35 | 1.7.10.4 | ||
| 36 | |||
