]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commit
Alter execute_alembic_command() to not assume all commands
authorMike Bayer <mike_mp@zzzcomputing.com>
Mon, 1 Dec 2014 03:18:32 +0000 (22:18 -0500)
committerAdam Gandelman <adamg@ubuntu.com>
Thu, 4 Dec 2014 01:51:16 +0000 (17:51 -0800)
commitd7ee2ff991e60ad54ff2125a397e9293ba966549
treeb15d5415476eb6f7b5b87dedc7c9b304f1549b24
parent181310428c8781fe5ffedd80d08b6955cb26524f
Alter execute_alembic_command() to not assume all commands

Alembic 0.7.1 is adding new diff directives add_fk and remove_fk.
The logic in execute_alembic_command() appears to assume all potential
commands are present in METHODS.   This patch adjusts it so that
it checks METHODS first, allowing the tests in neutron.tests.unit.db.test_migration
to pass.  However, if the design here requires that all potential
Alembic commands are handled, then this would not be the fix,
though I recommend that the potential list of Alembic directives
not be hardcoded.

Change-Id: Ib08685be252e1fdb0f10853f8fc9a7e19ac54a94
Closes-Bug: #1397796
(cherry picked from commit 6a9d1121472028f3583e577b62a7e576fff5806f)
neutron/db/migration/alembic_migrations/heal_script.py