primary key set to allow null

Bug #1103216 reported by Mark McClain
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Undecided
Mark McClain

Bug Description

The IPAvailabilityRange model declared allocation_pool_id as nullable and as a component of the primary_key. Under the hood, the primary_key declaration conflicts with the nullable attribute and causes the nullable attribute to be set to False within the database. The model should be updated to be consistent with proper SQL.

description: updated
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/20285

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

Reviewed: https://review.openstack.org/20285
Committed: http://github.com/openstack/quantum/commit/cbbb11ef6e9dff745720ffc45e10ee589af553b6
Submitter: Jenkins
Branch: master

commit cbbb11ef6e9dff745720ffc45e10ee589af553b6
Author: Mark McClain <email address hidden>
Date: Tue Jan 22 17:48:09 2013 -0500

    set allocation_pool_id nullable=False

    fixes bug 1103216

    SQL Primary Keys cannot be null, so this patch fixes an
    inconsistency between the models and SQL. The databases were
    correcting this automatically, so a migration is not required.

    Change-Id: I724294580de9b86a2edfd4fdc6907e9469b6c552

Changed in quantum:
status: In Progress → Fix Committed
Akihiro Motoki (amotoki)
Changed in quantum:
milestone: none → grizzly-3
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.