doc error in plugins.rst

Bug #1156814 reported by gordon chung
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
Undecided
Jason

Bug Description

in the example text:

In the ``CPUPollster`` plugin, the ``get_counters`` method is implemented as a loop
which, for each instances running on the local host, retrieves the cpu_time
from the hypervisor and sends back two ``Counter`` objects. The first one, named
"cpu", is of type "cumulative", meaning that between two polls, its value is
not reset, or in other word that the cpu value is always provided as a duration
that continuously increases since the creation of the instance. The second one,
named "instance", is of type "cumulative", meaning that it's value is just the
volume since the last poll. Here, the instance counter is only used as a way
to tell the system that the instance is still running, hence the hard coded
value of 1.

the second counter isn't named 'instance' nor is it cumulative.

Jason (zzs)
Changed in ceilometer:
assignee: nobody → Jason (zzs)
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/24735

Changed in ceilometer:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ceilometer (master)

Reviewed: https://review.openstack.org/24735
Committed: http://github.com/openstack/ceilometer/commit/320b77d3b261dccae2be0c6c8072f1922d33af15
Submitter: Jenkins
Branch: master

commit 320b77d3b261dccae2be0c6c8072f1922d33af15
Author: Jason Zhang <email address hidden>
Date: Mon Mar 18 17:55:09 2013 -0700

    Updated the description of get_counters routine.

    Modified the second counter description of
    get_counters.

    Fixes: bug 1156814

    Change-Id: Ie7d4d3d93f853598069d8676f6be4335ddd3afb9

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

Other bug subscribers

Remote bug watches

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