← Back to branch summary

~mirantis/nova/non-shared-migrate

« back to all changes in this revision

Viewing changes to nova/db/sqlalchemy/migrate_repo/versions/017_make_instance_type_id_an_integer.py

Convert instance_type_ids in the instances table from strings to integers to enable joins with instance_types. This in particular fixes a problem when using postgresql.

Show diffs side-by-side

added added

removed removed

Lines of Context:
nova/db/sqlalchemy/migrate_repo/versions/017_make_instance_type_id_an_integer.py
 
1
from sqlalchemy import Column, Integer, MetaData, String, Table
 
2
from nova import log as logging
 
3
 
 
4
meta = MetaData()
 
5
 
 
6
 
 
7
def upgrade(migrate_engine):
 
8
    meta.bind = migrate_engine
 
9
    instances = Table('instances', meta, autoload=True,
 
10
                      autoload_with=migrate_engine)
 
11
 
 
12
    types = {}
 
13
    for instance in migrate_engine.execute(instances.select()):
 
14
        if instance.instance_type_id is None:
 
15
            types[instance.id] = None
 
16
            continue
 
17
        try:
 
18
            types[instance.id] = int(instance.instance_type_id)
 
19
        except ValueError:
 
20
            logging.warn("Instance %s did not have instance_type_id "
 
21
                         "converted to an integer because its value is %s" %
 
22
                          (instance.id, instance.instance_type_id))
 
23
            types[instance.id] = None
 
24
 
 
25
    integer_column = Column('instance_type_id_int', Integer(), nullable=True)
 
26
    string_column = instances.c.instance_type_id
 
27
 
 
28
    integer_column.create(instances)
 
29
    for instance_id, instance_type_id in types.iteritems():
 
30
        update = instances.update().\
 
31
                where(instances.c.id == instance_id).\
 
32
                values(instance_type_id_int=instance_type_id)
 
33
        migrate_engine.execute(update)
 
34
 
 
35
    string_column.alter(name='instance_type_id_str')
 
36
    integer_column.alter(name='instance_type_id')
 
37
    string_column.drop()
 
38
 
 
39
 
 
40
def downgrade(migrate_engine):
 
41
    meta.bind = migrate_engine
 
42
    instances = Table('instances', meta, autoload=True,
 
43
                      autoload_with=migrate_engine)
 
44
 
 
45
    integer_column = instances.c.instance_type_id
 
46
    string_column = Column('instance_type_id_str',
 
47
                           String(length=255, convert_unicode=False,
 
48
                                  assert_unicode=None, unicode_error=None,
 
49
                                  _warn_on_bytestring=False),
 
50
                           nullable=True)
 
51
 
 
52
    types = {}
 
53
    for instance in migrate_engine.execute(instances.select()):
 
54
        if instance.instance_type_id is None:
 
55
            types[instance.id] = None
 
56
        else:
 
57
            types[instance.id] = str(instance.instance_type_id)
 
58
 
 
59
    string_column.create(instances)
 
60
    for instance_id, instance_type_id in types.iteritems():
 
61
        update = instances.update().\
 
62
                where(instances.c.id == instance_id).\
 
63
                values(instance_type_id_str=instance_type_id)
 
64
        migrate_engine.execute(update)
 
65
 
 
66
    integer_column.alter(name='instance_type_id_int')
 
67
    string_column.alter(name='instance_type_id')
 
68
    integer_column.drop()