]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commitdiff
fullstack: use migration scripts to create db schema
authorJakub Libosvar <libosvar@redhat.com>
Thu, 20 Aug 2015 16:02:11 +0000 (16:02 +0000)
committerJakub Libosvar <libosvar@redhat.com>
Thu, 20 Aug 2015 16:06:10 +0000 (16:06 +0000)
Previously, we used create_all() based on models. We don't use
create_all() in production code and there is no guarantee models and
scripts are in sync even though we have a good functional test that
validates that. There are still pieces that can't be compared by
alembic.

Change-Id: I72fa67811f0763298416e6e084a8b9b86619795b
Closes-Bug: 1486528

neutron/tests/fullstack/base.py

index 579831524f0f34da44c605ad5c73dcc279ecfe41..ca2218da957a9abe9f9dd675360b4be8c04e4975 100644 (file)
@@ -15,8 +15,7 @@
 from oslo_config import cfg
 from oslo_db.sqlalchemy import test_base
 
-from neutron.db.migration.models import head  # noqa
-from neutron.db import model_base
+from neutron.db.migration import cli as migration
 from neutron.tests.common import base
 from neutron.tests.fullstack.resources import client as client_resource
 
@@ -62,11 +61,13 @@ class BaseFullStackTestCase(base.MySQLTestCase):
                     'password': test_base.DbFixture.PASSWORD,
                     'db_name': self.engine.url.database})
 
+        alembic_config = migration.get_neutron_config()
+        alembic_config.neutron_config = cfg.CONF
         self.original_conn = cfg.CONF.database.connection
         self.addCleanup(self._revert_connection_address)
         cfg.CONF.set_override('connection', conn, group='database')
 
-        model_base.BASEV2.metadata.create_all(self.engine)
+        migration.do_alembic_command(alembic_config, 'upgrade', 'heads')
 
     def _revert_connection_address(self):
         cfg.CONF.set_override('connection',