)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"450592f19ec7badd4d04881383848b2e3cfdb0d5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"42146cbf_8429599b","updated":"2025-05-28 13:53:14.000000000","message":"See comment inline about the pep8 job problem.","commit_id":"e484b151b72bbbcbe10079a7afe8451147002f23"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"620d8a0707334a55cb8cefd7f440918a0389a869","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"d478390b_745b5b6d","updated":"2025-06-02 12:55:21.000000000","message":"1] if it helps, the documentation can be updated\nhttps://github.com/openstack/cinder/blob/master/doc/source/configuration/block-storage/drivers/ibm-storwize-svc-driver.rst\n\n2] does PBHA volume mean policy based high availability volume ?\n\n3] inline comment about version","commit_id":"f2e9a78e5986f00dfb60ed8fe7ee7b243511063c"}],"cinder/volume/drivers/ibm/storwize_svc/storwize_svc_common.py":[{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"620d8a0707334a55cb8cefd7f440918a0389a869","unresolved":true,"context_lines":[{"line_number":3576,"context_line":"              mode"},{"line_number":3577,"context_line":"        2.1.1 - Update replication to version 2.1"},{"line_number":3578,"context_line":"        2.1.2 - Added support volume_group (Flash copy)"},{"line_number":3579,"context_line":"    \"\"\""},{"line_number":3580,"context_line":""},{"line_number":3581,"context_line":"    VERSION \u003d \"2.1.2\""},{"line_number":3582,"context_line":"    VDISKCOPYOPS_INTERVAL \u003d 600"}],"source_content_type":"text/x-python","patch_set":2,"id":"8cec2b64_6b661be0","line":3579,"updated":"2025-06-02 12:55:21.000000000","message":"Since this is feature, the version can be incremented.","commit_id":"f2e9a78e5986f00dfb60ed8fe7ee7b243511063c"},{"author":{"_account_id":36800,"name":"Vrinda Dhakad","display_name":"vdhakad","email":"vrinda.dhakad@ibm.com","username":"vdhakad"},"change_message_id":"fc0c00bdcfc3ca84cbf6b34f1bbe786b005c4ddd","unresolved":false,"context_lines":[{"line_number":3576,"context_line":"              mode"},{"line_number":3577,"context_line":"        2.1.1 - Update replication to version 2.1"},{"line_number":3578,"context_line":"        2.1.2 - Added support volume_group (Flash copy)"},{"line_number":3579,"context_line":"    \"\"\""},{"line_number":3580,"context_line":""},{"line_number":3581,"context_line":"    VERSION \u003d \"2.1.2\""},{"line_number":3582,"context_line":"    VDISKCOPYOPS_INTERVAL \u003d 600"}],"source_content_type":"text/x-python","patch_set":2,"id":"8ab5b9be_425b9e69","line":3579,"in_reply_to":"74b74c8d_ed7db741","updated":"2025-11-20 06:00:21.000000000","message":"Version is bumped","commit_id":"f2e9a78e5986f00dfb60ed8fe7ee7b243511063c"},{"author":{"_account_id":36800,"name":"Vrinda Dhakad","display_name":"vdhakad","email":"vrinda.dhakad@ibm.com","username":"vdhakad"},"change_message_id":"3254a3672f912c768c8b5686504d66d481695ee0","unresolved":true,"context_lines":[{"line_number":3576,"context_line":"              mode"},{"line_number":3577,"context_line":"        2.1.1 - Update replication to version 2.1"},{"line_number":3578,"context_line":"        2.1.2 - Added support volume_group (Flash copy)"},{"line_number":3579,"context_line":"    \"\"\""},{"line_number":3580,"context_line":""},{"line_number":3581,"context_line":"    VERSION \u003d \"2.1.2\""},{"line_number":3582,"context_line":"    VDISKCOPYOPS_INTERVAL \u003d 600"}],"source_content_type":"text/x-python","patch_set":2,"id":"74b74c8d_ed7db741","line":3579,"in_reply_to":"8cec2b64_6b661be0","updated":"2025-06-05 09:29:01.000000000","message":"This feature is WIP, will definitely increment version along with documentation when the feature comes together.","commit_id":"f2e9a78e5986f00dfb60ed8fe7ee7b243511063c"},{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"6d29633f58c5f8af2139535425f83df1e11b056b","unresolved":true,"context_lines":[{"line_number":621,"context_line":"            with excutils.save_and_reraise_exception():"},{"line_number":622,"context_line":"                LOG.exception(\u0027Failed to create volumegroup.\u0027)"},{"line_number":623,"context_line":""},{"line_number":624,"context_line":"    def lsvolumegroup(self, volumegroup_id_or_name\u003dNone):"},{"line_number":625,"context_line":"        \"\"\"Return volume group attributes or None if it doesn\u0027t exist.\"\"\""},{"line_number":626,"context_line":"        ssh_cmd \u003d [\u0027svcinfo\u0027, \u0027lsvolumegroup\u0027, \u0027-bytes\u0027, \u0027-delim\u0027, \u0027!\u0027,"},{"line_number":627,"context_line":"                   \u0027\"%s\"\u0027 % volumegroup_id_or_name]"}],"source_content_type":"text/x-python","patch_set":40,"id":"a1bda28b_555bb9f2","line":624,"updated":"2026-04-07 14:55:23.000000000","message":"If the intent is to allow listing all volume groups (no filter), handle `None`. comment and change are not in sync.","commit_id":"9f32b5571eba1cef26c935e6cef357f18fc92edf"},{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"6d29633f58c5f8af2139535425f83df1e11b056b","unresolved":true,"context_lines":[{"line_number":6565,"context_line":"        # FlashCopy is not allowed inside partition"},{"line_number":6566,"context_line":"        if self._get_config_param_value(\u0027migrate_from_flashcopy\u0027,"},{"line_number":6567,"context_line":"                                        None) is False:"},{"line_number":6568,"context_line":"            msg \u003d _(\u0027FlashCopy is not allowed inside PBHA partition.\u0027"},{"line_number":6569,"context_line":"                    \u0027Set migrate_from_flashcopy to True.\u0027)"},{"line_number":6570,"context_line":"            LOG.error(msg)"},{"line_number":6571,"context_line":"            raise exception.InvalidInput(message\u003dmsg)"}],"source_content_type":"text/x-python","patch_set":40,"id":"1c35153e_f09eb201","line":6568,"updated":"2026-04-07 14:55:23.000000000","message":"Comment says \"FlashCopy is not allowed\" but the error message says \"Set to True\" (meaning allow FlashCopy migration). These contradict.","commit_id":"9f32b5571eba1cef26c935e6cef357f18fc92edf"}],"cinder/volume/drivers/ibm/storwize_svc/storwize_svc_iscsi.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"450592f19ec7badd4d04881383848b2e3cfdb0d5","unresolved":true,"context_lines":[{"line_number":197,"context_line":"                msg \u003d (_(\u0027Host already exists for connector \u0027"},{"line_number":198,"context_line":"                         \u0027%(conn)s\u0027), {\u0027conn\u0027: connector})"},{"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":1,"id":"fb9f69b3_751e310f","line":202,"range":{"start_line":200,"start_character":0,"end_line":202,"end_character":97},"updated":"2025-05-28 13:53:14.000000000","message":"This is causing your pep8 problem ... it\u0027s from patch set #8 of one of your unmerged ancestor patches, and was corrected in patch set #9:\n\nhttps://review.opendev.org/c/openstack/cinder/+/943364/8..9/cinder/volume/drivers/ibm/storwize_svc/storwize_svc_iscsi.py\n\nYou\u0027ll need to rebase your patch onto the current version(s) of parent patches.","commit_id":"e484b151b72bbbcbe10079a7afe8451147002f23"},{"author":{"_account_id":36538,"name":"Harsh Ailani","display_name":"Harsh Ailani","email":"harsh.ailani@ibm.com","username":"harshailani"},"change_message_id":"569bfe9e08b8f979945310c5782ebafb376fd267","unresolved":false,"context_lines":[{"line_number":197,"context_line":"                msg \u003d (_(\u0027Host already exists for connector \u0027"},{"line_number":198,"context_line":"                         \u0027%(conn)s\u0027), {\u0027conn\u0027: connector})"},{"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":1,"id":"bfe771c0_729a54a9","line":202,"range":{"start_line":200,"start_character":0,"end_line":202,"end_character":97},"in_reply_to":"fb9f69b3_751e310f","updated":"2025-06-06 07:40:48.000000000","message":"Resolved","commit_id":"e484b151b72bbbcbe10079a7afe8451147002f23"}]}