)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"ab55d3224052065fed941806aa69ae304369205b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"17bfe9e4_3e089d1c","updated":"2023-01-19 09:48:05.000000000","message":"few nits but changes LGTM. Thanks Gorka!\nRP+2 since DBMS plugin is broken and we need to backport and re-release cinderlib Zed to fix it.","commit_id":"5ded2caeb53fe8ce7c84ccac68b4a16d5788c66a"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"8188b246d1161128cf79b5a2ccd16ac7856824a5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"11db4d32_b9acb3e1","updated":"2023-01-20 04:03:17.000000000","message":"LGTM.","commit_id":"a123ff5f82938828367baaf04c70ffd917cff844"}],"cinderlib/tests/unit/persistence/helper.py":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"ab55d3224052065fed941806aa69ae304369205b","unresolved":true,"context_lines":[{"line_number":48,"context_line":""},{"line_number":49,"context_line":"        # Cannot just replace the context manager itself because it is already"},{"line_number":50,"context_line":"        # decorating cinder DB methods and those would continue accessing the"},{"line_number":51,"context_line":"        # old database, so we replace the existing CM\u0027sinternal transaction"},{"line_number":52,"context_line":"        # factory, efectively \"reseting\" the context manager."},{"line_number":53,"context_line":"        cm \u003d api.main_context_manager"},{"line_number":54,"context_line":"        if cm.is_started:"}],"source_content_type":"text/x-python","patch_set":1,"id":"4fd013a0_db656096","line":51,"range":{"start_line":51,"start_character":51,"end_line":51,"end_character":63},"updated":"2023-01-19 09:48:05.000000000","message":"nit: space needed\nCM\u0027s internal","commit_id":"1d80cf5167996e09e2995dfd1d902fdc64a8cbb7"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"ab55d3224052065fed941806aa69ae304369205b","unresolved":true,"context_lines":[{"line_number":49,"context_line":"        # Cannot just replace the context manager itself because it is already"},{"line_number":50,"context_line":"        # decorating cinder DB methods and those would continue accessing the"},{"line_number":51,"context_line":"        # old database, so we replace the existing CM\u0027sinternal transaction"},{"line_number":52,"context_line":"        # factory, efectively \"reseting\" the context manager."},{"line_number":53,"context_line":"        cm \u003d api.main_context_manager"},{"line_number":54,"context_line":"        if cm.is_started:"},{"line_number":55,"context_line":"            cm._root_factory \u003d api.enginefacade._TransactionFactory()"}],"source_content_type":"text/x-python","patch_set":1,"id":"77f2988e_ed98c502","line":52,"range":{"start_line":52,"start_character":31,"end_line":52,"end_character":39},"updated":"2023-01-19 09:48:05.000000000","message":"nit: resetting","commit_id":"1d80cf5167996e09e2995dfd1d902fdc64a8cbb7"}]}