compute counter units are not accurate/consistent

Bug #1165123 reported by gordon chung
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
High
Unassigned
Grizzly
Fix Released
High
Eoghan Glynn

Bug Description

relating to comments here: https://review.openstack.org/#/c/26115/3/doc/source/measurements.rst

in compute counters we return unit as B but they are really either MB or GB... only in notes can you tell this. also, in resource_metadata, the items are returned in lowercase which i would assume means bits not bytes (this isn't that important)

"resource_metadata": {
            "architecture": "",
            "availability_zone": "",
            "disk_gb": "",
            "display_name": "instName",
            "ephemeral_gb": "",
            "host": "c1901948957087ddde6f95c956f872e0220fca4e83b7663814f7c777",
            "image_ref": "5e482a99-d632-4cbc-9e25-2b3338e26aa9",
            "image_ref_url": "http://10.0.2.15:8774/af2180504258428db6072c35867d1c99/images/5e482a99-d632-4cbc-9e25-2b3338e26aa9",
            "instance_type": "1",
            "kernel_id": "",
            "memory_mb": "",
            "name": "instance-00000001",
            "os_type": "",
            "ramdisk_id": "",
            "reservation_id": "",
            "root_gb": "",
            "vcpus": ""
        },

in network counters pollster version of ip.floating* meters have 'ip' unit while notification version has 'floatingip' unit... not sure which is right but i would think one of them is wrong.

Revision history for this message
gordon chung (chungg) wrote :

happy to take a look at this, if someone can confirm they are valid and what the expected values are.

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

Reviewed: https://review.openstack.org/26462
Committed: http://github.com/openstack/ceilometer/commit/c498562b539fab1dc810b5113fede71df0f9de91
Submitter: Jenkins
Branch: master

commit c498562b539fab1dc810b5113fede71df0f9de91
Author: Julien Danjou <email address hidden>
Date: Tue Apr 9 11:50:05 2013 +0200

    instances: fix counter unit

    Fixes: bug#1165123

    Change-Id: Ib51c10fc8d622dc1ff121b3aedf479d62d7872f3
    Signed-off-by: Julien Danjou <email address hidden>

Changed in ceilometer:
status: New → Fix Committed
gordon chung (chungg)
summary: - counter units are not accurate/consistent
+ compute counter units are not accurate/consistent
Changed in ceilometer:
assignee: gordon chung (chungg) → nobody
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ceilometer (stable/grizzly)

Fix proposed to branch: stable/grizzly
Review: https://review.openstack.org/28151

Eoghan Glynn (eglynn)
tags: added: grizzly-backport-potential
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ceilometer (stable/grizzly)

Reviewed: https://review.openstack.org/28151
Committed: http://github.com/openstack/ceilometer/commit/a2dd8c0eb5e08b19a3b5bc919fd181a220ccfebe
Submitter: Jenkins
Branch: stable/grizzly

commit a2dd8c0eb5e08b19a3b5bc919fd181a220ccfebe
Author: Julien Danjou <email address hidden>
Date: Tue Apr 9 11:50:05 2013 +0200

    instances: fix counter unit

    Fixes: bug#1165123

    Change-Id: Ib51c10fc8d622dc1ff121b3aedf479d62d7872f3
    Signed-off-by: Julien Danjou <email address hidden>
    (cherry picked from commit c498562b539fab1dc810b5113fede71df0f9de91)

Alan Pevec (apevec)
tags: removed: grizzly-backport-potential
Eoghan Glynn (eglynn)
Changed in ceilometer:
importance: Undecided → High
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ceilometer (master)

Reviewed: https://review.openstack.org/29570
Committed: http://github.com/openstack/ceilometer/commit/b101e0f6d91c391de4ae54080ee9e73465051543
Submitter: Jenkins
Branch: master

commit b101e0f6d91c391de4ae54080ee9e73465051543
Author: Gordon Chung <email address hidden>
Date: Fri May 17 11:53:24 2013 -0400

    fix compute units in measurement doc

    related to bug#1165123 and patch (https://review.openstack.org/#/c/26462)

    memory: B to MB
    disk.root.size: B to GB
    disk.ephemeral.size: B to GB

    Change-Id: I59599c2cb7df02acc0d4349a8591a2c1bda4f9ec

Thierry Carrez (ttx)
Changed in ceilometer:
milestone: none → havana-1
status: Fix Committed → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ceilometer (stable/grizzly)

Fix proposed to branch: stable/grizzly
Review: https://review.openstack.org/31208

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ceilometer (stable/grizzly)

Reviewed: https://review.openstack.org/31208
Committed: http://github.com/openstack/ceilometer/commit/b4821c52c4754b2cf393751f1596d879b98e5f6a
Submitter: Jenkins
Branch: stable/grizzly

commit b4821c52c4754b2cf393751f1596d879b98e5f6a
Author: Gordon Chung <email address hidden>
Date: Fri May 17 11:53:24 2013 -0400

    fix compute units in measurement doc

    related to bug#1165123 and patch (https://review.openstack.org/#/c/26462)

    memory: B to MB
    disk.root.size: B to GB
    disk.ephemeral.size: B to GB

    Change-Id: I59599c2cb7df02acc0d4349a8591a2c1bda4f9ec
    (cherry picked from commit b101e0f6d91c391de4ae54080ee9e73465051543)

Thierry Carrez (ttx)
Changed in ceilometer:
milestone: havana-1 → 2013.2
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.