)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"23fca743d05979f748087151323b7eea267ab133","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7b55c766_98beadaa","updated":"2024-06-27 13:41:03.000000000","message":"Thank you!","commit_id":"b55924ee001c2296544051cdc24f057adfa1f579"}],"glance/api/v2/images.py":[{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"6b6c8564d8132e1c65add670d3a3e77f94091e9e","unresolved":true,"context_lines":[{"line_number":921,"context_line":"                    \"exc\": e.msg})"},{"line_number":922,"context_line":"            LOG.warning(msg)"},{"line_number":923,"context_line":"            raise webob.exc.HTTPConflict(explanation\u003dmsg)"},{"line_number":924,"context_line":"        except glance_store.exceptions.BackendException as e:"},{"line_number":925,"context_line":"            msg \u003d (_(\"Image %(id)s could not be deleted \""},{"line_number":926,"context_line":"                     \"because it is in use: %(exc)s\") %"},{"line_number":927,"context_line":"                   {\"id\": image_id,"},{"line_number":928,"context_line":"                    \"exc\": e})"},{"line_number":929,"context_line":"            LOG.warning(msg)"},{"line_number":930,"context_line":"            raise webob.exc.HTTPConflict(explanation\u003dmsg)"},{"line_number":931,"context_line":"        except glance_store.exceptions.HasSnapshot as e:"},{"line_number":932,"context_line":"            raise webob.exc.HTTPConflict(explanation\u003de.msg)"},{"line_number":933,"context_line":"        except exception.InvalidImageStatusTransition as e:"}],"source_content_type":"text/x-python","patch_set":1,"id":"fdfec9fb_0da7ea88","line":930,"range":{"start_line":924,"start_character":8,"end_line":930,"end_character":57},"updated":"2024-06-27 05:57:03.000000000","message":"I think this should be handled in cinder store and from there it should raise InUseByStore which is caught at line 917.\n\nThis will cause regression to stores other than cinder who were raising BackendException for any other reason.","commit_id":"b55924ee001c2296544051cdc24f057adfa1f579"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"23fca743d05979f748087151323b7eea267ab133","unresolved":true,"context_lines":[{"line_number":921,"context_line":"                    \"exc\": e.msg})"},{"line_number":922,"context_line":"            LOG.warning(msg)"},{"line_number":923,"context_line":"            raise webob.exc.HTTPConflict(explanation\u003dmsg)"},{"line_number":924,"context_line":"        except glance_store.exceptions.BackendException as e:"},{"line_number":925,"context_line":"            msg \u003d (_(\"Image %(id)s could not be deleted \""},{"line_number":926,"context_line":"                     \"because it is in use: %(exc)s\") %"},{"line_number":927,"context_line":"                   {\"id\": image_id,"},{"line_number":928,"context_line":"                    \"exc\": e})"},{"line_number":929,"context_line":"            LOG.warning(msg)"},{"line_number":930,"context_line":"            raise webob.exc.HTTPConflict(explanation\u003dmsg)"},{"line_number":931,"context_line":"        except glance_store.exceptions.HasSnapshot as e:"},{"line_number":932,"context_line":"            raise webob.exc.HTTPConflict(explanation\u003de.msg)"},{"line_number":933,"context_line":"        except exception.InvalidImageStatusTransition as e:"}],"source_content_type":"text/x-python","patch_set":1,"id":"8f497177_ab7e1ef3","line":930,"range":{"start_line":924,"start_character":8,"end_line":930,"end_character":57},"in_reply_to":"fdfec9fb_0da7ea88","updated":"2024-06-27 13:41:03.000000000","message":"ohh yeah right, will abandon this one.","commit_id":"b55924ee001c2296544051cdc24f057adfa1f579"}]}