)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":36800,"name":"Vrinda Dhakad","display_name":"vdhakad","email":"vrinda.dhakad@ibm.com","username":"vdhakad"},"change_message_id":"8d7448b75177a9f91a90531a502ec2934c948e1c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e4bda12c_edae6631","updated":"2025-03-05 06:58:14.000000000","message":"Hi Harsh and Nimesh,\n\nPlease review changes for PBHA driver init story.","commit_id":"d5a7c5f98dca770f17344d104fb1d5d984ca8548"},{"author":{"_account_id":37470,"name":"Dakshina Purohit","display_name":"Dakshina","email":"Dakshina.Purohit@ibm.com","username":"dakshina"},"change_message_id":"9487de8d6b64b64842dad655324f162a10b5c740","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1486bd72_7087abd0","updated":"2025-03-06 17:44:26.000000000","message":"LGTM","commit_id":"d5a7c5f98dca770f17344d104fb1d5d984ca8548"},{"author":{"_account_id":37470,"name":"Dakshina Purohit","display_name":"Dakshina","email":"Dakshina.Purohit@ibm.com","username":"dakshina"},"change_message_id":"210f180cf875d01094368e6aa69305471d1a73d6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"be3bcebb_98636ef1","updated":"2025-03-12 09:24:20.000000000","message":"LGTM","commit_id":"5b3a3d8001eb8cc5e4a992c9b3001df4be6739ef"},{"author":{"_account_id":37328,"name":"Nimesh Desai","display_name":"Nimesh Desai","email":"nimesh.desai@ibm.com","username":"nimeshdesai"},"change_message_id":"b740fb86ffe19cb420e199bbdb4ab387178378cb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":32,"id":"0fb45f57_19fd18bf","updated":"2025-11-10 08:15:17.000000000","message":"recheck","commit_id":"f10ea3c270058a7e2a9f1c4f516471723a95fb27"},{"author":{"_account_id":37328,"name":"Nimesh Desai","display_name":"Nimesh Desai","email":"nimesh.desai@ibm.com","username":"nimeshdesai"},"change_message_id":"f785f497eaf1fdf48549d0ebc30670f94274a1c1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":38,"id":"ce9cc3f6_ee5d9bd0","updated":"2025-11-15 07:31:48.000000000","message":"recheck cinder-tempest-plugin-lvm-lio-barbican","commit_id":"53a75c8770cf6e9879382c53b1ade4f3535dfb02"},{"author":{"_account_id":37328,"name":"Nimesh Desai","display_name":"Nimesh Desai","email":"nimesh.desai@ibm.com","username":"nimeshdesai"},"change_message_id":"5ef4d25ab0ed93cf0f74e8c723c2b2da879f27cd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":39,"id":"70e417a1_63baa6a7","updated":"2025-12-19 10:18:47.000000000","message":"LGTM","commit_id":"ce594e11f0cc9c6bb8f10dc4f8e903e3e5232253"},{"author":{"_account_id":36800,"name":"Vrinda Dhakad","display_name":"vdhakad","email":"vrinda.dhakad@ibm.com","username":"vdhakad"},"change_message_id":"4628f50393a0ca363f0dde806ceb694548b16163","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":39,"id":"cb5ff9c3_18bee751","updated":"2025-11-21 14:22:35.000000000","message":"recheck","commit_id":"ce594e11f0cc9c6bb8f10dc4f8e903e3e5232253"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"8de9598bf36f4a04b856e96e1bd25d31d21005cc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":39,"id":"88c36d4e_eca8679f","updated":"2025-11-21 14:23:46.000000000","message":"recheck cinder-plugin-ceph-tempest - cinder_tempest_plugin.scenario.test_volume_encrypted.TestEncryptedCinderVolumes.test_boot_cloned_encrypted_volume couldn\u0027t boot the instance in time, unrelated to this patch","commit_id":"ce594e11f0cc9c6bb8f10dc4f8e903e3e5232253"},{"author":{"_account_id":37328,"name":"Nimesh Desai","display_name":"Nimesh Desai","email":"nimesh.desai@ibm.com","username":"nimeshdesai"},"change_message_id":"0ee722a37d4fae2f860aaa24fea3f106683421c9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":41,"id":"07ef7e03_b5f3894b","updated":"2026-01-09 09:36:59.000000000","message":"recheck tempest-integrated-storage","commit_id":"f47e00ba9b5cb312cad716417954adab13e39db4"}],"cinder/tests/unit/volume/drivers/ibm/test_storwize_svc.py":[{"author":{"_account_id":36800,"name":"Vrinda Dhakad","display_name":"vdhakad","email":"vrinda.dhakad@ibm.com","username":"vdhakad"},"change_message_id":"406474a910fa73e86d88dea3b48214002ae4e396","unresolved":true,"context_lines":[{"line_number":3433,"context_line":"        return (\u0027\u0027, \u0027\u0027)"},{"line_number":3434,"context_line":""},{"line_number":3435,"context_line":"    def _cmd_lspartition(self, **kwargs):"},{"line_number":3436,"context_line":"        partition \u003d [{\u0027id\u0027: \u00270\u0027,"},{"line_number":3437,"context_line":"                      \u0027name\u0027: \u0027test\u0027,"},{"line_number":3438,"context_line":"                      \u0027preferred_management_location\u0027: \u00271\u0027,"},{"line_number":3439,"context_line":"                      \u0027preferred_management_system_id\u0027: \u00271234\u0027,"}],"source_content_type":"text/x-python","patch_set":5,"id":"87817c1e_e38546d4","line":3436,"range":{"start_line":3436,"start_character":0,"end_line":3436,"end_character":2},"updated":"2025-04-30 07:26:52.000000000","message":"Remove fields that are not used.","commit_id":"7427c1bb792d5b80441dba3dd916c00f03107ae2"},{"author":{"_account_id":36538,"name":"Harsh Ailani","display_name":"Harsh Ailani","email":"harsh.ailani@ibm.com","username":"harshailani"},"change_message_id":"bb00045fb0167151f2dfd89a4fe87a1412061deb","unresolved":false,"context_lines":[{"line_number":3433,"context_line":"        return (\u0027\u0027, \u0027\u0027)"},{"line_number":3434,"context_line":""},{"line_number":3435,"context_line":"    def _cmd_lspartition(self, **kwargs):"},{"line_number":3436,"context_line":"        partition \u003d [{\u0027id\u0027: \u00270\u0027,"},{"line_number":3437,"context_line":"                      \u0027name\u0027: \u0027test\u0027,"},{"line_number":3438,"context_line":"                      \u0027preferred_management_location\u0027: \u00271\u0027,"},{"line_number":3439,"context_line":"                      \u0027preferred_management_system_id\u0027: \u00271234\u0027,"}],"source_content_type":"text/x-python","patch_set":5,"id":"56fa7f3e_ace0fd3b","line":3436,"range":{"start_line":3436,"start_character":0,"end_line":3436,"end_character":2},"in_reply_to":"87817c1e_e38546d4","updated":"2025-05-08 13:07:35.000000000","message":"Removed.","commit_id":"7427c1bb792d5b80441dba3dd916c00f03107ae2"}],"cinder/volume/drivers/ibm/storwize_svc/storwize_svc_common.py":[{"author":{"_account_id":36538,"name":"Harsh Ailani","display_name":"Harsh Ailani","email":"harsh.ailani@ibm.com","username":"harshailani"},"change_message_id":"12db65d4135fe90df32698baa2b194e51c396e95","unresolved":true,"context_lines":[{"line_number":1966,"context_line":"        self.check_vdisk_opts(state, opts)"},{"line_number":1967,"context_line":"        return opts"},{"line_number":1968,"context_line":""},{"line_number":1969,"context_line":"    def check_data_reduction_pool_params(self, opts, p_name\u003dNone):"},{"line_number":1970,"context_line":"        \"\"\"Check the configured parameters if vol in data reduction pool.\"\"\""},{"line_number":1971,"context_line":"        if opts[\u0027warning\u0027] !\u003d 0:"},{"line_number":1972,"context_line":"            msg \u003d (_(\u0027You cannot specify -warning for thin-provisioned or \u0027"}],"source_content_type":"text/x-python","patch_set":5,"id":"d7ea21b9_2c9469e6","line":1969,"range":{"start_line":1969,"start_character":53,"end_line":1969,"end_character":59},"updated":"2025-05-06 07:20:42.000000000","message":"Variable \u0027p_name\u0027 can be renamed to \u0027partition_name\u0027 to keep it consistent throughout the whole code.","commit_id":"7427c1bb792d5b80441dba3dd916c00f03107ae2"},{"author":{"_account_id":36538,"name":"Harsh Ailani","display_name":"Harsh Ailani","email":"harsh.ailani@ibm.com","username":"harshailani"},"change_message_id":"bb00045fb0167151f2dfd89a4fe87a1412061deb","unresolved":false,"context_lines":[{"line_number":1966,"context_line":"        self.check_vdisk_opts(state, opts)"},{"line_number":1967,"context_line":"        return opts"},{"line_number":1968,"context_line":""},{"line_number":1969,"context_line":"    def check_data_reduction_pool_params(self, opts, p_name\u003dNone):"},{"line_number":1970,"context_line":"        \"\"\"Check the configured parameters if vol in data reduction pool.\"\"\""},{"line_number":1971,"context_line":"        if opts[\u0027warning\u0027] !\u003d 0:"},{"line_number":1972,"context_line":"            msg \u003d (_(\u0027You cannot specify -warning for thin-provisioned or \u0027"}],"source_content_type":"text/x-python","patch_set":5,"id":"063876c0_177fb7b0","line":1969,"range":{"start_line":1969,"start_character":53,"end_line":1969,"end_character":59},"in_reply_to":"d7ea21b9_2c9469e6","updated":"2025-05-08 13:07:35.000000000","message":"Corrected the variable name.","commit_id":"7427c1bb792d5b80441dba3dd916c00f03107ae2"},{"author":{"_account_id":36538,"name":"Harsh Ailani","display_name":"Harsh Ailani","email":"harsh.ailani@ibm.com","username":"harshailani"},"change_message_id":"12db65d4135fe90df32698baa2b194e51c396e95","unresolved":true,"context_lines":[{"line_number":6555,"context_line":"                        \"temporary_volume_group_enabled\"):"},{"line_number":6556,"context_line":"                    return model_update"},{"line_number":6557,"context_line":"                volumegroup_name \u003d self._get_volumegroup_name(group)"},{"line_number":6558,"context_line":"                LOG.info(\"partition FS5200: %s. \", self.partition_name)"},{"line_number":6559,"context_line":"                if self.partition_name:"},{"line_number":6560,"context_line":"                    self._helpers.create_volumegroup(volumegroup_name,"},{"line_number":6561,"context_line":"                                                     self.partition_name)"}],"source_content_type":"text/x-python","patch_set":5,"id":"3b95bf73_6539f036","line":6558,"updated":"2025-05-06 07:20:42.000000000","message":"This statement doesn\u0027t look correct.","commit_id":"7427c1bb792d5b80441dba3dd916c00f03107ae2"},{"author":{"_account_id":36538,"name":"Harsh Ailani","display_name":"Harsh Ailani","email":"harsh.ailani@ibm.com","username":"harshailani"},"change_message_id":"bb00045fb0167151f2dfd89a4fe87a1412061deb","unresolved":false,"context_lines":[{"line_number":6555,"context_line":"                        \"temporary_volume_group_enabled\"):"},{"line_number":6556,"context_line":"                    return model_update"},{"line_number":6557,"context_line":"                volumegroup_name \u003d self._get_volumegroup_name(group)"},{"line_number":6558,"context_line":"                LOG.info(\"partition FS5200: %s. \", self.partition_name)"},{"line_number":6559,"context_line":"                if self.partition_name:"},{"line_number":6560,"context_line":"                    self._helpers.create_volumegroup(volumegroup_name,"},{"line_number":6561,"context_line":"                                                     self.partition_name)"}],"source_content_type":"text/x-python","patch_set":5,"id":"5d5b9d0d_261152db","line":6558,"in_reply_to":"3b95bf73_6539f036","updated":"2025-05-08 13:07:35.000000000","message":"Corrected the log statement.","commit_id":"7427c1bb792d5b80441dba3dd916c00f03107ae2"},{"author":{"_account_id":38067,"name":"Sandip Rajbanshi","display_name":"Sandip27","email":"sandip.rajbanshi@ibm.com","username":"sandip-rajbanshi-27"},"change_message_id":"c6eb18c333cb83a194cac4257a43196e154447f8","unresolved":true,"context_lines":[{"line_number":990,"context_line":"        self.run_ssh_assert_no_output(ssh_cmd)"},{"line_number":991,"context_line":""},{"line_number":992,"context_line":"    def lspartition(self, partition_name):"},{"line_number":993,"context_line":"        ssh_cmd \u003d [\u0027svcinfo\u0027, \u0027lspartition\u0027, \u0027-gui\u0027, \u0027-delim\u0027, \u0027!\u0027,"},{"line_number":994,"context_line":"                   \u0027\"%s\"\u0027 % partition_name]"},{"line_number":995,"context_line":"        return self.run_ssh_info(ssh_cmd, with_header\u003dFalse)"},{"line_number":996,"context_line":""}],"source_content_type":"text/x-python","patch_set":22,"id":"3829d7f5_2fcc316b","line":993,"updated":"2025-09-26 05:54:52.000000000","message":"Hi @vrinda.dhakad@ibm.com\n\"-gui\" option should not be used as this option is for internal use and also it is not documented in information center, so recommend not to use.","commit_id":"fdf87a280a0d1a983e7c17e66ca6c7a3465e1095"},{"author":{"_account_id":36800,"name":"Vrinda Dhakad","display_name":"vdhakad","email":"vrinda.dhakad@ibm.com","username":"vdhakad"},"change_message_id":"0f86213f4c028b5486b361b445643e09cabae478","unresolved":false,"context_lines":[{"line_number":990,"context_line":"        self.run_ssh_assert_no_output(ssh_cmd)"},{"line_number":991,"context_line":""},{"line_number":992,"context_line":"    def lspartition(self, partition_name):"},{"line_number":993,"context_line":"        ssh_cmd \u003d [\u0027svcinfo\u0027, \u0027lspartition\u0027, \u0027-gui\u0027, \u0027-delim\u0027, \u0027!\u0027,"},{"line_number":994,"context_line":"                   \u0027\"%s\"\u0027 % partition_name]"},{"line_number":995,"context_line":"        return self.run_ssh_info(ssh_cmd, with_header\u003dFalse)"},{"line_number":996,"context_line":""}],"source_content_type":"text/x-python","patch_set":22,"id":"a6c47d22_aedccc0a","line":993,"in_reply_to":"3829d7f5_2fcc316b","updated":"2025-09-29 06:07:42.000000000","message":"As discussed, -gui is meant for internal use which includes GUI, plugins, drivers, etc. This option is necessary for extra fields that are being used in the code.","commit_id":"fdf87a280a0d1a983e7c17e66ca6c7a3465e1095"},{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"851400f3c2cb86f77456ce196867be3ff41b85d7","unresolved":true,"context_lines":[{"line_number":1659,"context_line":"                        \u0027for PBHA backends through cinder driver. Ensure/ \u0027"},{"line_number":1660,"context_line":"                        \u0027add following wwpns to host on both systems \u0027"},{"line_number":1661,"context_line":"                        \u0027and restart cinder. Retry host attach.\u0027"},{"line_number":1662,"context_line":"                        \u0027%(new_wwpns)s\u0027 % {\u0027new_wwpn\u0027: new_wwpn})"},{"line_number":1663,"context_line":"                raise exception.VolumeDriverException(message\u003dmsg)"},{"line_number":1664,"context_line":"            else:"},{"line_number":1665,"context_line":"                for port in new_wwpn:"}],"source_content_type":"text/x-python","patch_set":50,"id":"452dbec6_a10c823d","line":1662,"updated":"2026-04-08 05:59:23.000000000","message":"\u0027%(new_wwpn)s\u0027 % {\u0027new_wwpn\u0027: new_wwpn})","commit_id":"984605391cf0a647483e9c2466b6915fc5525646"},{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"851400f3c2cb86f77456ce196867be3ff41b85d7","unresolved":true,"context_lines":[{"line_number":1659,"context_line":"                        \u0027for PBHA backends through cinder driver. Ensure/ \u0027"},{"line_number":1660,"context_line":"                        \u0027add following wwpns to host on both systems \u0027"},{"line_number":1661,"context_line":"                        \u0027and restart cinder. Retry host attach.\u0027"},{"line_number":1662,"context_line":"                        \u0027%(new_wwpns)s\u0027 % {\u0027new_wwpn\u0027: new_wwpn})"},{"line_number":1663,"context_line":"                raise exception.VolumeDriverException(message\u003dmsg)"},{"line_number":1664,"context_line":"            else:"},{"line_number":1665,"context_line":"                for port in new_wwpn:"}],"source_content_type":"text/x-python","patch_set":50,"id":"a338e6c2_f85a8702","line":1662,"updated":"2026-04-08 05:59:23.000000000","message":"The format string uses %(new_wwpns)s (plural) but the dict key is \u0027new_wwpn\u0027 (singular). This will raise a KeyError at runtime, crashing the host attachment flow instead of producing a useful error message.","commit_id":"984605391cf0a647483e9c2466b6915fc5525646"}],"cinder/volume/drivers/ibm/storwize_svc/storwize_svc_fc.py":[{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"851400f3c2cb86f77456ce196867be3ff41b85d7","unresolved":true,"context_lines":[{"line_number":274,"context_line":"                        .get_conn_fc_wwpns(host_name)"},{"line_number":275,"context_line":"                    conn_wwpns.extend(aux_conn_wwpns)"},{"line_number":276,"context_line":"                except Exception as ex:"},{"line_number":277,"context_line":"                    msg \u003d (\"Unable to create PBHA host due to HA target is \""},{"line_number":278,"context_line":"                           \"not reachable. error\u003d%(ex)s\" % {\u0027error\u0027: ex})"},{"line_number":279,"context_line":"                    LOG.error(msg)"},{"line_number":280,"context_line":"                    raise exception.VolumeBackendAPIException(reason\u003dmsg)"}],"source_content_type":"text/x-python","patch_set":50,"id":"9d2c8027_4856591a","line":277,"updated":"2026-04-08 05:59:23.000000000","message":"The format string uses key %(ex)s but the dict provides key \u0027error\u0027. This will raise a KeyError at runtime. The same bug appears twice — once in the get_conn_fc_wwpns block and once in the get_npiv_wwpns block.","commit_id":"984605391cf0a647483e9c2466b6915fc5525646"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"c603bfeed1b3a38deaedca413da005e7fd2b01a9","unresolved":true,"context_lines":[{"line_number":274,"context_line":"                        .get_conn_fc_wwpns(host_name)"},{"line_number":275,"context_line":"                    conn_wwpns.extend(aux_conn_wwpns)"},{"line_number":276,"context_line":"                except Exception as ex:"},{"line_number":277,"context_line":"                    msg \u003d (\"Unable to create PBHA host due to HA target is \""},{"line_number":278,"context_line":"                           \"not reachable. error\u003d%(ex)s\" % {\u0027error\u0027: ex})"},{"line_number":279,"context_line":"                    LOG.error(msg)"},{"line_number":280,"context_line":"                    raise exception.VolumeBackendAPIException(reason\u003dmsg)"}],"source_content_type":"text/x-python","patch_set":55,"id":"976e4efd_b68f41de","line":277,"in_reply_to":"1b361bbb_4ce51a8b","updated":"2026-05-05 21:07:38.000000000","message":"\u003e pep8: F504 \u0027...\u0027 % ... has unused named argument(s): error\n\nPlease fix.","commit_id":"3c79700387d162077eef8bcc7f6298a952089cfa"}],"cinder/volume/drivers/ibm/storwize_svc/storwize_svc_iscsi.py":[{"author":{"_account_id":36538,"name":"Harsh Ailani","display_name":"Harsh Ailani","email":"harsh.ailani@ibm.com","username":"harshailani"},"change_message_id":"e21456c3426350815c090d8ede29f609f2a9e8fb","unresolved":false,"context_lines":[{"line_number":199,"context_line":"                LOG.info(msg)"},{"line_number":200,"context_line":"                host_name \u003d backend_helper.get_host_from_connector(connector,"},{"line_number":201,"context_line":"                                                                   iscsi\u003dTrue,"},{"line_number":202,"context_line":"                                                                   partition_name\u003dpartition_name)"},{"line_number":203,"context_line":"            else:"},{"line_number":204,"context_line":"                msg \u003d (_(\u0027Error creating host %(ex)s\u0027), {\u0027ex\u0027: excp.msg})"},{"line_number":205,"context_line":"                LOG.error(msg)"}],"source_content_type":"text/x-python","patch_set":8,"id":"73dc2a14_310b61ab","line":202,"in_reply_to":"1da13d7f_f6f58644","updated":"2025-05-28 09:42:11.000000000","message":"resolved.","commit_id":"b856589e7e79fc45f223abfa94959971924ed895"},{"author":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"change_message_id":"955a16b1c7028d0bc451137728112605482dc926","unresolved":true,"context_lines":[{"line_number":199,"context_line":"                LOG.info(msg)"},{"line_number":200,"context_line":"                host_name \u003d backend_helper.get_host_from_connector(connector,"},{"line_number":201,"context_line":"                                                                   iscsi\u003dTrue,"},{"line_number":202,"context_line":"                                                                   partition_name\u003dpartition_name)"},{"line_number":203,"context_line":"            else:"},{"line_number":204,"context_line":"                msg \u003d (_(\u0027Error creating host %(ex)s\u0027), {\u0027ex\u0027: excp.msg})"},{"line_number":205,"context_line":"                LOG.error(msg)"}],"source_content_type":"text/x-python","patch_set":8,"id":"1da13d7f_f6f58644","line":202,"in_reply_to":"b5aa07e1_b8dbe1e3","updated":"2025-05-28 08:31:45.000000000","message":"\u003e pep8: E501 line too long (97 \u003e 79 characters)\n\nPlease fix.","commit_id":"b856589e7e79fc45f223abfa94959971924ed895"}]}