openstack/aodh
Code Issues Proposed changes

Merge "sqlalchemy: allow to upgrade schema from Ceilometer Liberty"

Browse Source
This commit is contained in:
Jenkins 2016-07-14 03:10:57 +00:00 committed by Gerrit Code Review
commit 1bea859df0
1 changed files with 10 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
aodh/storage/impl_sqlalchemy.py
View File

@ -20,6 +20,7 @@ import os.path
from alembic import command
from alembic import config
from alembic import migration
from oslo_db import exception
from oslo_db.sqlalchemy import session as db_session
from oslo_db.sqlalchemy import utils as oslo_sql_utils
from oslo_log import log
@ -94,8 +95,15 @@ class Connection(base.Connection):
ctxt = migration.MigrationContext.configure(engine.connect())
current_version = ctxt.get_current_revision()
if current_version is None:
models.Base.metadata.create_all(engine)
command.stamp(cfg, "head")
try:
models.Base.metadata.create_all(engine, checkfirst=False)
except exception.DBError:
# Assume tables exist from Ceilometer, take control
# FIXME(jd) Remove in Ocata
command.stamp(cfg, "12fe8fac9fe4")
command.upgrade(cfg, "head")
else:
command.stamp(cfg, "head")
else:
command.upgrade(cfg, "head")