openstack/glance
Code Issues Proposed changes

Get rid of DeprecationWarning during db migration

Browse Source 
Use proper syntax to set column type, which avoids printing
silly warnings and future deprecation issues.
Fixes bug 854163.

Change-Id: Iab1c3e7a52d739a445cf52eb9a67f61a69075026
This commit is contained in:
Thierry Carrez 2012-02-03 16:54:30 +01:00
parent 300d4031a9
commit c6c458e6b6
1 changed files with 2 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
glance/registry/db/migrate_repo/versions/012_id_to_uuid.py
View File

@ -71,9 +71,7 @@ def _upgrade_sqlite(t_images, t_image_members, t_image_properties):
"""
Upgrade 011 -> 012 with special SQLite-compatible logic.
"""
t_images.c.id.alter(sqlalchemy.Column("id",
sqlalchemy.String(36),
primary_key=True))
t_images.c.id.alter(sqlalchemy.String(36), primary_key=True)
sql_commands = [
"""CREATE TABLE image_members_backup (
@ -121,9 +119,7 @@ def _downgrade_sqlite(t_images, t_image_members, t_image_properties):
"""
Downgrade 012 -> 011 with special SQLite-compatible logic.
"""
t_images.c.id.alter(sqlalchemy.Column("id",
sqlalchemy.Integer(),
primary_key=True))
t_images.c.id.alter(sqlalchemy.Integer(), primary_key=True)
sql_commands = [
"""CREATE TABLE image_members_backup (