diff options
| author | Dmitry Tantsur <dtantsur@protonmail.com> | 2022-10-24 11:42:33 +0200 |
|---|---|---|
| committer | Dmitry Tantsur <dtantsur@protonmail.com> | 2022-10-24 11:42:33 +0200 |
| commit | cec04bb04e621516e13d0ba1c600d7b3e0ab7c07 (patch) | |
| tree | 644f0ad5d430d80599738e19bbed1b1134003008 | |
| parent | ac8842a7755ea5d2f5086b5911abfb70261fc0df (diff) | |
| download | ironic-cec04bb04e621516e13d0ba1c600d7b3e0ab7c07.tar.gz | |
Do not disable autocommit until we fully migrate
Some dbapi calls are still relying on it (e.g. touch_conductor
apparently), causing sqlite databases to get locked.
Change-Id: If17d49ef434cf60876a81dae8e5ddaa6dc45e707
| -rw-r--r-- | ironic/db/sqlalchemy/__init__.py | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/ironic/db/sqlalchemy/__init__.py b/ironic/db/sqlalchemy/__init__.py index 0f792361a..173b91fcc 100644 --- a/ironic/db/sqlalchemy/__init__.py +++ b/ironic/db/sqlalchemy/__init__.py @@ -12,5 +12,7 @@ from oslo_db.sqlalchemy import enginefacade +# FIXME(stephenfin): we need to remove reliance on autocommit semantics ASAP +# since it's not compatible with SQLAlchemy 2.0 # NOTE(dtantsur): we want sqlite as close to a real database as possible. -enginefacade.configure(sqlite_fk=True) +enginefacade.configure(sqlite_fk=True, __autocommit=True) |