openstack
/
openstack
Code Issues Proposed changes

Update git submodules 

* Update heat from branch 'master'
  to 921b5b75908b877c8e20e7e914d9612f782bbd1e
  - Merge "db: Don't rely on branched connections"
  - db: Don't rely on branched connections
    
    We were previously calling 'connect()' on the 'connectable' object in
    'run_migrations_online', regardless of whether it was an 'Engine' or
    'Connection' object. This worked because, as noted in an inline comment,
    "when connectable is already a Connection object, calling 'connect()'
    gives us a *branched connection*." This is no longer the case. From the
    SQLAlchemy docs [1]:
    
      The Connection object does not support "branching", which was a
      pattern by which a sub "connection" would be used that refers to this
      connection as a parent.
    
    Update our code to reflect this change, using the newly updated example
    from the SQLAlchemy cookbook doc [2] as inspiration.
    
    [1] https://docs.sqlalchemy.org/en/14/core/future.html#sqlalchemy.future.Connection
    [2] https://alembic.sqlalchemy.org/en/latest/cookbook.html#connection-sharing
    
    Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
    Change-Id: Id41ae6d3fa3f343efdb727aae9bdef8ad2a098fd
This commit is contained in:
Zuul 2023-10-19 05:37:52 +00:00 committed by Gerrit Code Review
parent 5ee6790fa5
commit f424fe6392
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
heat

@ -1 +1 @@
Subproject commit 829f6c82be83f8460fd58161afbd71a31eb4914d
Subproject commit 921b5b75908b877c8e20e7e914d9612f782bbd1e