041 migration script mucked up column types

Bug #1423710 reported by gordon chung
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
High
gordon chung

Bug Description

sa.float should be sa.float(53) and sa.text should be sa.string(255) as it was originally

https://review.openstack.org/#/c/130869/21/ceilometer/storage/sqlalchemy/migrate_repo/versions/041_expand_event_traits.py

gordon chung (chungg)
Changed in ceilometer:
milestone: none → kilo-3
description: updated
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/157549

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

Reviewed: https://review.openstack.org/157549
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=a047282b0acb5120ae9bc1e6063874919e5b8fbd
Submitter: Jenkins
Branch: master

commit a047282b0acb5120ae9bc1e6063874919e5b8fbd
Author: gordon chung <email address hidden>
Date: Thu Feb 19 17:06:02 2015 -0500

    correct column types in events

    in migration script, we incorrectly set string traits to Text and
    float traits to Float. this fixes it so they point to String(255)
    and Float(53) as they did originally

    Change-Id: Ie06781a42ec7384d88bea8498140e52f109e9f7e
    Closes-Bug: #1423710

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: 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.