From 2f41030c17b01278e0c57b807da367c4393651f8 Mon Sep 17 00:00:00 2001 From: Zuul Date: Mon, 12 Jul 2021 16:03:14 +0000 Subject: [PATCH] Update git submodules * Update tacker from branch 'master' to a98cd4eaa9f2c1669f0a1d1c748d6d0028edd1f4 - Merge "Fix migration for SQLAlchemy 1.4" - Fix migration for SQLAlchemy 1.4 This patch fixes a database migration for SQLAlchemy 1.4. In tacker repository following issues are encounter: 1. Replace deprecated SQLAlchemy "with_lockmode" method. The method was deprecated in SQLAlchemy 0.9 [1][2] and finally removed in version 1.4. This patch replaces "with_lockmode" with new method "with_for_update", which has no end-user impact. 2. The clause column IN, in the new version it requires parameters passed to operator either be a list of literal value or a tuple, or an empty list [3]. 3. Observe an argument error in 'update' query. Error: sqlalchemy.exc.ArgumentError: subject table for an INSERT, UPDATE or DELETE expected, got Column('id', Uuid(length=36), table=, primary_key=True, nullable=False, default=ColumnDefault()) 4. The INSERT statement requires values to add in table as a list, tuple or dictionary [4]. Error: sqlalchemy.exc.ArgumentError: mapping or sequence expected for parameters This patch creates a dictionary of fields to be insert in VnfLcmOpOccs table. 5. Query fails on applying filters. When applying query to list vnf package or vnf instance using apply_filters() method exported from sqlalchemy_filters, it failed. AttributeError: 'Query' object has no attribute '_join_entities' In SQLAlchemy 1.4 unification of "query" and "select" construct took place [5][6]. The issue observe in tacker is an open bug of SQLAlchemy [7]. This patch drops "sqlalchemy_filters" and add customize method to apply filters in query. Note: This patch address sqlalchemy errors, SAWarnings will be resolve in future patches. [1] https://docs.sqlalchemy.org/en/13/changelog/migration_09.html?highlight=with_lockmode#new-for-update-support-on-select-query [2] https://docs.sqlalchemy.org/en/13/orm/query.html#sqlalchemy.orm.query.Query.with_lockmode [3] https://docs.sqlalchemy.org/en/14/core/sqlelement.html#sqlalchemy.sql.expression.ColumnElement.in_ [4] https://docs.sqlalchemy.org/en/14/core/dml.html#dml-class-documentation-constructors [5] https://github.com/sqlalchemy/sqlalchemy/issues/5159 [6] https://gerrit.sqlalchemy.org/c/sqlalchemy/sqlalchemy/+/1599/ [7] https://github.com/juliotrigo/sqlalchemy-filters/issues/61 Co-Authored-By: Ayumu Ueha Change-Id: Ifd1ae2753c639f22fc1afa020222416fe79469ef --- tacker | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tacker b/tacker index 987f60edd1..a98cd4eaa9 160000 --- a/tacker +++ b/tacker @@ -1 +1 @@ -Subproject commit 987f60edd1fc6dbbe74bce9cb48fb1f346b0afcc +Subproject commit a98cd4eaa9f2c1669f0a1d1c748d6d0028edd1f4