lp:~markmc/nova/sqlalchemy-0.7-migration-fixes

Branch merges

Propose for merging

No branches dependent on this one.

Merged into lp:~hudson-openstack/nova/trunk at revision 1499

Joshua McKenty (community): Approve on 2011-08-26

Brian Lamar (community): Approve on 2011-08-25

Diff: 92 lines (+25/-10)

3 files modified

Authors (+1/-0)
nova/db/sqlalchemy/migrate_repo/versions/016_make_quotas_key_and_value.py (+14/-4)
nova/db/sqlalchemy/migrate_repo/versions/036_change_flavor_id_in_migrations.py (+10/-6)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Related blueprints

1486. By Mark McLoughlin on 2011-08-24

Add me to Authors

1485. By Mark McLoughlin on 2011-08-24

Fix quotas migration failure

With sqlalchemy 0.7.2 and migrate 0.7.1, I was seeing:

   Traceback (most recent call last):
     File "/usr/lib/python2.7/site-packages/nova/db/migration.py", line 37, in db_sync
       ret = IMPL.db_sync(version=version)
     [..]
     File "/usr/lib/python2.7/site-packages/nova/db/sqlalchemy/migrate_repo/versions/016_make_quotas_key_and_value.py", line 189, in upgrade
       new_quotas.rename('quotas')
     [..]
     File "/usr/lib/python2.7/site-packages/migrate/changeset/schema.py", line 479, in deregister
       del meta.tables[key]
     File "/usr/lib64/python2.7/site-packages/sqlalchemy/util/_collections.py", line 38, in _immutable
       raise TypeError("%s object is immutable" % self.__class__.__name__)
   TypeError: immutabledict object is immutable

This is actually a bug in sqlalchemy-migrate:

  http://code.google.com/p/sqlalchemy-migrate/issues/detail?id=128

But it can be worked around by ensuring there isn't a 'quotas' table in
the metadata's table hash before renaming.

1484. By Mark McLoughlin on 2011-08-24

Fix flavorid migration failure

With sqlalchemy 0.7.2 and migrate 0.7.1, I was seeing:

   Traceback (most recent call last):
      File "/usr/lib/python2.7/site-packages/nova/db/migration.py", line 37, in db_sync
        ret = IMPL.db_sync(version=version)
      [...]
      File "/usr/lib/python2.7/site-packages/nova/db/sqlalchemy/migrate_repo/versions/036_change_flavor_id_in_migrations.py", line 46, in upgrade
        .values(old_instance_type_id=instance_type.id))
      [...]
      File "/usr/lib64/python2.7/site-packages/sqlalchemy/engine/default.py", line 301, in do_commit
        connection.commit()
    OperationalError: (OperationalError) database is locked None None

It looks like the database is being held open as we iterate over the
rows in the instance_types table.

1483. By Vish Ishaya on 2011-08-23

Adds a use_deprecated_auth flag to make sure creds generated using nova-manage commands will work with noauth.

1482. By Nachi Ueno on 2011-08-23

I added notifications decorator for each API call using monkey_patching.
By this merge, users can get API call notification from any modules.

1481. By Brian Waldon on 2011-08-23

Fixes bug that causes 400 status code when an instance wasn't attached to a network.

1480. By Dan Prince on 2011-08-23

Move use_ipv6 into flags. Its used in multiple places (network manager and the OSAPI) and should be defined at the top level.

1479. By Thierry Carrez on 2011-08-23

Refresh translations

1478. By Vish Ishaya on 2011-08-23

This branch does the final tear out of AuthManager from the main code. The NoAuth middlewares (active by default) allow a user to specify any user and project id through headers (os_api) or access key (ec2_api).

The plan is to leave the auth manager code in but mention that it is deprecated. There is a sample paste config in ini to still allow old auth. Immediately after the diablo release we can tear out all of the Auth related code and not support the deprecated auth anymore.

1477. By Christopher MacGown <email address hidden> on 2011-08-23

Implements first-pass of config-drive that adds a vfat format drive to a vm when config_drive is True (or an image id).