]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Make manage.py usable
authorMathieu Gagné <mgagne@iweb.com>
Thu, 7 Aug 2014 23:21:49 +0000 (19:21 -0400)
committerMathieu Gagné <mgagne@iweb.com>
Mon, 11 Aug 2014 16:47:02 +0000 (12:47 -0400)
commit186c3e99cf04b22ddbb1046cc9e42d1f7473e8cf
tree563b6f750bd46a1905b8e32a163678896a08d3f8
parente6e0348f95f0065dcf3c8af992f463324ba7fabb
Make manage.py usable

It's not possible to use manage.py in its current state due to
an exception thrown by oslo.config:

  oslo.config.cfg.ArgsAlreadyParsedError:
    arguments already parsed: cannot register CLI option

This exception is a side-effect of including
cinder.openstack.common.logging in most of the migration scripts.

This change therefore suggests removing oslo.config and other imports
from manage.py to make it usable again.

It is now possible to use manage.py:

  tox -evenv -- python cinder/db/sqlalchemy/migrate_repo/manage.py \
    version_control --url=sqlite:///./cinder.db

  tox -evenv -- python cinder/db/sqlalchemy/migrate_repo/manage.py \
    upgrade --url=sqlite:///./cinder.db

Closes-bug: #1355297
Change-Id: I843cfa9a4e4758a6bc8e5c04f064e9d3107b01fa
cinder/db/sqlalchemy/migrate_repo/manage.py