Optimize Ceilometer resource list query

Bug #1404076 reported by Rohit Jaiswal on 2014-12-19
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Ceilometer
High
Rohit Jaiswal
ceilometer (Ubuntu)
Undecided
Unassigned
Trusty
High
Jorge Niedbalski

Bug Description

[Impact]

 * Ceilometer resource-list requires an open-ended scan of resource and sample tables to retrieve a unique list of resources. There should be a way to optimize the scans.

[Test Case]

 * Deploy an openstack cloud using the current trusty package of ceilometer.
 * Run a 'ceilometer resource-list' without the patch the running
time for that command takes (if returns).

ubuntu@queen:~$ time ceilometer resource-list | wc -l
10031 real 1m3.975s user 0m3.572s sys 0m0.187s
ubuntu@queen:~$ time ceilometer resource-list | wc -l
10036 real 1m54.993s user 0m3.354s sys 0m0.184s

* With the included SRU patch :

real 0m15.693s
user 0m1.104s

sys 0m0.065s
[Regression Potential]

 * None expected.

[Other Info]

* Backport of https://review.openstack.org/142951

Changed in ceilometer:
status: New → In Progress

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

Changed in ceilometer:
assignee: nobody → Rohit Jaiswal (rohit-jaiswal-3)

Reviewed: https://review.openstack.org/142951
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=e5ad0bac86f2e17a9a0bc4cf53ce616f9280ed06
Submitter: Jenkins
Branch: master

commit e5ad0bac86f2e17a9a0bc4cf53ce616f9280ed06
Author: Rohit Jaiswal <email address hidden>
Date: Thu Dec 18 17:12:23 2014 -0800

    Optimize resource list query

    Instead of 2 queries per resource
    to fetch min and max sample timestamp
    value, replaced with a single query of
    similar cost. Another query of lesser
    cost to retrieve the resource details.

    Change-Id: I50d4a63a387e6a83c6090ebeafe7a330c5fda61c
    Closes-Bug: #1404076

Changed in ceilometer:
status: In Progress → Fix Committed
Eoghan Glynn (eglynn) on 2015-01-06
Changed in ceilometer:
importance: Undecided → High
milestone: none → kilo-2
Thierry Carrez (ttx) on 2015-02-04
Changed in ceilometer:
status: Fix Committed → Fix Released
Thierry Carrez (ttx) on 2015-04-30
Changed in ceilometer:
milestone: kilo-2 → 2015.1.0
Jorge Niedbalski (niedbalski) wrote :
Changed in ceilometer (Ubuntu):
status: New → Fix Released
Changed in ceilometer (Ubuntu Trusty):
status: New → In Progress
importance: Undecided → High
assignee: nobody → Jorge Niedbalski (niedbalski)
description: updated
tags: added: sts

Hello Rohit, or anyone else affected,

Accepted ceilometer into trusty-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/ceilometer/2014.1.5-0ubuntu2 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

Changed in ceilometer (Ubuntu Trusty):
status: In Progress → Fix Committed
tags: added: verification-needed
Billy Olsen (billy-olsen) wrote :

Verified today (trusty-icehouse) running through a tempest run to fill resources in the ceilometer database and issue resource, metric, and sample queries against.

tags: added: verification-done
removed: verification-needed

The verification of the Stable Release Update for ceilometer has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ceilometer - 2014.1.5-0ubuntu2

---------------
ceilometer (2014.1.5-0ubuntu2) trusty; urgency=medium

  * 0001-Optimize-resource-list-query.patch: Optimize resource list
    query. (LP: #1404076).

 -- Jorge Niedbalski <email address hidden> Tue, 14 Jul 2015 08:18:13 -0500

Changed in ceilometer (Ubuntu Trusty):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers