remove database access from agent pollsters

Bug #1012242 reported by Doug Hellmann
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
Medium
Unassigned

Bug Description

The libvirt agent pollsters connect to the database directly to get a list of instances on the current host. We should use some other mechanism to ask nova (or at least our own central service) for that information.

Revision history for this message
Yaguang Tang (heut2008) wrote :

when nova finish ec2-id to uuid translate,we may need not to access the db ,since at that time,the uuid is the only identifier of an instance .

Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Re: [Bug 1012242] Re: remove database access from agent pollsters

The pollster needs to know which instances are on the compute node where
the pollster is running so it knows which UUID values to pass to libvirt
when asking for details. It also needs to include a bunch of other metadata
from the instance in the metering messages it sends so the collector can
save them. Some providers will bill differently for an instance depending
on where it is located within the data center, for instance.

On Tue, Jun 12, 2012 at 6:22 PM, Yaguang Tang <email address hidden> wrote:

> when nova finish ec2-id to uuid translate,we may need not to access
> the db ,since at that time,the uuid is the only identifier of an
> instance .
>
> --
> You received this bug notification because you are a member of
> Ceilometer Drivers, which is the registrant for Ceilometer.
> https://bugs.launchpad.net/bugs/1012242
>
> Title:
> remove database access from agent pollsters
>
> Status in Openstack Metering (ceilometer):
> New
>
> Bug description:
> The libvirt agent pollsters connect to the database directly to get a
> list of instances on the current host. We should use some other
> mechanism to ask nova (or at least our own central service) for that
> information.
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/ceilometer/+bug/1012242/+subscriptions
>

tags: added: agent effort-l pollster
Changed in ceilometer:
status: New → Confirmed
importance: Undecided → Medium
John Tran (jtran)
Changed in ceilometer:
assignee: nobody → John Tran (jtran)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ceilometer (master)

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

Changed in ceilometer:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

John Tran (jtran)
Changed in ceilometer:
status: In Progress → Fix Released
John Tran (jtran)
Changed in ceilometer:
status: Fix Released → In Progress
Revision history for this message
John Tran (jtran) wrote :

I've abandoned the previous patch and waiting on the nova 'no-db-compute' patch to pass and will consider implementing their 'VirtAPI' calls instead of the nova api calls.

https://review.openstack.org/#/c/15084/

John Tran (jtran)
Changed in ceilometer:
assignee: John Tran (jtran) → nobody
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ceilometer (master)

Reviewed: https://review.openstack.org/15512
Committed: http://github.com/stackforge/ceilometer/commit/c9247f10698b017ca33ca99010e3d69719019905
Submitter: Jenkins
Branch: master

commit c9247f10698b017ca33ca99010e3d69719019905
Author: Julien Danjou <email address hidden>
Date: Mon Oct 8 22:14:46 2012 +0000

    Remove database access from agent pollsters

    Fixes bug #1012242.

    Patch stolen from John Tran <email address hidden>

    Change-Id: Iab59eb752199e0cd3c8134a29e05b53356a30d75

Changed in ceilometer:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in ceilometer:
milestone: none → grizzly-2
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in ceilometer:
milestone: grizzly-2 → 2013.1
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Related blueprints

Remote bug watches

Bug watches keep track of this bug in other bug trackers.