)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"5ef7b402b578da2d3a5e54849033cc4cddb376db","unresolved":false,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Error after patch applied:"},{"line_number":22,"context_line":"Volume status is reserved. Volume status must be ..."},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"Change-Id: I18c9d1ed822a75bad7daf8340b329d93a362f0e7"},{"line_number":25,"context_line":"Signed-off-by: Rajat Dhasmana \u003crajatdhasmana@gmail.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"eb4136f9_f4088edf","line":23,"updated":"2026-06-08 06:53:50.000000000","message":"+1","commit_id":"eb03f7789fbcd2cab0acd4307ff31f8c27a9d66c"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"5ef7b402b578da2d3a5e54849033cc4cddb376db","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"8a4225be_b957d6ee","updated":"2026-06-08 06:53:50.000000000","message":"a nit on wording, we can improve commit msg\notherwise we lgtm","commit_id":"eb03f7789fbcd2cab0acd4307ff31f8c27a9d66c"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"27641a80bdc9cc12b69fa7b82f560014177aea6e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ec8c3ca8_25041e9a","updated":"2026-06-07 19:08:03.000000000","message":"recheck unrelated rbd issue in py313 unit tests\n\n           ~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^\n  File \"/home/zuul/src/opendev.org/openstack/cinder/cinder/backup/drivers/ceph.py\", line 571, in _try_delete_base_image\n    with eventlet.tpool.Proxy(rbd_driver.RADOSClient(self,\n                              ~~~~~~~~~~~~~~~~~~~~~~^^^^^^\n                              backup.container)) as client:\n                              ^^^^^^^^^^^^^^^^^\n  File \"/home/zuul/src/opendev.org/openstack/cinder/cinder/volume/drivers/rbd.py\", line 268, in __init__\n    self.cluster, self.ioctx \u003d driver._connect_to_rados(pool)\n                               ~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^\n  File \"/home/zuul/src/opendev.org/openstack/cinder/cinder/backup/drivers/ceph.py\", line 348, in _connect_to_rados\n    except self.rados.Error:\n    ...\u003c2 lines\u003e...\n        raise\nTypeError: catching classes that do not inherit from BaseException is not allowed","commit_id":"eb03f7789fbcd2cab0acd4307ff31f8c27a9d66c"}],"cinder/volume/api.py":[{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"5ef7b402b578da2d3a5e54849033cc4cddb376db","unresolved":true,"context_lines":[{"line_number":511,"context_line":"                    \u0027attached, belong to a group, have snapshots, awaiting \u0027"},{"line_number":512,"context_line":"                    \u0027a transfer, or be disassociated from snapshots after \u0027"},{"line_number":513,"context_line":"                    \u0027volume transfer.\u0027) % (volume.status, status)"},{"line_number":514,"context_line":"            LOG.info(msg)"},{"line_number":515,"context_line":"            raise exception.InvalidVolume(reason\u003dmsg)"},{"line_number":516,"context_line":""},{"line_number":517,"context_line":"        if cascade:"}],"source_content_type":"text/x-python","patch_set":1,"id":"96717f62_1cf5643a","line":514,"updated":"2026-06-08 06:53:50.000000000","message":"nit: can we improve wording.\nfor ex it will read somehitng like:\nVolume status is reserved. Volume status must available or error or .. and must not be migrating, attached, belong to group, have snapshot, awaiting a transfer, or be disassociated from snapshots after volume trasfer.\n\n---\ncan we do:\n\nVolume delete failed. current Volume status is \u0027reserved\u0027, required Volume status must be \u0027available or ...\u0027. And volume must not be [migrating, attached, part of some group**, have snapshots, awaiting transfer, or be disassociated snapshot after volume transfer.\n\n**I am not sure what is \u0027belong to group\u0027 mean here! can we say belong to this particular vol group or along those lines?","commit_id":"eb03f7789fbcd2cab0acd4307ff31f8c27a9d66c"}]}
