)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":38286,"name":"Siddharth Kumar","display_name":"Siddharth","email":"siddharth.kumar1@dell.com","username":"siddhvrth"},"change_message_id":"3c917c47324bc4c25f958e4ba1f1441a89957f91","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e83eb5bb_6a4a6024","updated":"2026-02-13 06:21:00.000000000","message":"Hi team, can we have a review for this patch.","commit_id":"148f0f6c0240d73d9482aa92ae1e1d6be49c4cb4"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"700f928096ace6f21b5bea6ea62b83c8232294b2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"6b20fff5_afe12c9a","updated":"2026-02-12 08:33:05.000000000","message":"run-DellEMC PowerStore CI","commit_id":"148f0f6c0240d73d9482aa92ae1e1d6be49c4cb4"},{"author":{"_account_id":38286,"name":"Siddharth Kumar","display_name":"Siddharth","email":"siddharth.kumar1@dell.com","username":"siddhvrth"},"change_message_id":"d869e9936d0e21113dc5f07fde2254db4773c9bc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"970417d6_5ae6dadf","updated":"2026-01-28 08:39:38.000000000","message":"run-DellEMC PowerStore CI","commit_id":"148f0f6c0240d73d9482aa92ae1e1d6be49c4cb4"},{"author":{"_account_id":38286,"name":"Siddharth Kumar","display_name":"Siddharth","email":"siddharth.kumar1@dell.com","username":"siddhvrth"},"change_message_id":"2a0a23da3dd2dc4df285c1c3412e46dad9dd19d6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9a010d3c_ec03c10e","updated":"2026-02-12 06:10:53.000000000","message":"run-DellEMC PowerStore CI","commit_id":"148f0f6c0240d73d9482aa92ae1e1d6be49c4cb4"},{"author":{"_account_id":38286,"name":"Siddharth Kumar","display_name":"Siddharth","email":"siddharth.kumar1@dell.com","username":"siddhvrth"},"change_message_id":"6d190e9a9c20f7033ef77598ad0a33b6491875e1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"cf3b795c_dcef1331","updated":"2026-01-29 08:26:55.000000000","message":"run-DellEMC PowerStore CI","commit_id":"148f0f6c0240d73d9482aa92ae1e1d6be49c4cb4"},{"author":{"_account_id":38286,"name":"Siddharth Kumar","display_name":"Siddharth","email":"siddharth.kumar1@dell.com","username":"siddhvrth"},"change_message_id":"2bc9b4ebd61c8ef1ffc56d57880ac56f9ffcaa54","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e00408b1_220ed909","updated":"2026-02-05 09:25:25.000000000","message":"run-DellEMC PowerStore CI","commit_id":"148f0f6c0240d73d9482aa92ae1e1d6be49c4cb4"},{"author":{"_account_id":38286,"name":"Siddharth Kumar","display_name":"Siddharth","email":"siddharth.kumar1@dell.com","username":"siddhvrth"},"change_message_id":"c9af313a2bc4707a454a29c63efe5048e86ee97d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"bf955eb0_61815031","updated":"2026-03-02 05:59:58.000000000","message":"recheck","commit_id":"6324a5f1f43473318f9126981befc16fe8bab3fb"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"3f87ccfdd629895e05d59f720b9b9097da9fbf93","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"8f372e1b_5643d5ee","updated":"2026-05-07 12:37:47.000000000","message":"Please take a look at the comments.","commit_id":"da3716f3583a406a81e3a7ff1aae20e4fe2bdcc2"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"c728567357a2204bf3a7c4461e2617230437230d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"df22863d_647f74b2","updated":"2026-06-08 18:11:03.000000000","message":"LGTM","commit_id":"225aa8ff99c22de36bc5391887850fa264a0c107"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"650a0b32dfa52e2838df72b8f51e40b61555f1e8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"c1728f1d_af00ba9b","updated":"2026-05-26 15:37:59.000000000","message":"LGTM now.","commit_id":"225aa8ff99c22de36bc5391887850fa264a0c107"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"77c1f4b06382d17508cc44e924df399ce780c283","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"3b07cefb_15a74e3d","updated":"2026-06-16 13:37:43.000000000","message":"good test coverage, suggestion to extend use of PS_CLIENT\nelse lgtm","commit_id":"225aa8ff99c22de36bc5391887850fa264a0c107"},{"author":{"_account_id":38286,"name":"Siddharth Kumar","display_name":"Siddharth","email":"siddharth.kumar1@dell.com","username":"siddhvrth"},"change_message_id":"8e93273c31108a9ca67b404b25292fbf32905471","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"6ec1180a_80dd505e","updated":"2026-05-25 06:11:46.000000000","message":"recheck","commit_id":"225aa8ff99c22de36bc5391887850fa264a0c107"},{"author":{"_account_id":38286,"name":"Siddharth Kumar","display_name":"Siddharth","email":"siddharth.kumar1@dell.com","username":"siddhvrth"},"change_message_id":"6f7b34bcec4b74efec30a241646fe0af489574a1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"2cbf1cfe_05c30e2f","updated":"2026-06-08 15:57:25.000000000","message":"run-DellEMC PowerStore CI","commit_id":"225aa8ff99c22de36bc5391887850fa264a0c107"},{"author":{"_account_id":37792,"name":"Dell Openstack CI","display_name":"Dell Openstack CI","email":"dellopenstackci@dell.com","username":"dellopenstackci"},"change_message_id":"10042e8f3246cfa4077c20f7418e3dc92b4e7609","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"b3fced8d_f139bcba","updated":"2026-06-18 09:32:11.000000000","message":"run-DellEMC PowerStore CI","commit_id":"225aa8ff99c22de36bc5391887850fa264a0c107"},{"author":{"_account_id":37792,"name":"Dell Openstack CI","display_name":"Dell Openstack CI","email":"dellopenstackci@dell.com","username":"dellopenstackci"},"change_message_id":"3c014f147964183549b84f53cc3b082b4527e502","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"c46e84b8_80d611b7","updated":"2026-06-18 07:09:18.000000000","message":"run-DellEMC PowerStore CI","commit_id":"225aa8ff99c22de36bc5391887850fa264a0c107"},{"author":{"_account_id":37792,"name":"Dell Openstack CI","display_name":"Dell Openstack CI","email":"dellopenstackci@dell.com","username":"dellopenstackci"},"change_message_id":"7f140b38fdabf3b9a6c288ae8d8f73ebb802ca5a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"cc685992_edb3ea7c","updated":"2026-06-15 17:07:54.000000000","message":"run-DellEMC PowerStore CI","commit_id":"225aa8ff99c22de36bc5391887850fa264a0c107"},{"author":{"_account_id":37792,"name":"Dell Openstack CI","display_name":"Dell Openstack CI","email":"dellopenstackci@dell.com","username":"dellopenstackci"},"change_message_id":"ac07f69e8e1f588c668b3e0e0001563fe08a1647","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"e700570e_97bfe7a3","updated":"2026-06-10 05:54:31.000000000","message":"run-DellEMC PowerStore CI","commit_id":"225aa8ff99c22de36bc5391887850fa264a0c107"},{"author":{"_account_id":38286,"name":"Siddharth Kumar","display_name":"Siddharth","email":"siddharth.kumar1@dell.com","username":"siddhvrth"},"change_message_id":"99ddf131391804d8bfc076a2b9005765b220375a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"678d4f9f_2b4ec5dc","updated":"2026-06-19 06:32:08.000000000","message":"recheck","commit_id":"60ea3c8df486a7f443349b9c1ce588bd602783a9"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"df35f26160663bd0791ca1fa016d4508385d286a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"0b38a8c1_de2b0529","updated":"2026-06-23 15:10:15.000000000","message":"thanks for addressing last comments","commit_id":"60ea3c8df486a7f443349b9c1ce588bd602783a9"}],"cinder/tests/unit/volume/drivers/dell_emc/powerstore/test_client.py":[{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"77c1f4b06382d17508cc44e924df399ce780c283","unresolved":true,"context_lines":[{"line_number":112,"context_line":"PS_CLIENT \u003d ("},{"line_number":113,"context_line":"    \"cinder.volume.drivers.dell_emc.powerstore.client.\""},{"line_number":114,"context_line":"    \"PowerStoreClient\""},{"line_number":115,"context_line":")"},{"line_number":116,"context_line":""},{"line_number":117,"context_line":""},{"line_number":118,"context_line":"@ddt.ddt"}],"source_content_type":"text/x-python","patch_set":6,"id":"ca77ca91_7abbd3f8","line":115,"updated":"2026-06-16 13:37:43.000000000","message":"+1, can you also use this for existing rest of the tests.","commit_id":"225aa8ff99c22de36bc5391887850fa264a0c107"},{"author":{"_account_id":38286,"name":"Siddharth Kumar","display_name":"Siddharth","email":"siddharth.kumar1@dell.com","username":"siddhvrth"},"change_message_id":"10f6d756714abfc2c1bcfe4c7eb3c58a2a683013","unresolved":false,"context_lines":[{"line_number":112,"context_line":"PS_CLIENT \u003d ("},{"line_number":113,"context_line":"    \"cinder.volume.drivers.dell_emc.powerstore.client.\""},{"line_number":114,"context_line":"    \"PowerStoreClient\""},{"line_number":115,"context_line":")"},{"line_number":116,"context_line":""},{"line_number":117,"context_line":""},{"line_number":118,"context_line":"@ddt.ddt"}],"source_content_type":"text/x-python","patch_set":6,"id":"866e1d99_0783e359","line":115,"in_reply_to":"ca77ca91_7abbd3f8","updated":"2026-06-18 12:53:10.000000000","message":"Done","commit_id":"225aa8ff99c22de36bc5391887850fa264a0c107"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"77c1f4b06382d17508cc44e924df399ce780c283","unresolved":true,"context_lines":[{"line_number":399,"context_line":"                          \u0027host1\u0027,"},{"line_number":400,"context_line":"                          \u0027port1\u0027)"},{"line_number":401,"context_line":""},{"line_number":402,"context_line":"    @mock.patch(\u0027cinder.volume.drivers.dell_emc.powerstore.\u0027"},{"line_number":403,"context_line":"                \u0027client.PowerStoreClient._send_post_request\u0027)"},{"line_number":404,"context_line":"    def test_configure_metro_with_remote_appliance_name("},{"line_number":405,"context_line":"            self, mock_send_post_request):"}],"source_content_type":"text/x-python","patch_set":6,"id":"d0de92a5_20058641","line":402,"updated":"2026-06-16 13:37:43.000000000","message":"^^ PS_CLIENT","commit_id":"225aa8ff99c22de36bc5391887850fa264a0c107"},{"author":{"_account_id":38286,"name":"Siddharth Kumar","display_name":"Siddharth","email":"siddharth.kumar1@dell.com","username":"siddhvrth"},"change_message_id":"10f6d756714abfc2c1bcfe4c7eb3c58a2a683013","unresolved":false,"context_lines":[{"line_number":399,"context_line":"                          \u0027host1\u0027,"},{"line_number":400,"context_line":"                          \u0027port1\u0027)"},{"line_number":401,"context_line":""},{"line_number":402,"context_line":"    @mock.patch(\u0027cinder.volume.drivers.dell_emc.powerstore.\u0027"},{"line_number":403,"context_line":"                \u0027client.PowerStoreClient._send_post_request\u0027)"},{"line_number":404,"context_line":"    def test_configure_metro_with_remote_appliance_name("},{"line_number":405,"context_line":"            self, mock_send_post_request):"}],"source_content_type":"text/x-python","patch_set":6,"id":"d84d2860_fe7843a2","line":402,"in_reply_to":"d0de92a5_20058641","updated":"2026-06-18 12:53:10.000000000","message":"Done","commit_id":"225aa8ff99c22de36bc5391887850fa264a0c107"}],"cinder/volume/drivers/dell_emc/powerstore/adapter.py":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"c29d850af1ffdab04746580ea020a2f60f9bf9cb","unresolved":true,"context_lines":[{"line_number":1240,"context_line":"        elif \"source-name\" in existing_ref:"},{"line_number":1241,"context_line":"            details \u003d self.client.get_volume_details_by_name("},{"line_number":1242,"context_line":"                existing_ref[\"source-name\"], fields\u003dselect_fields)"},{"line_number":1243,"context_line":"        else:"},{"line_number":1244,"context_line":"            raise exception.InvalidInput("},{"line_number":1245,"context_line":"                reason\u003d(\"existing_ref must contain \u0027source-id\u0027 or \""},{"line_number":1246,"context_line":"                        \"\u0027source-name\u0027.\"))"}],"source_content_type":"text/x-python","patch_set":6,"id":"d09dbca5_6361d72a","line":1243,"updated":"2026-06-16 12:39:48.000000000","message":"Should an error be thrown if both source-id and source-name are provided? It looks like that case could result in unexpected behavior.","commit_id":"225aa8ff99c22de36bc5391887850fa264a0c107"},{"author":{"_account_id":38286,"name":"Siddharth Kumar","display_name":"Siddharth","email":"siddharth.kumar1@dell.com","username":"siddhvrth"},"change_message_id":"10f6d756714abfc2c1bcfe4c7eb3c58a2a683013","unresolved":false,"context_lines":[{"line_number":1240,"context_line":"        elif \"source-name\" in existing_ref:"},{"line_number":1241,"context_line":"            details \u003d self.client.get_volume_details_by_name("},{"line_number":1242,"context_line":"                existing_ref[\"source-name\"], fields\u003dselect_fields)"},{"line_number":1243,"context_line":"        else:"},{"line_number":1244,"context_line":"            raise exception.InvalidInput("},{"line_number":1245,"context_line":"                reason\u003d(\"existing_ref must contain \u0027source-id\u0027 or \""},{"line_number":1246,"context_line":"                        \"\u0027source-name\u0027.\"))"}],"source_content_type":"text/x-python","patch_set":6,"id":"11439281_3a0c40dc","line":1243,"in_reply_to":"d09dbca5_6361d72a","updated":"2026-06-18 12:53:10.000000000","message":"Done","commit_id":"225aa8ff99c22de36bc5391887850fa264a0c107"},{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"6694f708ce6dad94a90595c08773dc8f87baec89","unresolved":true,"context_lines":[{"line_number":1237,"context_line":"        has_source_id \u003d \"source-id\" in existing_ref"},{"line_number":1238,"context_line":"        has_source_name \u003d \"source-name\" in existing_ref"},{"line_number":1239,"context_line":"        if has_source_id and has_source_name:"},{"line_number":1240,"context_line":"            raise exception.InvalidInput("},{"line_number":1241,"context_line":"                reason\u003d(\"existing_ref must contain only one of \""},{"line_number":1242,"context_line":"                        \"\u0027source-id\u0027 or \u0027source-name\u0027, not both.\"))"},{"line_number":1243,"context_line":"        elif has_source_id:"}],"source_content_type":"text/x-python","patch_set":7,"id":"4b805468_842bc5b8","line":1240,"updated":"2026-06-30 05:08:37.000000000","message":"The invalid existing_ref cases should raise ManageExistingInvalidReference instead of InvalidInput.https://docs.openstack.org/cinder/latest/contributor/api/cinder.interface.volume_manageable_driver.html and https://docs.openstack.org/cinder/latest/contributor/api/cinder.exception.html#cinder.exception.ManageExistingInvalidReference","commit_id":"60ea3c8df486a7f443349b9c1ce588bd602783a9"},{"author":{"_account_id":38286,"name":"Siddharth Kumar","display_name":"Siddharth","email":"siddharth.kumar1@dell.com","username":"siddhvrth"},"change_message_id":"44171b681d1c5ee59da15c99e120c693a02821ff","unresolved":false,"context_lines":[{"line_number":1237,"context_line":"        has_source_id \u003d \"source-id\" in existing_ref"},{"line_number":1238,"context_line":"        has_source_name \u003d \"source-name\" in existing_ref"},{"line_number":1239,"context_line":"        if has_source_id and has_source_name:"},{"line_number":1240,"context_line":"            raise exception.InvalidInput("},{"line_number":1241,"context_line":"                reason\u003d(\"existing_ref must contain only one of \""},{"line_number":1242,"context_line":"                        \"\u0027source-id\u0027 or \u0027source-name\u0027, not both.\"))"},{"line_number":1243,"context_line":"        elif has_source_id:"}],"source_content_type":"text/x-python","patch_set":7,"id":"8b964479_ac49723e","line":1240,"in_reply_to":"4b805468_842bc5b8","updated":"2026-07-13 09:18:45.000000000","message":"Done","commit_id":"60ea3c8df486a7f443349b9c1ce588bd602783a9"}],"cinder/volume/drivers/dell_emc/powerstore/client.py":[{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"3f87ccfdd629895e05d59f720b9b9097da9fbf93","unresolved":true,"context_lines":[{"line_number":1043,"context_line":"            LOG.error(\"Failed to query PowerStore volumes by name.\")"},{"line_number":1044,"context_line":"            raise exception.VolumeBackendAPIException("},{"line_number":1045,"context_line":"                data\u003d\"Failed to query PowerStore volumes by name.\")"},{"line_number":1046,"context_line":"        if not response:"},{"line_number":1047,"context_line":"            raise exception.VolumeBackendAPIException("},{"line_number":1048,"context_line":"                data\u003d(\"PowerStore volume %s is not found.\" % name))"},{"line_number":1049,"context_line":"        return response[0]"}],"source_content_type":"text/x-python","patch_set":5,"id":"68da2776_ecd66cac","line":1046,"updated":"2026-05-07 12:37:47.000000000","message":"Should we also check if the response has one response before returning response[0]? Also, this should have a corresponding UT addressing response length?","commit_id":"da3716f3583a406a81e3a7ff1aae20e4fe2bdcc2"},{"author":{"_account_id":38286,"name":"Siddharth Kumar","display_name":"Siddharth","email":"siddharth.kumar1@dell.com","username":"siddhvrth"},"change_message_id":"7236c2d7a27c26adcf1da74847d334f1f8d467a3","unresolved":false,"context_lines":[{"line_number":1043,"context_line":"            LOG.error(\"Failed to query PowerStore volumes by name.\")"},{"line_number":1044,"context_line":"            raise exception.VolumeBackendAPIException("},{"line_number":1045,"context_line":"                data\u003d\"Failed to query PowerStore volumes by name.\")"},{"line_number":1046,"context_line":"        if not response:"},{"line_number":1047,"context_line":"            raise exception.VolumeBackendAPIException("},{"line_number":1048,"context_line":"                data\u003d(\"PowerStore volume %s is not found.\" % name))"},{"line_number":1049,"context_line":"        return response[0]"}],"source_content_type":"text/x-python","patch_set":5,"id":"3f508d19_c74efe99","line":1046,"in_reply_to":"68da2776_ecd66cac","updated":"2026-05-22 10:19:14.000000000","message":"Done","commit_id":"da3716f3583a406a81e3a7ff1aae20e4fe2bdcc2"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"3f87ccfdd629895e05d59f720b9b9097da9fbf93","unresolved":true,"context_lines":[{"line_number":1062,"context_line":"                r, response \u003d self._send_get_request(path, params\u003dparams)"},{"line_number":1063,"context_line":"                if r.status_code in self.ok_codes and response:"},{"line_number":1064,"context_line":"                    return True"},{"line_number":1065,"context_line":"            except Exception:"},{"line_number":1066,"context_line":"                pass"},{"line_number":1067,"context_line":"        return False"}],"source_content_type":"text/x-python","patch_set":5,"id":"dbc64ac4_5d0be6b6","line":1065,"updated":"2026-05-07 12:37:47.000000000","message":"Shouldn\u0027t the failure be logged here?","commit_id":"da3716f3583a406a81e3a7ff1aae20e4fe2bdcc2"},{"author":{"_account_id":38286,"name":"Siddharth Kumar","display_name":"Siddharth","email":"siddharth.kumar1@dell.com","username":"siddhvrth"},"change_message_id":"7236c2d7a27c26adcf1da74847d334f1f8d467a3","unresolved":false,"context_lines":[{"line_number":1062,"context_line":"                r, response \u003d self._send_get_request(path, params\u003dparams)"},{"line_number":1063,"context_line":"                if r.status_code in self.ok_codes and response:"},{"line_number":1064,"context_line":"                    return True"},{"line_number":1065,"context_line":"            except Exception:"},{"line_number":1066,"context_line":"                pass"},{"line_number":1067,"context_line":"        return False"}],"source_content_type":"text/x-python","patch_set":5,"id":"a564819f_a57cb789","line":1065,"in_reply_to":"dbc64ac4_5d0be6b6","updated":"2026-05-22 10:19:14.000000000","message":"Done","commit_id":"da3716f3583a406a81e3a7ff1aae20e4fe2bdcc2"},{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"6694f708ce6dad94a90595c08773dc8f87baec89","unresolved":true,"context_lines":[{"line_number":1068,"context_line":"                if r.status_code in self.ok_codes and response:"},{"line_number":1069,"context_line":"                    return True"},{"line_number":1070,"context_line":"            except Exception:"},{"line_number":1071,"context_line":"                LOG.warning(\"Failed to query %(path)s for volume \""},{"line_number":1072,"context_line":"                            \"%(volume_id)s.\","},{"line_number":1073,"context_line":"                            {\"path\": path, \"volume_id\": volume_id})"},{"line_number":1074,"context_line":"        return False"}],"source_content_type":"text/x-python","patch_set":7,"id":"e93dc989_76303615","line":1071,"updated":"2026-06-30 05:08:37.000000000","message":"failed to query mapping and HTTP exception are thread in same way. Current logic combines both of them and return false both the cases... Is this fine?","commit_id":"60ea3c8df486a7f443349b9c1ce588bd602783a9"},{"author":{"_account_id":38286,"name":"Siddharth Kumar","display_name":"Siddharth","email":"siddharth.kumar1@dell.com","username":"siddhvrth"},"change_message_id":"44171b681d1c5ee59da15c99e120c693a02821ff","unresolved":false,"context_lines":[{"line_number":1068,"context_line":"                if r.status_code in self.ok_codes and response:"},{"line_number":1069,"context_line":"                    return True"},{"line_number":1070,"context_line":"            except Exception:"},{"line_number":1071,"context_line":"                LOG.warning(\"Failed to query %(path)s for volume \""},{"line_number":1072,"context_line":"                            \"%(volume_id)s.\","},{"line_number":1073,"context_line":"                            {\"path\": path, \"volume_id\": volume_id})"},{"line_number":1074,"context_line":"        return False"}],"source_content_type":"text/x-python","patch_set":7,"id":"3739c390_e71a9f13","line":1071,"in_reply_to":"e93dc989_76303615","updated":"2026-07-13 09:18:45.000000000","message":"Good catch. volume_is_mapped() now raises \n VolumeBackendAPIException on any query failure (exception or non-OK HTTP status), and only returns False when the API successfully confirms no mappings exist.","commit_id":"60ea3c8df486a7f443349b9c1ce588bd602783a9"}],"cinder/volume/drivers/dell_emc/powerstore/driver.py":[{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"3f87ccfdd629895e05d59f720b9b9097da9fbf93","unresolved":true,"context_lines":[{"line_number":58,"context_line":"        1.2.5 - Added cinder manage/unmanage volume support"},{"line_number":59,"context_line":"    \"\"\""},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"    VERSION \u003d \"1.2.4\""},{"line_number":62,"context_line":"    VENDOR \u003d \"Dell EMC\""},{"line_number":63,"context_line":"    SUPPORTS_ACTIVE_ACTIVE \u003d True"},{"line_number":64,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"fafe60e0_9eb09768","line":61,"updated":"2026-05-07 12:37:47.000000000","message":"Shouldn\u0027t the version change to 1.2.5?","commit_id":"da3716f3583a406a81e3a7ff1aae20e4fe2bdcc2"},{"author":{"_account_id":38286,"name":"Siddharth Kumar","display_name":"Siddharth","email":"siddharth.kumar1@dell.com","username":"siddhvrth"},"change_message_id":"7236c2d7a27c26adcf1da74847d334f1f8d467a3","unresolved":false,"context_lines":[{"line_number":58,"context_line":"        1.2.5 - Added cinder manage/unmanage volume support"},{"line_number":59,"context_line":"    \"\"\""},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"    VERSION \u003d \"1.2.4\""},{"line_number":62,"context_line":"    VENDOR \u003d \"Dell EMC\""},{"line_number":63,"context_line":"    SUPPORTS_ACTIVE_ACTIVE \u003d True"},{"line_number":64,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"42775afc_d041bce8","line":61,"in_reply_to":"fafe60e0_9eb09768","updated":"2026-05-22 10:19:14.000000000","message":"Done","commit_id":"da3716f3583a406a81e3a7ff1aae20e4fe2bdcc2"}],"doc/source/configuration/block-storage/drivers/dell-emc-powerstore-driver.rst":[{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"cd05fc613cf2e2d3cb124acd76dcbbfe16c95d14","unresolved":true,"context_lines":[{"line_number":410,"context_line":"* Volume must **not** be in an active **migration** or"},{"line_number":411,"context_line":"  **metro replication** session."},{"line_number":412,"context_line":"* Volume must **not** be **mapped/attached** to any host or host group."},{"line_number":413,"context_line":"* Targets must be accessible and **allowed** for this backend:"},{"line_number":414,"context_line":"  - **FC**: at least one allowed **WWN** is available."},{"line_number":415,"context_line":"  - **iSCSI / NVMe‑TCP**: at least one allowed **portal/IP** is available."},{"line_number":416,"context_line":"* The ``existing_ref`` **must** contain either **``source-id``** or"}],"source_content_type":"text/x-rst","patch_set":4,"id":"4e4ba147_04bf11b0","line":413,"updated":"2026-04-09 12:08:48.000000000","message":"Why is connectivity required? \nWhen we do a manage operation, we simply add volume metadata into cinder DB. It has nothing to do with FC or iSCSI.","commit_id":"4015ff29723f24363af4d6aa6d0d0f7bf5c4ec4f"},{"author":{"_account_id":38286,"name":"Siddharth Kumar","display_name":"Siddharth","email":"siddharth.kumar1@dell.com","username":"siddhvrth"},"change_message_id":"1c0d7f72e799f36260e523b01515fba85b1fe7a3","unresolved":false,"context_lines":[{"line_number":410,"context_line":"* Volume must **not** be in an active **migration** or"},{"line_number":411,"context_line":"  **metro replication** session."},{"line_number":412,"context_line":"* Volume must **not** be **mapped/attached** to any host or host group."},{"line_number":413,"context_line":"* Targets must be accessible and **allowed** for this backend:"},{"line_number":414,"context_line":"  - **FC**: at least one allowed **WWN** is available."},{"line_number":415,"context_line":"  - **iSCSI / NVMe‑TCP**: at least one allowed **portal/IP** is available."},{"line_number":416,"context_line":"* The ``existing_ref`` **must** contain either **``source-id``** or"}],"source_content_type":"text/x-rst","patch_set":4,"id":"174d400f_606caec7","line":413,"in_reply_to":"4e4ba147_04bf11b0","updated":"2026-04-16 05:43:00.000000000","message":"Done","commit_id":"4015ff29723f24363af4d6aa6d0d0f7bf5c4ec4f"}]}