summaryrefslogtreecommitdiffstats
path: root/meta-openstack/recipes-devtools/python/python-ceilometer/0001-Fix-for-get_resources-with-postgresql.patch
diff options
context:
space:
mode:
authorBruce Ashfield <bruce.ashfield@windriver.com>2014-01-08 00:50:15 -0500
committerBruce Ashfield <bruce.ashfield@windriver.com>2014-01-15 00:33:53 -0500
commitb45880a1a11007476446c7e2f53b0fee43c43453 (patch)
tree160739eceae26c926c0b93ebaf5a039788f025ee /meta-openstack/recipes-devtools/python/python-ceilometer/0001-Fix-for-get_resources-with-postgresql.patch
parent9ee16ea7922a1777e95cd5ecdf577c6d5935bbd2 (diff)
downloadmeta-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.patch36
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 @@
1From f28a381b58516018ca35cdf7b4e2879a5bcac6ad Mon Sep 17 00:00:00 2001
2From: Thomas Maddox <thomas.maddox@rackspace.com>
3Date: Mon, 21 Oct 2013 15:55:49 +0000
4Subject: [PATCH 1/2] Fix for get_resources with postgresql
5
6Add max_ts and min_ts to GROUP BY in sub-query, since they need to be aggregated to SELECT them.
7
8Closes-Bug: #1241526
9Change-Id: Ifdd2bc661b5da31bd40d1c3fa1fc442d7417399f
10(cherry picked from commit 0a98159bc9c727a89bd8c15347ac380a21acaa59)
11
12Signed-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
17diff --git a/ceilometer/storage/impl_sqlalchemy.py b/ceilometer/storage/impl_sqlalchemy.py
18index 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--
351.7.10.4
36