)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"bb55320d140b0a32a9119269e6a3ddb8424fc36e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ff6ea483_98d2d804","updated":"2022-05-25 07:16:23.000000000","message":"1] A release note may be required.\n\n2] Small query/comment inline.\n","commit_id":"fbcb41a2785ea1217095b9609fa00b3f8a10b470"},{"author":{"_account_id":33648,"name":"Nahim Alves de Souza","email":"nahimsouza@outlook.com","username":"nahimsouza"},"change_message_id":"31e75a07474b3ab0930f88e3944732d44370f788","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"76e10cd7_20228792","updated":"2022-05-24 17:11:02.000000000","message":"Code and UTs look good to me. Thanks for fixing this, Matheus.","commit_id":"fbcb41a2785ea1217095b9609fa00b3f8a10b470"},{"author":{"_account_id":12988,"name":"Peter Penchev","email":"openstack-dev@storpool.com","username":"ppenchev"},"change_message_id":"fa96ded587952cfcb9ac81bfb9b8c4381e03a5a1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"34a8821f_84dbc0ba","updated":"2022-05-24 23:16:19.000000000","message":"run-storpoolci","commit_id":"fbcb41a2785ea1217095b9609fa00b3f8a10b470"},{"author":{"_account_id":33301,"name":"Caique Mello","email":"caiquemellosbo@gmail.com","username":"MelloCaique"},"change_message_id":"62b8bec0c6ab77610a2ef9346f91f959347765b6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"6b50bb37_244f7e94","updated":"2022-07-06 16:48:18.000000000","message":"LGTM!","commit_id":"3cf4ad1a23d12558230fae7dc1bc628b61057a2b"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"a6b7e30c8de1d13709d76267e932feb88d6ad0eb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"0df4519d_dc4ee6f7","updated":"2022-06-02 06:31:12.000000000","message":"My comment from previous patchset has been addressed. Zuul also passed. so +1.\nRegarding release notes, let core reviewers decide.","commit_id":"3cf4ad1a23d12558230fae7dc1bc628b61057a2b"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"bc4ddb68194cc50de94b61e4432207524ea31bb0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"d573c352_7f262b8d","updated":"2026-03-25 06:11:00.000000000","message":"Release notes are missing. Please add release notes.","commit_id":"3cf4ad1a23d12558230fae7dc1bc628b61057a2b"},{"author":{"_account_id":35586,"name":"Renan Vitor","email":"renanv@netapp.com","username":"renan_vitor"},"change_message_id":"aebf4be468b55f3ba3880f866b90846105f92a9a","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":4,"id":"ac91809c_f12a64a5","updated":"2023-07-04 13:33:59.000000000","message":"Thank you for the fix Matheus, please take a look at the comments inline.\nAlso could you please add a release note as well?","commit_id":"3cf4ad1a23d12558230fae7dc1bc628b61057a2b"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"c3aded2c0b814d590aca0b4e1cd111d9db7e82dd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"e08a90f9_518453b8","updated":"2026-03-25 06:11:42.000000000","message":"run-NetApp CI","commit_id":"3cf4ad1a23d12558230fae7dc1bc628b61057a2b"}],"cinder/volume/drivers/netapp/dataontap/client/client_cmode.py":[{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"bb55320d140b0a32a9119269e6a3ddb8424fc36e","unresolved":true,"context_lines":[{"line_number":682,"context_line":"    def start_file_copy(self, file_name, dest_ontap_volume,"},{"line_number":683,"context_line":"                        src_ontap_volume\u003dNone,"},{"line_number":684,"context_line":"                        dest_file_name\u003dNone,"},{"line_number":685,"context_line":"                        src_vserver\u003dNone, dst_vserver\u003dNone):"},{"line_number":686,"context_line":"        \"\"\"Starts a file copy operation between ONTAP volumes.\"\"\""},{"line_number":687,"context_line":"        if dst_vserver is None:"},{"line_number":688,"context_line":"            dst_vserver \u003d src_vserver"}],"source_content_type":"text/x-python","patch_set":2,"id":"1c0c4df4_13d4e92b","line":685,"range":{"start_line":685,"start_character":24,"end_line":685,"end_character":40},"updated":"2022-05-25 07:16:23.000000000","message":"in my opinion, the \"src_vserver\" should always have some value; eg. source backend name.\nThus, default value of src_vserver shouldn\u0027t be None.\nAgree ?","commit_id":"fbcb41a2785ea1217095b9609fa00b3f8a10b470"},{"author":{"_account_id":34489,"name":"Matheus Andrade","email":"matheus.handrade15@gmail.com","username":"matheusandrade"},"change_message_id":"fcc433673a2c656d182f36927c034045c063852b","unresolved":false,"context_lines":[{"line_number":682,"context_line":"    def start_file_copy(self, file_name, dest_ontap_volume,"},{"line_number":683,"context_line":"                        src_ontap_volume\u003dNone,"},{"line_number":684,"context_line":"                        dest_file_name\u003dNone,"},{"line_number":685,"context_line":"                        src_vserver\u003dNone, dst_vserver\u003dNone):"},{"line_number":686,"context_line":"        \"\"\"Starts a file copy operation between ONTAP volumes.\"\"\""},{"line_number":687,"context_line":"        if dst_vserver is None:"},{"line_number":688,"context_line":"            dst_vserver \u003d src_vserver"}],"source_content_type":"text/x-python","patch_set":2,"id":"796779b4_364c2e95","line":685,"range":{"start_line":685,"start_character":24,"end_line":685,"end_character":40},"in_reply_to":"1c0c4df4_13d4e92b","updated":"2022-05-26 14:19:19.000000000","message":"Done","commit_id":"fbcb41a2785ea1217095b9609fa00b3f8a10b470"},{"author":{"_account_id":35586,"name":"Renan Vitor","email":"renanv@netapp.com","username":"renan_vitor"},"change_message_id":"aebf4be468b55f3ba3880f866b90846105f92a9a","unresolved":true,"context_lines":[{"line_number":686,"context_line":"                        dst_vserver\u003dNone):"},{"line_number":687,"context_line":"        \"\"\"Starts a file copy operation between ONTAP volumes.\"\"\""},{"line_number":688,"context_line":"        if dst_vserver is None:"},{"line_number":689,"context_line":"            dst_vserver \u003d src_vserver"},{"line_number":690,"context_line":"        if src_ontap_volume is None:"},{"line_number":691,"context_line":"            src_ontap_volume \u003d dest_ontap_volume"},{"line_number":692,"context_line":"        if dest_file_name is None:"}],"source_content_type":"text/x-python","patch_set":4,"id":"e016277a_cf6f6bff","line":689,"updated":"2023-07-04 13:33:59.000000000","message":"Do you thinks we should add a test case where this line is tested? a case where the dest_vserver is equal to src_vserver","commit_id":"3cf4ad1a23d12558230fae7dc1bc628b61057a2b"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"fb2e6220ae45682afc9be0dd7d1205d9ecb282ac","unresolved":true,"context_lines":[{"line_number":694,"context_line":""},{"line_number":695,"context_line":"        api_args \u003d {"},{"line_number":696,"context_line":"            \u0027source-paths\u0027: [{"},{"line_number":697,"context_line":"                \u0027sfod-operation-path\u0027: \u0027%s:%s/%s\u0027 % (src_vserver,"},{"line_number":698,"context_line":"                                                     src_ontap_volume,"},{"line_number":699,"context_line":"                                                     file_name)"},{"line_number":700,"context_line":"            }],"}],"source_content_type":"text/x-python","patch_set":4,"id":"8500dd30_7dca13f9","line":697,"updated":"2026-04-08 13:46:21.000000000","message":"I have doubts that this will work without SVM peers configured between the two SVMs. ONTAP will throw error if the SVMs are not peered.","commit_id":"3cf4ad1a23d12558230fae7dc1bc628b61057a2b"}]}