diff options
Diffstat (limited to 'meta-openstack/recipes-devtools/python/python-ceilometer')
2 files changed, 36 insertions, 3 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 | |||
diff --git a/meta-openstack/recipes-devtools/python/python-ceilometer/ceilometer.conf b/meta-openstack/recipes-devtools/python/python-ceilometer/ceilometer.conf index abeb65c..4632f40 100644 --- a/meta-openstack/recipes-devtools/python/python-ceilometer/ceilometer.conf +++ b/meta-openstack/recipes-devtools/python/python-ceilometer/ceilometer.conf | |||
| @@ -65,7 +65,7 @@ | |||
| 65 | 65 | ||
| 66 | # Inspector to use for inspecting the hypervisor layer (string | 66 | # Inspector to use for inspecting the hypervisor layer (string |
| 67 | # value) | 67 | # value) |
| 68 | #hypervisor_inspector=libvirt | 68 | hypervisor_inspector=libvirt |
| 69 | 69 | ||
| 70 | 70 | ||
| 71 | # | 71 | # |
| @@ -86,7 +86,7 @@ libvirt_type=kvm | |||
| 86 | # | 86 | # |
| 87 | 87 | ||
| 88 | # Exchange name for Glance notifications (string value) | 88 | # Exchange name for Glance notifications (string value) |
| 89 | #glance_control_exchange=glance | 89 | glance_control_exchange=glance |
| 90 | 90 | ||
| 91 | 91 | ||
| 92 | # | 92 | # |
| @@ -94,7 +94,7 @@ libvirt_type=kvm | |||
| 94 | # | 94 | # |
| 95 | 95 | ||
| 96 | # Exchange name for Neutron notifications (string value) | 96 | # Exchange name for Neutron notifications (string value) |
| 97 | #neutron_control_exchange=neutron | 97 | neutron_control_exchange=neutron |
| 98 | 98 | ||
| 99 | 99 | ||
| 100 | # | 100 | # |
