config_drive DB column cleanup

Bug #1193438 reported by Chris Behrens
16
This bug affects 2 people
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Low
Unassigned

Bug Description

config_drive doesn't support image IDs anymore. The config_drive column currently is in a weird state where an empty string value means False... and code relies on the DB casting True to a string.

Chris Behrens (cbehrens)
Changed in nova:
status: New → Triaged
importance: Undecided → Low
Michael Still (mikal)
tags: added: db
Yaguang Tang (heut2008)
Changed in nova:
assignee: nobody → Yaguang Tang (heut2008)
Changed in nova:
status: Triaged → In Progress
Revision history for this message
Michael Still (mikal) wrote :

I've reviewed the code review for this one, and I think changing this is a bad idea.

Revision history for this message
Matt Riedemann (mriedem) wrote :

Marking this as New since I'm not seeing any progress or review linked to this.

Changed in nova:
status: In Progress → New
Revision history for this message
Matt Riedemann (mriedem) wrote :

We should at least get the docs updated for the config-drive API extension since it lists the parameter as a string uuid value:

http://api.openstack.org/api-ref-compute-v2-ext.html

config_drive (Optional) plain csapi:UUID ID of image to mount within server as a configuration drive.

tags: added: documentation
Revision history for this message
Tracy Jones (tjones-i) wrote :

are we just going to close this as a doc bug?

Revision history for this message
Chinmay Naik (chinmay-naik1) wrote :

Have created https://bugs.launchpad.net/openstack-api-site/+bug/1339392 to fix the documentation in the meanwhile. Will put in the change once the bug gets confirmed.

Revision history for this message
Sean Dague (sdague) wrote :

Doc fix has landed

Changed in nova:
status: New → Invalid
Revision history for this message
Stephen Finucane (stephenfinucane) wrote :

There has been no progress on this since 2013 so I'm going to have a look. Hope that's OK

Changed in nova:
assignee: Yaguang Tang (heut2008) → Stephen Finucane (sfinucan)
Revision history for this message
Stephen Finucane (stephenfinucane) wrote :

This was partially fixed in '296479e1a'

Changed in nova:
assignee: Stephen Finucane (sfinucan) → nobody
Revision history for this message
Stephen Finucane (stephenfinucane) wrote :

In fact, it's completely fixed seeing as we'll never mess with the actual DB (doing so would involve a migration: not worth it for something non-devs don't care about).

Changed in nova:
status: Invalid → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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