Missing migration for NVP plugin

Bug #1099895 reported by Salvatore Orlando
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Medium
Salvatore Orlando

Bug Description

A model used by Nicira NVP plugin is not included in folsom_initial migration, so a new migration is required:

class NvpNetworkBinding(model_base.BASEV2):
    """Represents a binding of a virtual network with a transport zone.

    This model class associates a Quantum network with a transport zone;
    optionally a vlan ID might be used if the binding type is 'bridge'
    """
    __tablename__ = 'nvp_network_bindings'

    network_id = Column(String(36),
                        ForeignKey('networks.id', ondelete="CASCADE"),
                        primary_key=True)
    # 'flat', 'vlan', stt' or 'gre'
    binding_type = Column(Enum('flat', 'vlan', 'stt', 'gre'), nullable=False)
    tz_uuid = Column(String(36))
    vlan_id = Column(Integer)

Each plugin is currently providing its own network binding model, so the ones created for instance, for the OVS plugin, won't work with the NVP plugin.

This is likely to be happened because the patches for alembic migrations and provider networks on NVP where concurrently under review.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to quantum (master)

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

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

Reviewed: https://review.openstack.org/19709
Committed: http://github.com/openstack/quantum/commit/2d4431d7bc6b7ab362ce2cf084b901be45c51c2f
Submitter: Jenkins
Branch: master

commit 2d4431d7bc6b7ab362ce2cf084b901be45c51c2f
Author: Salvatore Orlando <email address hidden>
Date: Tue Jan 15 07:49:04 2013 -0800

    Add migration for network bindings in NVP plugin

    Bug 1099895

    Ensures the table nvp_network_bindings is created when upgrading
    database to head, by adding an appropriate alembic migration

    Change-Id: I4a794ed0ec6866d657cb2470d5aa67828e81aa75

Changed in quantum:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in quantum:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in quantum:
milestone: grizzly-3 → 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.