Skip to content

Commit

Permalink
Update git submodules
Browse files Browse the repository at this point in the history
* Update nova from branch 'master'
  to 9c2d9fd1eb5bbfd32fb99f367d0da1d1de51c330
  - Merge "db: De-duplicate list of removed table columns"
  - db: De-duplicate list of removed table columns
    
    We have a policy of removing fields from SQLAlchemy models at least one
    cycle before we remove the underlying database columns. This can result
    in a discrepancy between the state that our newfangled database
    migration tool, alembic, sees and what's actually in the database. We
    were ignoring these removed fields (and one foreign key constraint) in
    two different locations for both databases: as part of the alembic
    configuration and as part of the tests we have to ensure our migrations
    are in sync with our models (note: the tests actually use the alembic
    mechanism to detect the changes [1]). De-duplicate these.
    
    [1] sqlalchemy/alembic#724 (comment)
    
    Change-Id: I978b4e44cf7f522a70cc5ca76e6d6f1a985d5469
    Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
  • Loading branch information
Zuul authored and openstack-gerrit committed Nov 3, 2021
1 parent 244396e commit b12596e
Showing 1 changed file with 1 addition and 1 deletion.

0 comments on commit b12596e

Please sign in to comment.