)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"3c6d088ebd9f24ea3535a8a2b653fdd0720dc257","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"7b2de718_fd7897c7","updated":"2026-01-26 15:23:16.000000000","message":"Code needs updates to pass pep8","commit_id":"b1f9a930497752d5159a58eaa47a31ef6c8512a4"},{"author":{"_account_id":37784,"name":"Abhishek Gupta","display_name":"Abhishek Gupta","email":"Abhishek_Gupta14@Dell.com","username":"abhishek-gupta14"},"change_message_id":"55ae37b07e7c37b70b014e82a6a11975564139b7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"3ff18942_9ccbdb03","updated":"2026-01-22 08:49:24.000000000","message":"run-DellEMC PowerMax CI","commit_id":"b1f9a930497752d5159a58eaa47a31ef6c8512a4"},{"author":{"_account_id":37784,"name":"Abhishek Gupta","display_name":"Abhishek Gupta","email":"Abhishek_Gupta14@Dell.com","username":"abhishek-gupta14"},"change_message_id":"19d96475b6b02e18be327ecd7c6726be5c83e301","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"0b383370_567d5955","in_reply_to":"7b2de718_fd7897c7","updated":"2026-01-29 19:43:09.000000000","message":"@eharney@redhat.com pep8 has been passed.","commit_id":"b1f9a930497752d5159a58eaa47a31ef6c8512a4"},{"author":{"_account_id":37784,"name":"Abhishek Gupta","display_name":"Abhishek Gupta","email":"Abhishek_Gupta14@Dell.com","username":"abhishek-gupta14"},"change_message_id":"dcc340592cd4e76c6fe06eceb9c8d04b6ca9cdf6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"63160a90_19e5c69c","updated":"2026-02-17 19:01:55.000000000","message":"@eharney@redhat.com @rosmaita.fossdev@gmail.com @rajatdhasmana@gmail.com\nCould you please help in reviewing this patch. \nThank you","commit_id":"dd667d8c41fcdf4b48b7f272105c6e83711ce21f"},{"author":{"_account_id":37784,"name":"Abhishek Gupta","display_name":"Abhishek Gupta","email":"Abhishek_Gupta14@Dell.com","username":"abhishek-gupta14"},"change_message_id":"2ff25b5f2cf6a136fd1d904341cf6b2d34f28d67","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"83bcecc5_55f6cc4e","updated":"2026-02-01 19:41:00.000000000","message":"@rosmaita.fossdev@gmail.com @rajatdhasmana@gmail.com\nCould you please help in reviewing this patch.\nThank you.","commit_id":"dd667d8c41fcdf4b48b7f272105c6e83711ce21f"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"162826379b52b625b5c7f025e328cf7a5e0ffb3c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"23790381_456418b2","updated":"2026-04-07 05:04:40.000000000","message":"recheck","commit_id":"dd667d8c41fcdf4b48b7f272105c6e83711ce21f"},{"author":{"_account_id":37784,"name":"Abhishek Gupta","display_name":"Abhishek Gupta","email":"Abhishek_Gupta14@Dell.com","username":"abhishek-gupta14"},"change_message_id":"f9e47e28629dce30c705c0525d33d9c9c641b359","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"cefe6dcd_b062df09","updated":"2026-01-29 18:36:36.000000000","message":"recheck","commit_id":"dd667d8c41fcdf4b48b7f272105c6e83711ce21f"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"7d1ddce77fd4b33e36f24ea1fcce04a17ff77588","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"1efa6a53_8808eca2","updated":"2026-09-09 14:49:11.000000000","message":"run-DellEMC PowerMax CI","commit_id":"67a8f2f1a4c0b3f6415170e0fbc0b8380dbc5445"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"f367928d5ffe9993551d28ed33774c30823e56f1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"223165f2_9501af42","updated":"2026-09-10 10:07:24.000000000","message":"run-DellEMC PowerMax CI","commit_id":"67a8f2f1a4c0b3f6415170e0fbc0b8380dbc5445"},{"author":{"_account_id":38286,"name":"Siddharth Kumar","display_name":"Siddharth","email":"siddharth.kumar1@dell.com","username":"siddhvrth"},"change_message_id":"b449a7225abe06856e7f43a19d9d39f49a69352b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"3c6b599e_e19704a2","updated":"2026-11-12 11:58:59.000000000","message":"run-DellEMC PowerMax CI","commit_id":"67a8f2f1a4c0b3f6415170e0fbc0b8380dbc5445"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"23610533ab9ba89c18aac9305932caa7a4afb5dc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"57654f62_e402ad5d","updated":"2026-09-09 12:45:26.000000000","message":"run-DellEMC PowerMax CI","commit_id":"67a8f2f1a4c0b3f6415170e0fbc0b8380dbc5445"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"8c98612e73cda78a69d7f8e6aa034852dca284d5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"be1bcb76_9e8b9fb7","updated":"2026-09-09 18:25:14.000000000","message":"run-DellEMC PowerMax CI","commit_id":"67a8f2f1a4c0b3f6415170e0fbc0b8380dbc5445"}],"cinder/tests/unit/volume/drivers/dell_emc/powermax/test_powermax_common.py":[{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"2ce1824033ca36c091f06415cbf9bcc52bca808a","unresolved":true,"context_lines":[{"line_number":1707,"context_line":"            mock_add.assert_not_called()"},{"line_number":1708,"context_line":""},{"line_number":1709,"context_line":"    @mock.patch.object(utils.PowerMaxUtils, \u0027is_volume_failed_over\u0027,"},{"line_number":1710,"context_line":"                       retrun_value\u003dTrue)"},{"line_number":1711,"context_line":"    @mock.patch.object(common.PowerMaxCommon, \u0027_get_replication_extra_specs\u0027,"},{"line_number":1712,"context_line":"                       return_value\u003dtpd.PowerMaxData.rep_extra_specs)"},{"line_number":1713,"context_line":"    def test_get_target_wwns_from_masking_view(self, mock_rep_specs, mock_fo):"}],"source_content_type":"text/x-python","patch_set":6,"id":"07cd5892_b0bb8bd0","line":1710,"updated":"2026-04-08 16:47:03.000000000","message":"return?","commit_id":"dd667d8c41fcdf4b48b7f272105c6e83711ce21f"},{"author":{"_account_id":37784,"name":"Abhishek Gupta","display_name":"Abhishek Gupta","email":"Abhishek_Gupta14@Dell.com","username":"abhishek-gupta14"},"change_message_id":"26ee6b4e34a114a37a38b41da2a5aa3bf5fc9b88","unresolved":false,"context_lines":[{"line_number":1707,"context_line":"            mock_add.assert_not_called()"},{"line_number":1708,"context_line":""},{"line_number":1709,"context_line":"    @mock.patch.object(utils.PowerMaxUtils, \u0027is_volume_failed_over\u0027,"},{"line_number":1710,"context_line":"                       retrun_value\u003dTrue)"},{"line_number":1711,"context_line":"    @mock.patch.object(common.PowerMaxCommon, \u0027_get_replication_extra_specs\u0027,"},{"line_number":1712,"context_line":"                       return_value\u003dtpd.PowerMaxData.rep_extra_specs)"},{"line_number":1713,"context_line":"    def test_get_target_wwns_from_masking_view(self, mock_rep_specs, mock_fo):"}],"source_content_type":"text/x-python","patch_set":6,"id":"c11fa99c_c6c8cf2e","line":1710,"in_reply_to":"07cd5892_b0bb8bd0","updated":"2026-04-17 21:08:53.000000000","message":"Done","commit_id":"dd667d8c41fcdf4b48b7f272105c6e83711ce21f"},{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"2ce1824033ca36c091f06415cbf9bcc52bca808a","unresolved":true,"context_lines":[{"line_number":5366,"context_line":"        mock_get_sg.assert_called_once()"},{"line_number":5367,"context_line":"        mock_del_sg.assert_called_once()"},{"line_number":5368,"context_line":"        mock_remove_vol.assert_called_once()"},{"line_number":5369,"context_line":"        mock_remove_vol.mock_remove_vol()"},{"line_number":5370,"context_line":""},{"line_number":5371,"context_line":"    @mock.patch.object("},{"line_number":5372,"context_line":"        common.PowerMaxCommon, \u0027_get_src_device_id_for_group_snap\u0027)"}],"source_content_type":"text/x-python","patch_set":6,"id":"fdf11db3_5a3de48f","line":5369,"updated":"2026-04-08 16:47:03.000000000","message":"what this method is doing at end of function?","commit_id":"dd667d8c41fcdf4b48b7f272105c6e83711ce21f"},{"author":{"_account_id":37784,"name":"Abhishek Gupta","display_name":"Abhishek Gupta","email":"Abhishek_Gupta14@Dell.com","username":"abhishek-gupta14"},"change_message_id":"26ee6b4e34a114a37a38b41da2a5aa3bf5fc9b88","unresolved":false,"context_lines":[{"line_number":5366,"context_line":"        mock_get_sg.assert_called_once()"},{"line_number":5367,"context_line":"        mock_del_sg.assert_called_once()"},{"line_number":5368,"context_line":"        mock_remove_vol.assert_called_once()"},{"line_number":5369,"context_line":"        mock_remove_vol.mock_remove_vol()"},{"line_number":5370,"context_line":""},{"line_number":5371,"context_line":"    @mock.patch.object("},{"line_number":5372,"context_line":"        common.PowerMaxCommon, \u0027_get_src_device_id_for_group_snap\u0027)"}],"source_content_type":"text/x-python","patch_set":6,"id":"a51b2dd5_da8c69be","line":5369,"in_reply_to":"fdf11db3_5a3de48f","updated":"2026-04-17 21:08:53.000000000","message":"Done","commit_id":"dd667d8c41fcdf4b48b7f272105c6e83711ce21f"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"dfe9b4b739cc788e1a55c1e9faa57ab716a630b5","unresolved":true,"context_lines":[{"line_number":8637,"context_line":"        target_extra_specs[utils.REP_CONFIG] \u003d self.data.rep_config_sync"},{"line_number":8638,"context_line":"        target_extra_specs[\u0027rep_mode\u0027] \u003d utils.REP_SYNC"},{"line_number":8639,"context_line":"        device_info \u003d {\u0027storageGroupId\u0027: [\u0027sg1\u0027, \u0027mgmt_sg\u0027]}"},{"line_number":8640,"context_line":"        with mock.patch.object(self.rest, \u0027get_volume\u0027,"},{"line_number":8641,"context_line":"                               return_value\u003ddevice_info):"},{"line_number":8642,"context_line":"            with mock.patch.object("},{"line_number":8643,"context_line":"                    self.utils, \u0027get_rdf_management_group_name\u0027,"}],"source_content_type":"text/x-python","patch_set":11,"id":"96b6fec1_b9a94732","line":8640,"updated":"2026-11-12 14:09:24.000000000","message":"This test (and many here) would really benefit by structuring mocks in a way to avoid the arrow anti-pattern of continually indented with blocks.\n\nThere are a few ways to do this, one idea is\n\n```\nwith (mock.patch.object(a, b),\n      mock.patch.object(c, d),\n      mock.patch.object(e, f)):\n  self.assert()\n```\n  \nOr you can use self.mock_object() calls instead of mock.patch.object.\n\nThis would make this much more readable.","commit_id":"67a8f2f1a4c0b3f6415170e0fbc0b8380dbc5445"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"dfe9b4b739cc788e1a55c1e9faa57ab716a630b5","unresolved":true,"context_lines":[{"line_number":8796,"context_line":"        extra_specs \u003d deepcopy(self.data.extra_specs)"},{"line_number":8797,"context_line":"        extra_specs[utils.REP_CONFIG] \u003d self.data.rep_config_sync"},{"line_number":8798,"context_line":"        rep_extra_specs \u003d deepcopy(self.data.rep_extra_specs)"},{"line_number":8799,"context_line":"        with mock.patch.object(self.common, \u0027_initial_setup\u0027,"},{"line_number":8800,"context_line":"                               return_value\u003dextra_specs):"},{"line_number":8801,"context_line":"            with mock.patch.object("},{"line_number":8802,"context_line":"                    self.common, \u0027_get_replication_extra_specs\u0027,"}],"source_content_type":"text/x-python","patch_set":11,"id":"caeacb22_c2aa8456","line":8799,"updated":"2026-11-12 14:09:24.000000000","message":"Same here","commit_id":"67a8f2f1a4c0b3f6415170e0fbc0b8380dbc5445"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"dfe9b4b739cc788e1a55c1e9faa57ab716a630b5","unresolved":true,"context_lines":[{"line_number":8890,"context_line":"        group \u003d mock.MagicMock()"},{"line_number":8891,"context_line":"        group.id \u003d \u0027grp_id\u0027"},{"line_number":8892,"context_line":"        volume \u003d deepcopy(self.data.test_volume)"},{"line_number":8893,"context_line":"        with mock.patch.object(self.common, \u0027_initial_setup\u0027,"},{"line_number":8894,"context_line":"                               return_value\u003dextra_specs):"},{"line_number":8895,"context_line":"            with mock.patch.object("},{"line_number":8896,"context_line":"                    self.common, \u0027get_rdf_details\u0027,"}],"source_content_type":"text/x-python","patch_set":11,"id":"77d8858b_11178dce","line":8893,"updated":"2026-11-12 14:09:24.000000000","message":"Same","commit_id":"67a8f2f1a4c0b3f6415170e0fbc0b8380dbc5445"}],"cinder/tests/unit/volume/drivers/dell_emc/powermax/test_powermax_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":"2ce1824033ca36c091f06415cbf9bcc52bca808a","unresolved":true,"context_lines":[{"line_number":304,"context_line":"            mock_unmanage.assert_called_once_with("},{"line_number":305,"context_line":"                self.data.test_volume)"},{"line_number":306,"context_line":""},{"line_number":307,"context_line":"    def manage_existing_snapshot(self):"},{"line_number":308,"context_line":"        with mock.patch.object(self.common, \u0027manage_existing_snapshot\u0027,"},{"line_number":309,"context_line":"                               return_value\u003d{}) as mock_manage_snapshot:"},{"line_number":310,"context_line":"            self.driver.manage_existing_snapshot("}],"source_content_type":"text/x-python","patch_set":6,"id":"2e0c394a_b0842272","line":307,"updated":"2026-04-08 16:47:03.000000000","message":"test prefix is missing. what is the intent of this function?","commit_id":"dd667d8c41fcdf4b48b7f272105c6e83711ce21f"},{"author":{"_account_id":37784,"name":"Abhishek Gupta","display_name":"Abhishek Gupta","email":"Abhishek_Gupta14@Dell.com","username":"abhishek-gupta14"},"change_message_id":"26ee6b4e34a114a37a38b41da2a5aa3bf5fc9b88","unresolved":false,"context_lines":[{"line_number":304,"context_line":"            mock_unmanage.assert_called_once_with("},{"line_number":305,"context_line":"                self.data.test_volume)"},{"line_number":306,"context_line":""},{"line_number":307,"context_line":"    def manage_existing_snapshot(self):"},{"line_number":308,"context_line":"        with mock.patch.object(self.common, \u0027manage_existing_snapshot\u0027,"},{"line_number":309,"context_line":"                               return_value\u003d{}) as mock_manage_snapshot:"},{"line_number":310,"context_line":"            self.driver.manage_existing_snapshot("}],"source_content_type":"text/x-python","patch_set":6,"id":"d628084c_83d8e144","line":307,"in_reply_to":"2e0c394a_b0842272","updated":"2026-04-17 21:08:53.000000000","message":"Done","commit_id":"dd667d8c41fcdf4b48b7f272105c6e83711ce21f"}]}