Bug 1379519

Summary: Store glance image id as UUID
Product: [Retired] Beaker Reporter: matt jia <mjia>
Component: generalAssignee: Anwesha Chatterjee <achatter>
Status: CLOSED CURRENTRELEASE QA Contact: Dan Callaghan <dcallagh>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 23CC: achatter, dcallagh, mjia, rjoost
Target Milestone: 25.0Keywords: Patch
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2018-03-19 04:18:17 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description matt jia 2016-09-27 03:32:56 UTC
We should store the glance image id as UUID to be consistent with OpenStack instance id, network id and so on.

Comment 1 Dan Callaghan 2017-07-20 02:04:04 UTC
Specifically this is referring to the SQLAlchemy-level database type of the openstack_region.ipxe_image_id column, which is currently Unicode in SQLAlchemy (VARCHAR in the database). It should be UUID in SQLAlchemy (BINARY or whatever it is in the database).

All the places that refer to this property in Python code will need to be adjusted to handle the UUID type (everything might just work as is).

And we will need a database migration script to change the column's type *and* correctly preserve the existing values. This might be the hardest part.

Comment 2 Anwesha Chatterjee 2017-08-02 06:46:23 UTC
https://gerrit.beaker-project.org/#/c/5774/

Comment 5 Roman Joost 2018-03-19 04:18:17 UTC
Beaker 25.0 has been released.

Release notes are available upstream: https://beaker-project.org/docs/whats-new/release-25.html