Fix Migration 50e86cb2637a

Bug #1267619 reported by Aaron Rosen
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
High
Aaron Rosen

Bug Description

The following migration 50e86cb2637a called
op.rename_table('neutron_nvp_port_mapping', 'neutron_nsx_port_mappings')
though the table name was actually quantum_nvp_port_mapping. Because of this
the quantum_id->nvp_id mapping was never migrated over to the new table and
you would be left with a quantum_nvp_port_mapping table hanging around.

In addition, the downgrade would rename the table to neutron_nvp_port_mapping
instead of quantum_nvp_port_mapping. This patch addresses this issues.

Aaron Rosen (arosen)
Changed in neutron:
assignee: nobody → Aaron Rosen (arosen)
importance: Undecided → High
tags: added: havana-backport-potential
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (master)

Fix proposed to branch: master
Review: https://review.openstack.org/65792

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

Reviewed: https://review.openstack.org/65792
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=251159c90a69dfb61d9927093d5eae41cd99e2a7
Submitter: Jenkins
Branch: master

commit 251159c90a69dfb61d9927093d5eae41cd99e2a7
Author: Aaron Rosen <email address hidden>
Date: Thu Jan 9 13:49:10 2014 -0800

    Fix Migration 50e86cb2637a and 38335592a0dc

    When the rename of quantum->neutron occurred here ee3fe4e8 it also renamed
    the the table creation from quantum_nvp_port_mapping to
    neutron_nvp_port_mapping. This went undetected for a long time because
    when neutron-server starts up it pushes down the scheme for tables that
    are not there so the table would be created.

    Because of this the following migration 50e86cb2637a called
    op.rename_table('neutron_nvp_port_mapping', 'neutron_nsx_port_mappings')
    though the table name being used was quantum_nvp_port_mapping. Because of this
    the quantum_id->nvp_id mapping was never migrated over to the new table and
    you would be left with a quantum_nvp_port_mapping table hanging around.

    In addition, the downgrade would rename the table to neutron_nvp_port_mapping
    instead of quantum_nvp_port_mapping. This patch addresses this issues.

    Change-Id: I4f80b7b9dc56996ecd83826ee65918f5311c7c4f
    Closes-bug: #1267619

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