)]}'
{"/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":38081,"name":"Anthony Galica","display_name":"agalica","email":"anthony.galica@hitachivantara.com","username":"agalica","status":"Hitachi Vantara"},"change_message_id":"ae621093717924f16231830b83e6d73a60c6b08d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"20985cd4_3a5d9bed","updated":"2026-10-02 16:07:17.000000000","message":"LGTM","commit_id":"eb03f7789fbcd2cab0acd4307ff31f8c27a9d66c"},{"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"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"81ce34f8b7fa4710c8406c3b98af5fe22571a360","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"01142f63_9e03ddaf","updated":"2026-10-07 06:31:37.000000000","message":"This is the new error after the patch is applied\n\nDelete for volume d7c4ccb8-f0d4-4c75-b2d9-688f420a6eeb failed: Invalid volume: Volume deletion failed because one or more conditions are not met. Current volume status is \u0027reserved\u0027. Required conditions: volume status must be available or error or error_restoring or error_extending or error_managing, and volume must not be migrating, attached, or be part of a volume group or consistency group; it must not have snapshots, await a transfer, or be disassociated from snapshots after volume transfer. (HTTP 400) (Request-ID: req-feb8cbda-1683-49ea-bd4a-c2c92960b5b9)","commit_id":"1cccfb5f8c8c4ba740f71c2c7f08babb7d004d96"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"7240c056136ba09a0fdccb6518446848e4dedbe6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"5115e9f7_8eb920ad","updated":"2026-10-08 05:14:03.000000000","message":"recheck CI hasn\u0027t reported since a day","commit_id":"1cccfb5f8c8c4ba740f71c2c7f08babb7d004d96"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"6edebeaf524496eaad5c06922b2af160f8539863","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"aa0dcd0b_11100f0d","updated":"2026-10-08 09:50:20.000000000","message":"recheck tempest-integrated-storage failed","commit_id":"1cccfb5f8c8c4ba740f71c2c7f08babb7d004d96"}],"cinder/volume/api.py":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"9dae42614f47cf6a1334821dc95d94e4d2ae5872","unresolved":true,"context_lines":[{"line_number":507,"context_line":"        if not result:"},{"line_number":508,"context_line":"            status \u003d utils.build_or_str(expected.get(\u0027status\u0027),"},{"line_number":509,"context_line":"                                        _(\u0027status must be %s and\u0027))"},{"line_number":510,"context_line":"            msg \u003d _(\u0027Volume status is %s. Volume %s must not be migrating, \u0027"},{"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)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1dce43b8_e3b0c6a2","line":510,"updated":"2026-10-05 14:26:30.000000000","message":"I\u0027m not sure if this wording change is necessarily better -- for cases like the volume being in a group, it will result in messages like \"Volume status is available. Volume \u003cid\u003e must not be ...\" etc. which seems confusing because the status isn\u0027t the reason for the failure.","commit_id":"eb03f7789fbcd2cab0acd4307ff31f8c27a9d66c"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"56ec388e2eeac327c51143dfd0544f0a3565d76c","unresolved":false,"context_lines":[{"line_number":507,"context_line":"        if not result:"},{"line_number":508,"context_line":"            status \u003d utils.build_or_str(expected.get(\u0027status\u0027),"},{"line_number":509,"context_line":"                                        _(\u0027status must be %s and\u0027))"},{"line_number":510,"context_line":"            msg \u003d _(\u0027Volume status is %s. Volume %s must not be migrating, \u0027"},{"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)"}],"source_content_type":"text/x-python","patch_set":1,"id":"18b1e06f_36305c02","line":510,"in_reply_to":"1dce43b8_e3b0c6a2","updated":"2026-10-07 06:28:13.000000000","message":"Done","commit_id":"eb03f7789fbcd2cab0acd4307ff31f8c27a9d66c"},{"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"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"56ec388e2eeac327c51143dfd0544f0a3565d76c","unresolved":false,"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":"e2386de8_73eeede2","line":514,"in_reply_to":"96717f62_1cf5643a","updated":"2026-10-07 06:28:13.000000000","message":"Done\nbelong to a group means part of a volume group, it can be easily seen when we show a volume there is a group_id field. we need to remove the volume from that group to be able to successfully delete it.","commit_id":"eb03f7789fbcd2cab0acd4307ff31f8c27a9d66c"}]}