)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"fd455f5c787edb75f1c868325edb27464c24cc9e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"53e36911_c69e1f40","updated":"2026-08-20 04:03:00.000000000","message":"recheck","commit_id":"1520effc90624e129b4ae94664fc6c95113ab594"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"c1c5ab6bbde0594098acc5700eb2961badc03c5b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"27bb08c4_1007af85","updated":"2026-08-26 17:13:30.000000000","message":"Code wise looks good to me. Please address Anthony\u0027s comments.","commit_id":"e21a690ee0d86edc7fdb67fbbe876e37da8b12b6"},{"author":{"_account_id":38081,"name":"Anthony Galica","display_name":"agalica","email":"anthony.galica@hitachivantara.com","username":"agalica","status":"Hitachi Vantara"},"change_message_id":"811fbb92a7e15bf9ea1323c460b0685b64c2c8c0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"459db189_7cf93cb9","updated":"2026-08-24 22:26:03.000000000","message":"Looks like a simple enough bugfix. \n\n-1 is mostly to get your attention to minor issues in the UT file (override_config, and duplicate line).","commit_id":"e21a690ee0d86edc7fdb67fbbe876e37da8b12b6"}],"cinder/tests/unit/volume/drivers/netapp/dataontap/utils/test_data_motion.py":[{"author":{"_account_id":38081,"name":"Anthony Galica","display_name":"agalica","email":"anthony.galica@hitachivantara.com","username":"agalica","status":"Hitachi Vantara"},"change_message_id":"811fbb92a7e15bf9ea1323c460b0685b64c2c8c0","unresolved":true,"context_lines":[{"line_number":106,"context_line":"                          group\u003dself.dest_backend)"},{"line_number":107,"context_line":"        CONF.set_override(\u0027netapp_server_hostname\u0027, \u0027127.0.0.1\u0027,"},{"line_number":108,"context_line":"                          group\u003dself.src_backend)"},{"line_number":109,"context_line":"        CONF.set_override(\u0027netapp_server_hostname\u0027, \u0027127.0.0.1\u0027,"},{"line_number":110,"context_line":"                          group\u003dself.dest_backend)"},{"line_number":111,"context_line":""},{"line_number":112,"context_line":"    @ddt.data(None, [], [{\u0027some_key\u0027: \u0027some_value\u0027}])"}],"source_content_type":"text/x-python","patch_set":2,"id":"7ac7faa4_67beb70b","line":109,"updated":"2026-08-24 22:26:03.000000000","message":"duplicate","commit_id":"e21a690ee0d86edc7fdb67fbbe876e37da8b12b6"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"c1c5ab6bbde0594098acc5700eb2961badc03c5b","unresolved":true,"context_lines":[{"line_number":106,"context_line":"                          group\u003dself.dest_backend)"},{"line_number":107,"context_line":"        CONF.set_override(\u0027netapp_server_hostname\u0027, \u0027127.0.0.1\u0027,"},{"line_number":108,"context_line":"                          group\u003dself.src_backend)"},{"line_number":109,"context_line":"        CONF.set_override(\u0027netapp_server_hostname\u0027, \u0027127.0.0.1\u0027,"},{"line_number":110,"context_line":"                          group\u003dself.dest_backend)"},{"line_number":111,"context_line":""},{"line_number":112,"context_line":"    @ddt.data(None, [], [{\u0027some_key\u0027: \u0027some_value\u0027}])"}],"source_content_type":"text/x-python","patch_set":2,"id":"1fd04337_ae9a14bc","line":109,"in_reply_to":"7ac7faa4_67beb70b","updated":"2026-08-26 17:13:30.000000000","message":"+1. Needs fixing.","commit_id":"e21a690ee0d86edc7fdb67fbbe876e37da8b12b6"},{"author":{"_account_id":38081,"name":"Anthony Galica","display_name":"agalica","email":"anthony.galica@hitachivantara.com","username":"agalica","status":"Hitachi Vantara"},"change_message_id":"811fbb92a7e15bf9ea1323c460b0685b64c2c8c0","unresolved":true,"context_lines":[{"line_number":1643,"context_line":"            utils.get_backend_configuration.side_effect \u003d ("},{"line_number":1644,"context_line":"                exception.ConfigNotFound(message\u003d\u0027missing dest backend\u0027))"},{"line_number":1645,"context_line":"        else:"},{"line_number":1646,"context_line":"            self.mock_dest_config.netapp_server_hostname \u003d None"},{"line_number":1647,"context_line":"        self.dm_mixin._migrate_volume_to_pool \u003d mock.Mock()"},{"line_number":1648,"context_line":"        self.dm_mixin._migrate_volume_to_vserver \u003d mock.Mock()"},{"line_number":1649,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"4ace5adf_f8119f39","line":1646,"updated":"2026-08-24 22:26:03.000000000","message":"I think you also want to use override_config here as well","commit_id":"e21a690ee0d86edc7fdb67fbbe876e37da8b12b6"},{"author":{"_account_id":38081,"name":"Anthony Galica","display_name":"agalica","email":"anthony.galica@hitachivantara.com","username":"agalica","status":"Hitachi Vantara"},"change_message_id":"32177c8d8f8d7a8bbd00ea589b18ab4198ebe513","unresolved":false,"context_lines":[{"line_number":1643,"context_line":"            utils.get_backend_configuration.side_effect \u003d ("},{"line_number":1644,"context_line":"                exception.ConfigNotFound(message\u003d\u0027missing dest backend\u0027))"},{"line_number":1645,"context_line":"        else:"},{"line_number":1646,"context_line":"            self.mock_dest_config.netapp_server_hostname \u003d None"},{"line_number":1647,"context_line":"        self.dm_mixin._migrate_volume_to_pool \u003d mock.Mock()"},{"line_number":1648,"context_line":"        self.dm_mixin._migrate_volume_to_vserver \u003d mock.Mock()"},{"line_number":1649,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"078d8e69_d372b1a3","line":1646,"in_reply_to":"4ace5adf_f8119f39","updated":"2026-08-28 22:30:29.000000000","message":"Cancel that - I believe I looked at this incorrectly during review.  Just the duplicate needs fixing.","commit_id":"e21a690ee0d86edc7fdb67fbbe876e37da8b12b6"}]}