ceilometer.py meter info descriptions should be clearer

Bug #1288487 reported by Cindy Lu
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Fix Released
Low
Kamil Rykowski

Bug Description

The meter info is reflected in the Resources Usage report and stats tabs.

In general, units need to be provided in each description.

"Number of update on the image"
--> update should be plural

"Number of upload of the image"
--> upload should be plural

"Number of delete on the image"
--> delete should be plural

"Image is downloaded"/"Image is served out"
--> Should have some type of metric unit in the description

"Duration of Volume" - should be reworded, not really sure what that means

"Uploaded image size" needs units

"Size of volume" needs units

"Average CPU Utilization" needs units

"CPU time used" needs units

Revision history for this message
Cindy Lu (clu-m) wrote :
Cindy Lu (clu-m)
description: updated
description: updated
Changed in horizon:
status: New → Confirmed
tags: added: low-hanging-fruit
Cindy Lu (clu-m)
description: updated
Liusheng (liusheng)
Changed in horizon:
assignee: nobody → Liusheng (liusheng)
Liusheng (liusheng)
Changed in horizon:
assignee: Liusheng (liusheng) → nobody
Changed in horizon:
assignee: nobody → Facundo Farias (facundo-farias)
Revision history for this message
Facundo Farias (facundo-farias) wrote :
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to horizon (master)

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

Changed in horizon:
status: Confirmed → In Progress
Revision history for this message
Cindy Lu (clu-m) wrote :

Thanks for the related docs page Facundo.

Revision history for this message
Cindy Lu (clu-m) wrote :

Hey Facundo!

Any update on this?

David Lyle (david-lyle)
Changed in horizon:
importance: Undecided → Low
Revision history for this message
Pawel Skowron (pawel-skowron) wrote :

Hey Facundo!

Any update on this?

Changed in horizon:
assignee: Facundo Farias (facundo-farias) → Pawel Skowron (pawel-skowron)
Revision history for this message
Julie Pichon (jpichon) wrote :

Pawel, you can see in comment #3 that there is a patch already up for review to fix this: https://review.openstack.org/#/c/78602/ , sadly it got stuck in review limbo, but it's pretty much ready to go. You can help move things along by reviewing this and other patches. Thanks!

Changed in horizon:
assignee: Pawel Skowron (pawel-skowron) → Facundo Farias (facundo-farias)
Changed in horizon:
assignee: Facundo Farias (facundo-farias) → Juan Manuel Ollé (juan-m-olle)
Revision history for this message
Juan Manuel Ollé (juan-m-olle) wrote :

Sorry, I upload the change because I see Julie's comment. and It is just a cosmetic

Changed in horizon:
assignee: Juan Manuel Ollé (juan-m-olle) → nobody
Julie Pichon (jpichon)
Changed in horizon:
assignee: nobody → Facundo Farias (facundo-farias)
Changed in horizon:
assignee: Facundo Farias (facundo-farias) → Cindy Lu (clu-m)
Revision history for this message
Pawel Skowron (pawel-skowron) wrote :

Julie, I have reviewed https://review.openstack.org/#/c/78602/
Pawel

Changed in horizon:
assignee: Cindy Lu (clu-m) → Kamil Rykowski (kamil-rykowski)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to horizon (master)

Reviewed: https://review.openstack.org/78602
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=e77db390f948b8216c56f03510372b48535411b3
Submitter: Jenkins
Branch: master

commit e77db390f948b8216c56f03510372b48535411b3
Author: Kamil Rykowski <email address hidden>
Date: Fri Feb 13 14:22:46 2015 +0100

    Fixing Ceilometer measurements descriptions and comment typos

    On file ceilometer.py some changes were made regarding with
    units and clarifying some other descriptions.

    Additionally few minor improvements have been applied for existing
    comments and redundant conditional parentheses have been removed.

    Change-Id: I379033bf40a86a624605ae8f0f39654bfd15cd4d
    Closes-Bug: #1288487
    Co-Authored-By: Facundo Farias <email address hidden>
    Co-Authored-By: Kamil Rykowski <email address hidden>

Changed in horizon:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in horizon:
milestone: none → kilo-3
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in horizon:
milestone: kilo-3 → 2015.1.0
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.