heckyel/mediagoblin
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix #5391 - Alembic migrations would only work for SQLite

Browse Source 
The database connection was being set in a alembic.ini config file, if the user
had specified postgres the "sqlite" connection URL in alembic.ini would override
that. We probably should look into this more so i've opened #5395.
This commit is contained in:
Jessica Tallon 2016-01-18 09:39:34 +00:00
parent beb5a4a150
commit 2ddebb97bf
2 changed files with 1 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
alembic.ini
View File

@ -20,9 +20,6 @@ script_location = %(here)s/mediagoblin/db/migrations
# versions/ directory # versions/ directory
# sourceless = false # sourceless = false
sqlalchemy.url = sqlite:///mediagoblin.db
# Logging configuration # Logging configuration
[loggers] [loggers]
keys = root,sqlalchemy,alembic keys = root,sqlalchemy,alembic

1
mediagoblin/db/migration_tools.py
View File

@ -42,6 +42,7 @@ class AlembicMigrationManager(object):
os.path.dirname(__file__)))) os.path.dirname(__file__))))
alembic_cfg_path = os.path.join(root_dir, 'alembic.ini') alembic_cfg_path = os.path.join(root_dir, 'alembic.ini')
self.alembic_cfg = Config(alembic_cfg_path) self.alembic_cfg = Config(alembic_cfg_path)
self.alembic_cfg.set_main_option("sqlalchemy.url", str(session.get_bind().url))
self.session = session self.session = session
def get_current_revision(self): def get_current_revision(self):