)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"b2a1b8f16b11719486759111885b9ac3e8f7c645","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"08c1a838_d5e204c9","updated":"2023-09-28 20:29:29.000000000","message":"The cinder change that requires this (I39448af0eb8f) is present in cinder 2023.2 (Bobcat), so I think it\u0027s ok to change this now (while cinderlib is still doing Bobcat development).  Only a +1 because I think you should restore the comment noted inline.","commit_id":"705bce47fe0ddf521fec86aeb7c4814d02f462c3"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"ca1af377dd1a77b2df4bf1fe68edab5a0db16f76","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c017b47c_c4f5e5ec","updated":"2023-09-29 14:30:25.000000000","message":"Revision LGTM.  Thanks, Stephen!","commit_id":"f1af46dc6acd8026ed2ce612086c5b03c152af70"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"5d9b68b0d85497cfa5426e9f64d83bf118abd4c1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c92dc787_77bb6217","updated":"2023-09-29 13:01:11.000000000","message":"Thanks, LGTM","commit_id":"f1af46dc6acd8026ed2ce612086c5b03c152af70"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"e51c4893390d6856342d705f44b20a6b51ea288f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"3dc62153_83651c0a","updated":"2023-09-29 14:30:03.000000000","message":"recheck os-brick-src-tempest-lvm-lio-barbican : tempest.scenario.test_shelve_instance.TestShelveInstance.test_shelve_volume_backed_instance could not ssh into the instance","commit_id":"f1af46dc6acd8026ed2ce612086c5b03c152af70"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"e1cc64e8f9856186e5b93c891f2aa30e34623e56","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ee945aff_4e4f77a4","updated":"2023-09-29 17:37:36.000000000","message":"recheck os-brick-src-tempest-lvm-lio-barbican : this time it was cinder_tempest_plugin.api.volume.test_volume_backup.VolumesBackupsTest.test_incremental_backup that could not ssh into the instance","commit_id":"f1af46dc6acd8026ed2ce612086c5b03c152af70"}],"cinderlib/persistence/dbms.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"b2a1b8f16b11719486759111885b9ac3e8f7c645","unresolved":true,"context_lines":[{"line_number":100,"context_line":"        try:"},{"line_number":101,"context_line":"            migration.db_sync()"},{"line_number":102,"context_line":"        except exception.DBMigrationError as exc:"},{"line_number":103,"context_line":"            # We can be running 2 Cinder versions at the same time on the same"},{"line_number":104,"context_line":"            # DB while we upgrade, so we must ignore the fact that the DB is"},{"line_number":105,"context_line":"            # now on a newer version."},{"line_number":106,"context_line":"            if not isinstance(getattr(exc, \u0027inner_exception\u0027, None),"},{"line_number":107,"context_line":"                              migrate.exceptions.VersionNotFoundError):"},{"line_number":108,"context_line":"                raise"}],"source_content_type":"text/x-python","patch_set":1,"id":"252e5646_df13e004","side":"PARENT","line":105,"range":{"start_line":103,"start_character":0,"end_line":105,"end_character":37},"updated":"2023-09-28 20:29:29.000000000","message":"I think you should keep this comment so it\u0027s clear why we\u0027re ignoring this particular error.","commit_id":"98fc4b6c688950372c366bd431483da0c155768d"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"c3f94615410b70b9df271657ee8776e4d3e6303f","unresolved":false,"context_lines":[{"line_number":100,"context_line":"        try:"},{"line_number":101,"context_line":"            migration.db_sync()"},{"line_number":102,"context_line":"        except exception.DBMigrationError as exc:"},{"line_number":103,"context_line":"            # We can be running 2 Cinder versions at the same time on the same"},{"line_number":104,"context_line":"            # DB while we upgrade, so we must ignore the fact that the DB is"},{"line_number":105,"context_line":"            # now on a newer version."},{"line_number":106,"context_line":"            if not isinstance(getattr(exc, \u0027inner_exception\u0027, None),"},{"line_number":107,"context_line":"                              migrate.exceptions.VersionNotFoundError):"},{"line_number":108,"context_line":"                raise"}],"source_content_type":"text/x-python","patch_set":1,"id":"70af2f6f_80227772","side":"PARENT","line":105,"range":{"start_line":103,"start_character":0,"end_line":105,"end_character":37},"in_reply_to":"252e5646_df13e004","updated":"2023-09-29 11:19:04.000000000","message":"Done","commit_id":"98fc4b6c688950372c366bd431483da0c155768d"}]}