Ceilometer

sqlalchemy record_meter resource merge find duplicate rows

Bug #1066145 reported by John Tran
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
Undecided
John Tran
Ceilometer 2013.1 "grizzly"

Bug Description

Resource model is expecting 'id' column to be of String type yet the data['resource_id'] in line 149 of impl_sqlalchemy.py ; resource = self.session.merge(Resource(id=data['resource_id'])) ; is sometimes sending an integer such as '1' and causing sqlalchemy backend to think there are more than 1 rows.

John Tran (jtran)
Changed in ceilometer:
assignee: nobody → John Tran (jtran)
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/14707

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/14707
Committed: http://github.com/stackforge/ceilometer/commit/6ddc4159d557aefd517cd9c2712c14e5b86ed707
Submitter: Jenkins
Branch: master

commit 6ddc4159d557aefd517cd9c2712c14e5b86ed707
Author: John Tran <email address hidden>
Date: Tue Oct 23 20:46:30 2012 +0000

    sqlalchemy record_meter merge objs not string

    Fixes bug # 1066145. Models expecting String

    Change-Id: Idbf39c29267f7d82f28cb22e5353415f6acb738e

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

Other bug subscribers

Remote bug watches

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