028 migration script incorrectly skips over section

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

Bug Description

in https://github.com/openstack/ceilometer/blob/master/ceilometer/storage/sqlalchemy/migrate_repo/versions/028_alembic_migrations.py#L133

the if clause incorrectly compares db engine name... because of this, it always skips applying former alembic script '17738166b91'.

gordon chung (chungg)
Changed in ceilometer:
importance: Undecided → High
status: New → In Progress
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/80455

Changed in ceilometer:
assignee: nobody → gordon chung (chungg)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ceilometer (master)

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

commit 8fde5b3a3e292e3cea2e952d874f0e1f1626a6e6
Author: Gordon Chung <email address hidden>
Date: Thu Mar 13 22:40:50 2014 -0400

    028 migration script incorrectly skips over section

    028 migration patch for alembic scripts incorrectly skips over
    applying former alembic script '17738166b91' when it is needed.

    Change-Id: I86a49f1979d8c5a8d5e5f15da8c007216b626928
    Closes-Bug: #1292312

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