"Image polling -> it (still) exists" is hard to understand

Bug #1292937 reported by Akihiro Motoki
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Fix Released
Medium
Rob Raymond

Bug Description

api/ceilometer.py has a message "Image polling -> it (still) exists" but it is hard to understand and difficult to translate.
While I know it is a copy of ceilometer API definition [1], it should be a better wording.
I hope someone who is familiar with ceilometer rewords this.

[1] http://docs.openstack.org/developer/ceilometer/measurements.html#image-glance

Revision history for this message
Facundo Farias (facundo-farias) wrote :

Somehow related with this bug: https://bugs.launchpad.net/horizon/+bug/1288487. Isn't it a documentation bug in the first place?

David Lyle (david-lyle)
Changed in horizon:
assignee: nobody → Rob Raymond (rob-raymond)
Revision history for this message
Akihiro Motoki (amotoki) wrote :

It seems better we review https://review.openstack.org/78602 .
The review changes a lot of strings but it looks a reasonable change.

Revision history for this message
Akihiro Motoki (amotoki) wrote :

My previous comment looks unrelated to this bug. Sorry for the confusion.

As Horizon side, we need to check they are visible or not. If invisible, we can defer this bug to the next release and first fix this as a documentation bug.

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

Hi Akihiro,

I've seen the text "Image polling -> it (still) exists" in 2 places.

Admin > Resource Usage.

1. In the Daily Report table
2. Stats tab Metrics dropdown. If you over over the item 'Image' you see it.

Revision history for this message
Eoghan Glynn (eglynn) wrote :

The intent behind the "Image polling -> it (still) exists" string in the ceilometer documentation is I believe to capture the following connotations:

1. samples for that the meter are periodically emitted as long as the image *continues* to exist

2. these data are produced by polling the glance RESTful API, as opposed to relying entirely on incoming AMQP notifications informing ceilometer of image lifecycle events

This string was then apperently copied from the ceilometer docco into the horizon tree:

  https://github.com/openstack/horizon/commit/44b302f6

However I'd agree that the current description is mangled and difficult even for a native English speaker to understand.

I'd suggest changing to:

  "Period confirmation of continued image existence"

Revision history for this message
Eoghan Glynn (eglynn) wrote :

s/Period confirmation of continued image existence/Periodic confirmation of continued image existence/

Changed in horizon:
status: New → In Progress
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/82842

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to horizon (master)

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

commit 39098ff2030ad7887994e012b88cfbdb386b0892
Author: Rob Raymond <email address hidden>
Date: Tue Mar 25 08:59:09 2014 -0600

    Clarify description string for image metric

    This metric seems to have an existential crisis.
    Change to more succinct "Image existence check"

    Change-Id: I953c55214e73bb93c452b25f97960e9637f1008c
    Closes-Bug: #1292937

Changed in horizon:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in horizon:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in horizon:
milestone: icehouse-rc1 → 2014.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.