need to handle missing units values in existing mongodb data

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

Bug Description

The changeset that added meter units did not update the existing mongo database contents: https://review.openstack.org/#/c/18413/

Changed in ceilometer:
importance: Undecided → Medium
Revision history for this message
Julien Danjou (jdanjou) wrote :

It's a NoSQL db, do we really need to update all already stored documents?

Revision history for this message
Doug Hellmann (doug-hellmann) wrote :

We need to provide a units value, but I guess we could do that in the query. I'll change the description.

summary: - need a migration script for mongodb data to add 'counter_unit' to
- existing events
+ need to handle missing units values in existing mongodb data
Changed in ceilometer:
milestone: none → grizzly-3
status: New → Confirmed
Nick Barcet (nijaba)
Changed in ceilometer:
milestone: grizzly-3 → grizzly-rc1
Lianhao Lu (lianhao-lu)
Changed in ceilometer:
assignee: nobody → Lianhao Lu (lianhao-lu)
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/23665

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

Reviewed: https://review.openstack.org/23665
Committed: http://github.com/openstack/ceilometer/commit/43a99336f67a6cd2ae860c01678a52f7d286f391
Submitter: Jenkins
Branch: master

commit 43a99336f67a6cd2ae860c01678a52f7d286f391
Author: Lianhao Lu <email address hidden>
Date: Wed Mar 6 14:36:12 2013 +0800

    Handle missing units values in mongodb data.

    Provide a units value if it's missing in the existing mongodb data. This
    is for the backward compatibility issue when the data had been already
    stored before the counter_unit field was added in our meter definition.

    Fixed bug #1098603.

    Change-Id: I1ff3e325dc9b226e4441de6f5f9118fced67cf33

Changed in ceilometer:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in ceilometer:
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.