)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"b53570b6dc0a8ebc2888450c810aa4fc4ba87c8b","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"93f1f653_9ecc9280","updated":"2026-04-15 14:08:31.000000000","message":"This is a variation of Sean\u0027s suggestion from https://review.opendev.org/c/openstack/nova/+/983069  \n\nI think both patches is descirbing a valid direction (file base SQLite and possible WAL) but both misses the fact that we have different code paths creating DBs and only handle some of them. We need to apply the pattern consistently.","commit_id":"214c78291d15592afcef198a35bc873ce6e0706b"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"b9c4e1902e489a8428d0b70237ba4847c78c081a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"5df43dc7_59236d77","updated":"2026-04-14 18:42:19.000000000","message":"recheck flaky unit test fix proposed https://review.opendev.org/c/openstack/nova/+/984617","commit_id":"214c78291d15592afcef198a35bc873ce6e0706b"}],"nova/tests/fixtures/nova.py":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"b53570b6dc0a8ebc2888450c810aa4fc4ba87c8b","unresolved":true,"context_lines":[{"line_number":519,"context_line":"        # will house the sqlite:// connection for this cell\u0027s in-memory"},{"line_number":520,"context_line":"        # database. Store/index it by the connection string, which is"},{"line_number":521,"context_line":"        # how we identify cells in CellMapping."},{"line_number":522,"context_line":"        ctxt_mgr \u003d main_db_api.create_context_manager()"},{"line_number":523,"context_line":"        self._ctxt_mgrs[connection_str] \u003d ctxt_mgr"},{"line_number":524,"context_line":""},{"line_number":525,"context_line":"        # NOTE(melwitt): The first DB access through service start is"}],"source_content_type":"text/x-python","patch_set":1,"id":"4177ec20_0ce08120","line":522,"updated":"2026-04-15 14:08:31.000000000","message":"Note that this also creates a DB. If this fixture is used the the cell DBs are not created at L610 but created here instead. We need to apply the same file based pattern everywhere where we create a DB.","commit_id":"14cda299bbb19928cda3dc96855d63c64ed6cb19"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"b53570b6dc0a8ebc2888450c810aa4fc4ba87c8b","unresolved":true,"context_lines":[{"line_number":546,"context_line":"        ):"},{"line_number":547,"context_line":"            engine \u003d ctxt_mgr.writer.get_engine()"},{"line_number":548,"context_line":"            engine.dispose()"},{"line_number":549,"context_line":"            self._cache_schema(connection_str)"},{"line_number":550,"context_line":"            conn \u003d engine.connect()"},{"line_number":551,"context_line":"            conn.connection.executescript(DB_SCHEMA[(\u0027main\u0027, None)])"},{"line_number":552,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"09136692_e1f67693","line":549,"updated":"2026-04-15 14:08:31.000000000","message":"This is another place we apply schema to the DB, so a potential place to apply WAL","commit_id":"14cda299bbb19928cda3dc96855d63c64ed6cb19"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"b53570b6dc0a8ebc2888450c810aa4fc4ba87c8b","unresolved":true,"context_lines":[{"line_number":679,"context_line":"        else:"},{"line_number":680,"context_line":"            api_db_api.context_manager.configure(connection\u003ddb_url)"},{"line_number":681,"context_line":""},{"line_number":682,"context_line":"    def _apply_schema(self):"},{"line_number":683,"context_line":"        if not DB_SCHEMA[(self.database, self.version)]:"},{"line_number":684,"context_line":"            # apply and cache schema"},{"line_number":685,"context_line":"            engine \u003d self.get_engine()"}],"source_content_type":"text/x-python","patch_set":1,"id":"2882f3ac_7b1e6297","line":682,"updated":"2026-04-15 14:08:31.000000000","message":"We have another apply_schema codepath, see above","commit_id":"14cda299bbb19928cda3dc96855d63c64ed6cb19"}]}
