)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":38623,"name":"Sanketh Shetty","display_name":"Sanketh Shetty","email":"sanketh.shetty@netapp.com","username":"sanketh-netapp","status":"NetApp"},"change_message_id":"1278eeba41b0251372b6fea56baf08f4c7e91838","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"915bd8b9_55db94cf","updated":"2026-03-10 05:04:35.000000000","message":"Gating this merge. It must go after merge of - https://review.opendev.org/c/openstack/cinder/+/975880","commit_id":"1e34a731582fd2ad1fd025e4b82176e09ffb0d32"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"efb25d6e27cdb863555ec97d611a8e89ae9a2a6b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"82212553_db0e6720","updated":"2026-02-21 04:42:06.000000000","message":"LGTM","commit_id":"1e34a731582fd2ad1fd025e4b82176e09ffb0d32"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"f86b8a26d295ce6fde2166c4ea4c172d5587c0ba","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"66d81df6_26065412","updated":"2026-03-09 18:32:17.000000000","message":"One minor things with reraising an exception that would be good to fix, but I don\u0027t think it\u0027s blocking. Otherwise this looks fine to me, with the known third party CI issues.","commit_id":"1e34a731582fd2ad1fd025e4b82176e09ffb0d32"},{"author":{"_account_id":38623,"name":"Sanketh Shetty","display_name":"Sanketh Shetty","email":"sanketh.shetty@netapp.com","username":"sanketh-netapp","status":"NetApp"},"change_message_id":"fbb6ab03afc4cae2ba26d40a11d1f5eda90d7f3a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"9717e145_ac4aefdd","updated":"2026-02-20 14:08:28.000000000","message":"Reviewed","commit_id":"1e34a731582fd2ad1fd025e4b82176e09ffb0d32"},{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"540940d5d89f1834658b29ac7d2a64e12dda921e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"f0f2c8a6_50cd9660","updated":"2026-03-05 16:35:02.000000000","message":"Run-NetApp CI","commit_id":"1e34a731582fd2ad1fd025e4b82176e09ffb0d32"},{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"d8e2ed431483c88b5ee858f3f05c3e3d5d341690","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"230346ef_fce7f018","updated":"2026-03-10 05:21:29.000000000","message":"Thank you! Sean for review. addressed review comments. Please review again.","commit_id":"40c1ea6d9f61b2d65d11860092c610b5b5d41465"},{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"de487ef106a7548924ac263e9e4e024a1d8d638a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"09164bb2_328ebafb","updated":"2026-03-10 15:00:21.000000000","message":"Thank you! Sean, Addressed your review comment.Please review again.","commit_id":"40c1ea6d9f61b2d65d11860092c610b5b5d41465"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"dbc780e5c2c1583f893bfc5310b46757d6476c64","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"8ed4be89_d57ee470","updated":"2026-03-10 16:24:15.000000000","message":"LGTM","commit_id":"54cfce7946fc6e8e1ab6f6ffea0e33d610c9a7c4"},{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"683174fb2b17cd0d86c8ac576c9907354e9f5452","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"6fad8961_958c8664","updated":"2026-03-11 04:31:08.000000000","message":"Run-NetApp CI","commit_id":"54cfce7946fc6e8e1ab6f6ffea0e33d610c9a7c4"},{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"9597bd774694dc6f30cc17d827b05c3456b3ee22","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"a3dadf16_5e7504c1","updated":"2026-03-11 13:57:23.000000000","message":"Run-NetApp CI","commit_id":"54cfce7946fc6e8e1ab6f6ffea0e33d610c9a7c4"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"b59c9ce03e83bb4a308616d733d2d9b0d895a533","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"69529706_3fae5a76","updated":"2026-03-10 15:06:11.000000000","message":"Thanks!","commit_id":"54cfce7946fc6e8e1ab6f6ffea0e33d610c9a7c4"},{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"e2936c8b94dfcfafe0c7b8d6f0480cbef9867963","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"537ef0ca_152121f1","updated":"2026-03-12 03:57:14.000000000","message":"recheck","commit_id":"54cfce7946fc6e8e1ab6f6ffea0e33d610c9a7c4"},{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"7a9791b1126176c5d901362f13d8df7909b3d7b9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"5e914c3c_b5d5fe33","updated":"2026-03-13 05:06:30.000000000","message":"recheck","commit_id":"54cfce7946fc6e8e1ab6f6ffea0e33d610c9a7c4"},{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"2774fe2946a1266d51d0d8198e8d4da147d2ce5e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"bb531ca7_6785e9b3","updated":"2026-03-17 10:30:43.000000000","message":"recheck","commit_id":"54cfce7946fc6e8e1ab6f6ffea0e33d610c9a7c4"}],"cinder/volume/drivers/netapp/dataontap/utils/data_motion.py":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"f86b8a26d295ce6fde2166c4ea4c172d5587c0ba","unresolved":true,"context_lines":[{"line_number":574,"context_line":"                    replication_policy)"},{"line_number":575,"context_line":"            except netapp_api.NaApiError as e:"},{"line_number":576,"context_line":"                LOG.exception(\"Failed to create SnapMirror for CG.\")"},{"line_number":577,"context_line":"                raise e"},{"line_number":578,"context_line":"        # Try to repair SnapMirror if existing"},{"line_number":579,"context_line":"        else:"},{"line_number":580,"context_line":"            LOG.debug(\"Updating snapmirror for cg from %s to %s on \""}],"source_content_type":"text/x-python","patch_set":4,"id":"2592a638_eca0e28e","line":577,"updated":"2026-03-09 18:32:17.000000000","message":"Should just be `raise`, not `raise e`, otherwise it loses its actual sourcing.","commit_id":"1e34a731582fd2ad1fd025e4b82176e09ffb0d32"},{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"d8e2ed431483c88b5ee858f3f05c3e3d5d341690","unresolved":false,"context_lines":[{"line_number":574,"context_line":"                    replication_policy)"},{"line_number":575,"context_line":"            except netapp_api.NaApiError as e:"},{"line_number":576,"context_line":"                LOG.exception(\"Failed to create SnapMirror for CG.\")"},{"line_number":577,"context_line":"                raise e"},{"line_number":578,"context_line":"        # Try to repair SnapMirror if existing"},{"line_number":579,"context_line":"        else:"},{"line_number":580,"context_line":"            LOG.debug(\"Updating snapmirror for cg from %s to %s on \""}],"source_content_type":"text/x-python","patch_set":4,"id":"4833a096_a9e0ced0","line":577,"in_reply_to":"2592a638_eca0e28e","updated":"2026-03-10 05:21:29.000000000","message":"Thank Sean, update to raise.","commit_id":"1e34a731582fd2ad1fd025e4b82176e09ffb0d32"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"f86b8a26d295ce6fde2166c4ea4c172d5587c0ba","unresolved":true,"context_lines":[{"line_number":1482,"context_line":"            secondary_backend_name)"},{"line_number":1483,"context_line":"        dest_vserver \u003d dest_backend_config.netapp_vserver"},{"line_number":1484,"context_line":""},{"line_number":1485,"context_line":"        # Failing back snapmirror from destiation back to source"},{"line_number":1486,"context_line":"        src_client.failover_snapmirror_active_sync(dest_vserver, cg_name,"},{"line_number":1487,"context_line":"                                                   src_vserver, cg_name)"},{"line_number":1488,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"475c5032_70af81e2","line":1485,"updated":"2026-03-09 18:32:17.000000000","message":"s/distiation/destination/","commit_id":"1e34a731582fd2ad1fd025e4b82176e09ffb0d32"},{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"d8e2ed431483c88b5ee858f3f05c3e3d5d341690","unresolved":false,"context_lines":[{"line_number":1482,"context_line":"            secondary_backend_name)"},{"line_number":1483,"context_line":"        dest_vserver \u003d dest_backend_config.netapp_vserver"},{"line_number":1484,"context_line":""},{"line_number":1485,"context_line":"        # Failing back snapmirror from destiation back to source"},{"line_number":1486,"context_line":"        src_client.failover_snapmirror_active_sync(dest_vserver, cg_name,"},{"line_number":1487,"context_line":"                                                   src_vserver, cg_name)"},{"line_number":1488,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"11e88bed_ed3abb39","line":1485,"in_reply_to":"475c5032_70af81e2","updated":"2026-03-10 05:21:29.000000000","message":"corrected spelling.","commit_id":"1e34a731582fd2ad1fd025e4b82176e09ffb0d32"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"9db5f8ffe21940802de97f7e8407ca948a86bb07","unresolved":true,"context_lines":[{"line_number":572,"context_line":"                    dest_cg_name,"},{"line_number":573,"context_line":"                    storage_object_names,"},{"line_number":574,"context_line":"                    replication_policy)"},{"line_number":575,"context_line":"            except netapp_api.NaApiError as e:"},{"line_number":576,"context_line":"                LOG.exception(\"Failed to create SnapMirror for CG.\")"},{"line_number":577,"context_line":"                raise"},{"line_number":578,"context_line":"        # Try to repair SnapMirror if existing"}],"source_content_type":"text/x-python","patch_set":5,"id":"2100cd32_07c5a675","line":575,"range":{"start_line":575,"start_character":12,"end_line":575,"end_character":45},"updated":"2026-03-10 13:24:42.000000000","message":"You\u0027ll also need to change this to just `except netapp_api.NaApiError:` now so the exception doesn\u0027t get assigned to the `e` variable that is no longer used.","commit_id":"40c1ea6d9f61b2d65d11860092c610b5b5d41465"},{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"de487ef106a7548924ac263e9e4e024a1d8d638a","unresolved":false,"context_lines":[{"line_number":572,"context_line":"                    dest_cg_name,"},{"line_number":573,"context_line":"                    storage_object_names,"},{"line_number":574,"context_line":"                    replication_policy)"},{"line_number":575,"context_line":"            except netapp_api.NaApiError as e:"},{"line_number":576,"context_line":"                LOG.exception(\"Failed to create SnapMirror for CG.\")"},{"line_number":577,"context_line":"                raise"},{"line_number":578,"context_line":"        # Try to repair SnapMirror if existing"}],"source_content_type":"text/x-python","patch_set":5,"id":"725fe299_b6c53d05","line":575,"range":{"start_line":575,"start_character":12,"end_line":575,"end_character":45},"in_reply_to":"2100cd32_07c5a675","updated":"2026-03-10 15:00:21.000000000","message":"Thank you! Sean, miss from my side.","commit_id":"40c1ea6d9f61b2d65d11860092c610b5b5d41465"}]}
