diff options
Diffstat (limited to 'meta-openstack/recipes-devtools/python/python-ceilometer/0001-sqlalchemy-Fix-for-get_statistics-with-postgresql.patch')
| -rw-r--r-- | meta-openstack/recipes-devtools/python/python-ceilometer/0001-sqlalchemy-Fix-for-get_statistics-with-postgresql.patch | 33 |
1 files changed, 33 insertions, 0 deletions
diff --git a/meta-openstack/recipes-devtools/python/python-ceilometer/0001-sqlalchemy-Fix-for-get_statistics-with-postgresql.patch b/meta-openstack/recipes-devtools/python/python-ceilometer/0001-sqlalchemy-Fix-for-get_statistics-with-postgresql.patch new file mode 100644 index 0000000..2a3ae9d --- /dev/null +++ b/meta-openstack/recipes-devtools/python/python-ceilometer/0001-sqlalchemy-Fix-for-get_statistics-with-postgresql.patch | |||
| @@ -0,0 +1,33 @@ | |||
| 1 | From a784cd8f552e1f8ead86230f96e5b9ac8217efea Mon Sep 17 00:00:00 2001 | ||
| 2 | From: Bruce Ashfield <bruce.ashfield@windriver.com> | ||
| 3 | Date: Wed, 29 Jan 2014 18:05:46 -0500 | ||
| 4 | Subject: [PATCH] sqlalchemy: Fix for get_statistics with postgresql | ||
| 5 | |||
| 6 | Based on: https://review.openstack.org/#/c/59214/ | ||
| 7 | |||
| 8 | Add group by counter_unit in statistics queryI | ||
| 9 | Fixes: bug #1256318 | ||
| 10 | |||
| 11 | Signed-off-by: Bruce Ashfield <bruce.ashfield@windriver.com> | ||
| 12 | --- | ||
| 13 | ceilometer/storage/impl_sqlalchemy.py | 4 +++- | ||
| 14 | 1 file changed, 3 insertions(+), 1 deletion(-) | ||
| 15 | |||
| 16 | diff --git a/ceilometer/storage/impl_sqlalchemy.py b/ceilometer/storage/impl_sqlalchemy.py | ||
| 17 | index 8a3deba3a4ea..92d3a5901f5a 100644 | ||
| 18 | --- a/ceilometer/storage/impl_sqlalchemy.py | ||
| 19 | +++ b/ceilometer/storage/impl_sqlalchemy.py | ||
| 20 | @@ -574,7 +574,9 @@ class Connection(base.Connection): | ||
| 21 | group_attributes = [getattr(models.Meter, g) for g in groupby] | ||
| 22 | select.extend(group_attributes) | ||
| 23 | |||
| 24 | - query = session.query(*select) | ||
| 25 | + #Group by counter_unit for postgresql queries | ||
| 26 | + #In mysql this doesn't change anything because one meter has one unit | ||
| 27 | + query = session.query(*select).group_by(models.Meter.counter_unit) | ||
| 28 | |||
| 29 | if groupby: | ||
| 30 | query = query.group_by(*group_attributes) | ||
| 31 | -- | ||
| 32 | 1.7.10.4 | ||
| 33 | |||
