openstack
/
octavia
Code Issues Proposed changes

Fix migration scripts with SQLAlchemy 2.0 

select no longer accepts list/tuple as first argument but it takes
multiple arguments [0]

[0] https://docs.sqlalchemy.org/en/14/changelog/migration_20.html#select-no-longer-accepts-varied-constructor-arguments-columns-are-passed-positionally

Closes-Bug: #2061348
Change-Id: I271986146f485b2cd4ae3391d1a27f4a57f97560
This commit is contained in:
Gregory Thiemonge 2024-04-15 03:30:33 -04:00
parent d0c29009e6
commit 4e6121d9bc
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
octavia/db/migration/alembic_migrations/versions/29ff921a6eb_shared_pools.py
View File

@ -54,8 +54,8 @@ def upgrade():
sa.ForeignKey('load_balancer.id'), nullable=True))
# Populate this new column appropriately
select_obj = sa.select([listener.c.load_balancer_id,
listener.c.default_pool_id]).where(
select_obj = sa.select(listener.c.load_balancer_id,
listener.c.default_pool_id).where(
listener.c.default_pool_id is not None)
result = conn.execute(select_obj)
for row in result:

2
octavia/db/migration/alembic_migrations/versions/dcf88e59aae4_add_lb_algorithm_source_ip_port.py
View File

@ -43,7 +43,7 @@ def migrate_pools():
j = pool_table.join(lb_table,
pool_table.c.load_balancer_id == lb_table.c.id)
stmt = sa.select([pool_table.c.id]).select_from(j).where(
stmt = sa.select(pool_table.c.id).select_from(j).where(
lb_table.c.provider == 'ovn')
result = conn.execute(stmt)