Magnum

The nullable parameter is not necessary

Bug #1482122 reported by Hua Wang
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Magnum
Won't Fix
Undecided
Hua Wang
Magnum mitaka-1

Bug Description

The default of the nullable parameter in sqlalchemy Column class is true. So we don't need to use the parameter when we need a nullable column in allembic.

Hua Wang (humble00)
Changed in magnum:
assignee: nobody → Hua Wang (humble00)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to magnum (master)

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

Changed in magnum:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on magnum (master)

Change abandoned by Hua Wang (<email address hidden>) on branch: master
Review: https://review.openstack.org/209826

Adrian Otto (aotto)
Changed in magnum:
milestone: none → mitaka-1
Revision history for this message
yatin (yatinkarel) wrote :

I think the bug should be marked as won't fix as per comments in the patch submitted.

Tom Cammann (tom-cammann)
Changed in magnum:
status: In Progress → Won't Fix
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.