Comment 5 for bug 1397796

Reviewed: https://review.openstack.org/138933
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=d7ee2ff991e60ad54ff2125a397e9293ba966549
Submitter: Jenkins
Branch: stable/juno

commit d7ee2ff991e60ad54ff2125a397e9293ba966549
Author: Mike Bayer <email address hidden>
Date: Sun Nov 30 22:18:32 2014 -0500

    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)