)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"13c7e9148da5b7a67ef46bf234cbc660dc56cc94","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"When snpashot are soft deleted i.e. they are renamed if delete fails,"},{"line_number":10,"context_line":"sometime we face issue in rename operation. This is due to busy"},{"line_number":11,"context_line":"snapshots. So stop clone split, perform rename and start clone split"},{"line_number":12,"context_line":"again."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Closes-bug: #2025641"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":12,"id":"4f65359c_b6c910f2","line":11,"range":{"start_line":11,"start_character":11,"end_line":11,"end_character":30},"updated":"2024-05-20 12:47:13.000000000","message":"You mentioned that you are now stopping the split clone, but I couldn\u0027t find this in the code added. I can see split_start after the renaming.","commit_id":"2f952875e1f2761072db445d9c4599b5b4da14cb"},{"author":{"_account_id":32919,"name":"kiran pawar","display_name":"Kiran Pawar","email":"kinpaa@gmail.com","username":"kpdev"},"change_message_id":"846e909885dba5a9162abc83bef86ef57bf1d1f7","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"When snpashot are soft deleted i.e. they are renamed if delete fails,"},{"line_number":10,"context_line":"sometime we face issue in rename operation. This is due to busy"},{"line_number":11,"context_line":"snapshots. So stop clone split, perform rename and start clone split"},{"line_number":12,"context_line":"again."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Closes-bug: #2025641"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":12,"id":"74842e96_f7cc4b2d","line":11,"range":{"start_line":11,"start_character":11,"end_line":11,"end_character":30},"in_reply_to":"4f65359c_b6c910f2","updated":"2024-05-20 13:08:20.000000000","message":"previously, we used start clone split and then call soft delete. This was failing as clone split ongoing. Now, we call soft delete and then call start clone split. So commit message is written as\nstop the clone split, perform rename and start clone split.\nI will reword to remove stop clone split as this is confusing.","commit_id":"2f952875e1f2761072db445d9c4599b5b4da14cb"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"d1e0332e418a7c1788492bf029e753ac84997c8d","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"When snpashot are soft deleted i.e. they are renamed if delete fails,"},{"line_number":10,"context_line":"sometime we face issue in rename operation. This is due to busy"},{"line_number":11,"context_line":"snapshots. So stop clone split, perform rename and start clone split"},{"line_number":12,"context_line":"again."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Closes-bug: #2025641"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":12,"id":"3ef45d69_a973d727","line":11,"range":{"start_line":11,"start_character":11,"end_line":11,"end_character":30},"in_reply_to":"74842e96_f7cc4b2d","updated":"2024-05-20 16:42:41.000000000","message":"good call out; ty for the question Douglas; and for the fix Kiran!","commit_id":"2f952875e1f2761072db445d9c4599b5b4da14cb"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":32919,"name":"kiran pawar","display_name":"Kiran Pawar","email":"kinpaa@gmail.com","username":"kpdev"},"change_message_id":"a1c9eac71f0038c42930b3681c329a4fa6588239","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"9ac83885_fd807b26","updated":"2023-12-05 11:03:12.000000000","message":"After dependent PR https://review.opendev.org/c/openstack/manila/+/888927 merged, rebased this commit.","commit_id":"bfee54fdce4009f5ff59b4af561caf2e8271d271"},{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"cbaee36d5169bc76da470be6ec8b4b7912e1d73b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"02b98dd9_25fc8c05","updated":"2023-12-20 04:50:17.000000000","message":"Thank you! Kiran, Without checking error condition will results in unnecessary and misleading errors in LOG. Can you check \"ESNAPSHOTNOTALLOWED\" may be relevant to your case. Also, can you take care of REST part.","commit_id":"f97d6413ed53fef11553b9208dd21a7ad7b59071"},{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"3846f8a79be5b7b01009dd98870b7f9ee422748f","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":7,"id":"dcb58694_fec852b8","updated":"2023-12-15 12:03:13.000000000","message":"Thank you! Kiran. Can you check below comment \n\nNaAPI error is generic error and ZAPI documentation lists 15 different errors possible during snapshot-delete. How can code concludes that clone is causing snap delete issue.","commit_id":"f97d6413ed53fef11553b9208dd21a7ad7b59071"},{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"e9a4fc1f73ba5db60d6fe087c245f5c3acd6f845","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":7,"id":"c204b047_f7949729","updated":"2023-12-20 04:50:37.000000000","message":"Thank you! kiran,\nwithout checking error condition will results in unnecessary and misleading errors in LOG. Can you check \"ESNAPSHOTNOTALLOWED\" may be relevant to your case. Also, can you take care of REST part.","commit_id":"f97d6413ed53fef11553b9208dd21a7ad7b59071"},{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"86d6d50454653b260ef510d95ed0dca0ba60aa9d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"36aabb6c_6fa20580","updated":"2023-12-20 04:49:41.000000000","message":"without checking error condition will results in unnecessary and misleading errors in LOG. Can you check \"ESNAPSHOTNOTALLOWED\" may be relevant to your case. Also, can you take care of REST part.","commit_id":"f97d6413ed53fef11553b9208dd21a7ad7b59071"},{"author":{"_account_id":32919,"name":"kiran pawar","display_name":"Kiran Pawar","email":"kinpaa@gmail.com","username":"kpdev"},"change_message_id":"cfa4327119bddf13206cf7d4905023f52b2f8c29","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"11fc4c25_fcd46731","in_reply_to":"c204b047_f7949729","updated":"2023-12-22 15:51:05.000000000","message":"no need to handle ESNAPSHOTNOTALLOWED in this case. Added REST function for soft delete and test case.","commit_id":"f97d6413ed53fef11553b9208dd21a7ad7b59071"},{"author":{"_account_id":32919,"name":"kiran pawar","display_name":"Kiran Pawar","email":"kinpaa@gmail.com","username":"kpdev"},"change_message_id":"cfa4327119bddf13206cf7d4905023f52b2f8c29","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"9042561a_69144d3e","in_reply_to":"dcb58694_fec852b8","updated":"2023-12-22 15:51:05.000000000","message":"we call soft delete only in case locked by clone.","commit_id":"f97d6413ed53fef11553b9208dd21a7ad7b59071"},{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"887d995138e81fa3fc3085c124f8fc10422c1c55","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"6e0fb44f_83deb8d9","updated":"2024-01-04 05:57:11.000000000","message":"LGTM","commit_id":"a17570b1c275364fbaa592054677bd603d17fc4b"},{"author":{"_account_id":32919,"name":"kiran pawar","display_name":"Kiran Pawar","email":"kinpaa@gmail.com","username":"kpdev"},"change_message_id":"07a25ac83a57a66d4d8de53ba5dbf5bf56ecf98c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"17f56199_e293ca0e","updated":"2024-01-10 14:45:49.000000000","message":"re-run NetApp-CI","commit_id":"a17570b1c275364fbaa592054677bd603d17fc4b"},{"author":{"_account_id":36625,"name":"Rick Liu","email":"rick.liu@dell.com"},"change_message_id":"3987c813f5a5a041509c8f8882adbfb94caf348a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"2b1dfee9_238c922b","updated":"2023-12-25 08:01:48.000000000","message":"run-DellEMC PowerStore CI","commit_id":"a17570b1c275364fbaa592054677bd603d17fc4b"},{"author":{"_account_id":32919,"name":"kiran pawar","display_name":"Kiran Pawar","email":"kinpaa@gmail.com","username":"kpdev"},"change_message_id":"896a4d323a1aadcc18f8ac23406b62c6876c5013","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"287541a5_375dc2ce","updated":"2024-03-04 09:20:17.000000000","message":"rebased on master. NetApp engineers need to review and approve this to get merged in 2024.1","commit_id":"1a1a8a53feb11ccb1a23feced0135f210fa11816"},{"author":{"_account_id":36178,"name":"Saravanan Manickam","display_name":"msaravan","email":"manicsaran@gmail.com","username":"msaravan"},"change_message_id":"56b1040294343edf4e95711166b745f6e895e673","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":10,"id":"34948f6e_63461db3","updated":"2024-03-05 04:24:41.000000000","message":"run-NetApp CI","commit_id":"ed279ee000918b4d5eaee201f8400755f4542579"},{"author":{"_account_id":36178,"name":"Saravanan Manickam","display_name":"msaravan","email":"manicsaran@gmail.com","username":"msaravan"},"change_message_id":"601dfacf9bbc7b4acfce32d7b5d82afae1e5fa4f","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":10,"id":"886f8c2d_8266a5ad","updated":"2024-03-04 18:09:50.000000000","message":"run-NetApp CI","commit_id":"ed279ee000918b4d5eaee201f8400755f4542579"},{"author":{"_account_id":36178,"name":"Saravanan Manickam","display_name":"msaravan","email":"manicsaran@gmail.com","username":"msaravan"},"change_message_id":"8e47de52caebc54715217664d2ea25105fe1b375","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":10,"id":"e067b94c_7d433b9d","updated":"2024-03-04 17:13:26.000000000","message":"run-NetApp CI","commit_id":"ed279ee000918b4d5eaee201f8400755f4542579"},{"author":{"_account_id":32919,"name":"kiran pawar","display_name":"Kiran Pawar","email":"kinpaa@gmail.com","username":"kpdev"},"change_message_id":"6c7bc48f721e35e323dc82f5bddb27ce8053c82b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"274289d9_477ecd51","in_reply_to":"34948f6e_63461db3","updated":"2024-05-03 11:01:49.000000000","message":"Done","commit_id":"ed279ee000918b4d5eaee201f8400755f4542579"},{"author":{"_account_id":32919,"name":"kiran pawar","display_name":"Kiran Pawar","email":"kinpaa@gmail.com","username":"kpdev"},"change_message_id":"6c7bc48f721e35e323dc82f5bddb27ce8053c82b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"7fd7fcd0_ff12aab5","in_reply_to":"886f8c2d_8266a5ad","updated":"2024-05-03 11:01:49.000000000","message":"Done","commit_id":"ed279ee000918b4d5eaee201f8400755f4542579"},{"author":{"_account_id":32919,"name":"kiran pawar","display_name":"Kiran Pawar","email":"kinpaa@gmail.com","username":"kpdev"},"change_message_id":"6c7bc48f721e35e323dc82f5bddb27ce8053c82b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"9b7ace32_1390244a","in_reply_to":"e067b94c_7d433b9d","updated":"2024-05-03 11:01:49.000000000","message":"Done","commit_id":"ed279ee000918b4d5eaee201f8400755f4542579"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"afbab8633ac3dbeec3da127529bf073a95e42732","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"28419355_b69b9bb2","updated":"2024-05-02 15:44:31.000000000","message":"Hi Kiran; thanks for working on this.. can you please add a bug fix release note?","commit_id":"eab64cae67227bb5a51aacffc4bfeb044f266e74"},{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"3f7983caaf84ab359162fae0ce5c285bae99b0d2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"2265acd7_7f2cac62","updated":"2024-05-07 06:13:05.000000000","message":"LGTM","commit_id":"2f952875e1f2761072db445d9c4599b5b4da14cb"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"6391c4455529854425604f0f75d80caf48c66602","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"b2f570be_0d1b448c","updated":"2024-05-03 20:41:48.000000000","message":"LGTM; NetApp CI is hosed; if NetApp folks can confirm this patch works despite, we can get this in..","commit_id":"2f952875e1f2761072db445d9c4599b5b4da14cb"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"a896f9aa5d747902b501e12da0beb0d11f96945b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"3611d125_39814c35","updated":"2024-05-22 22:35:19.000000000","message":"Does this change have to be backported to older stable branches? If yes, could you please go ahead and propose a backport?","commit_id":"7bcf51e962d68c6344662b9ccdc7c63dd2c847b6"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"6edeb0b9b437edddf7ea85b5a29c83bdd3fd9335","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"2ead4c0a_37d7b411","updated":"2024-05-21 17:57:39.000000000","message":"LGTM. It would be better if we could get NetApp CI testing this change, but it seems that it is still hosed.","commit_id":"7bcf51e962d68c6344662b9ccdc7c63dd2c847b6"},{"author":{"_account_id":18816,"name":"Maurice Escher","display_name":"carthaca","email":"maurice.escher@sap.com","username":"mapocace"},"change_message_id":"a0f7ca3edc4a9aa7eae640b62feaaed5af9aade7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"169c3f0e_4606a89b","updated":"2024-05-29 11:40:17.000000000","message":"Sorry, I\u0027m late - can you have a look at my inline comment and if needed work on a followup?","commit_id":"7bcf51e962d68c6344662b9ccdc7c63dd2c847b6"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"7a9af551f5a857dab5204a125c07e037e85ca4ec","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"da2754bd_a6495bdb","in_reply_to":"2ead4c0a_37d7b411","updated":"2024-05-21 20:36:15.000000000","message":"yeah :(\n\nif there are bugs piling up, hope the NetApp folks can address them as soon as their CI starts working","commit_id":"7bcf51e962d68c6344662b9ccdc7c63dd2c847b6"}],"manila/share/drivers/netapp/dataontap/client/client_cmode.py":[{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"3846f8a79be5b7b01009dd98870b7f9ee422748f","unresolved":true,"context_lines":[{"line_number":3508,"context_line":"            msg_args \u003d {\u0027snapshot\u0027: snapshot_name, \u0027volume\u0027: volume_name}"},{"line_number":3509,"context_line":"            LOG.info(msg, msg_args)"},{"line_number":3510,"context_line":""},{"line_number":3511,"context_line":"            # Snapshots are locked by clone(s), so split the clone(s)"},{"line_number":3512,"context_line":"            snapshot_children \u003d self.get_clone_children_for_snapshot("},{"line_number":3513,"context_line":"                volume_name, snapshot_name)"},{"line_number":3514,"context_line":"            for snapshot_child in snapshot_children:"}],"source_content_type":"text/x-python","patch_set":7,"id":"d3ab7e3f_612183ab","line":3511,"updated":"2023-12-15 12:03:13.000000000","message":"NaAPI error is generic error and ZAPI documentation lists 15 different errors possible during snapshot-delete. How can code concludes that clone is causing snap delete issue.","commit_id":"f97d6413ed53fef11553b9208dd21a7ad7b59071"},{"author":{"_account_id":32919,"name":"kiran pawar","display_name":"Kiran Pawar","email":"kinpaa@gmail.com","username":"kpdev"},"change_message_id":"cfa4327119bddf13206cf7d4905023f52b2f8c29","unresolved":false,"context_lines":[{"line_number":3508,"context_line":"            msg_args \u003d {\u0027snapshot\u0027: snapshot_name, \u0027volume\u0027: volume_name}"},{"line_number":3509,"context_line":"            LOG.info(msg, msg_args)"},{"line_number":3510,"context_line":""},{"line_number":3511,"context_line":"            # Snapshots are locked by clone(s), so split the clone(s)"},{"line_number":3512,"context_line":"            snapshot_children \u003d self.get_clone_children_for_snapshot("},{"line_number":3513,"context_line":"                volume_name, snapshot_name)"},{"line_number":3514,"context_line":"            for snapshot_child in snapshot_children:"}],"source_content_type":"text/x-python","patch_set":7,"id":"c67db5fb_c5abd1b7","line":3511,"in_reply_to":"2adc41a4_a72c4fbb","updated":"2023-12-22 15:51:05.000000000","message":"Done","commit_id":"f97d6413ed53fef11553b9208dd21a7ad7b59071"},{"author":{"_account_id":32919,"name":"kiran pawar","display_name":"Kiran Pawar","email":"kinpaa@gmail.com","username":"kpdev"},"change_message_id":"f276db08323fb374d4adec336d0fc0295bef933b","unresolved":true,"context_lines":[{"line_number":3508,"context_line":"            msg_args \u003d {\u0027snapshot\u0027: snapshot_name, \u0027volume\u0027: volume_name}"},{"line_number":3509,"context_line":"            LOG.info(msg, msg_args)"},{"line_number":3510,"context_line":""},{"line_number":3511,"context_line":"            # Snapshots are locked by clone(s), so split the clone(s)"},{"line_number":3512,"context_line":"            snapshot_children \u003d self.get_clone_children_for_snapshot("},{"line_number":3513,"context_line":"                volume_name, snapshot_name)"},{"line_number":3514,"context_line":"            for snapshot_child in snapshot_children:"}],"source_content_type":"text/x-python","patch_set":7,"id":"2adc41a4_a72c4fbb","line":3511,"in_reply_to":"d3ab7e3f_612183ab","updated":"2023-12-15 12:14:51.000000000","message":"we call soft delete only in case locked by clone.","commit_id":"f97d6413ed53fef11553b9208dd21a7ad7b59071"},{"author":{"_account_id":18816,"name":"Maurice Escher","display_name":"carthaca","email":"maurice.escher@sap.com","username":"mapocace"},"change_message_id":"a0f7ca3edc4a9aa7eae640b62feaaed5af9aade7","unresolved":true,"context_lines":[{"line_number":3524,"context_line":""},{"line_number":3525,"context_line":"            # Snapshots are locked by clone(s), so split the clone(s)"},{"line_number":3526,"context_line":"            snapshot_children \u003d self.get_clone_children_for_snapshot("},{"line_number":3527,"context_line":"                volume_name, snapshot_name)"},{"line_number":3528,"context_line":"            for snapshot_child in snapshot_children:"},{"line_number":3529,"context_line":"                self.volume_clone_split_start(snapshot_child[\u0027name\u0027])"},{"line_number":3530,"context_line":""}],"source_content_type":"text/x-python","patch_set":13,"id":"9e6a9749_d1b0e0b6","line":3527,"range":{"start_line":3527,"start_character":29,"end_line":3527,"end_character":42},"updated":"2024-05-29 11:40:17.000000000","message":"We just renamed the snapshot by adding a prefix. Don\u0027t we have to look for the children of the newly named snapshot?\n\n```\n            snapshot_children \u003d self.get_clone_children_for_snapshot(\n                volume_name, DELETED_PREFIX + snapshot_name)\n```","commit_id":"7bcf51e962d68c6344662b9ccdc7c63dd2c847b6"},{"author":{"_account_id":32919,"name":"kiran pawar","display_name":"Kiran Pawar","email":"kinpaa@gmail.com","username":"kpdev"},"change_message_id":"e57309c56eb7c5df0101cde6868eaaf0d17bfd47","unresolved":false,"context_lines":[{"line_number":3524,"context_line":""},{"line_number":3525,"context_line":"            # Snapshots are locked by clone(s), so split the clone(s)"},{"line_number":3526,"context_line":"            snapshot_children \u003d self.get_clone_children_for_snapshot("},{"line_number":3527,"context_line":"                volume_name, snapshot_name)"},{"line_number":3528,"context_line":"            for snapshot_child in snapshot_children:"},{"line_number":3529,"context_line":"                self.volume_clone_split_start(snapshot_child[\u0027name\u0027])"},{"line_number":3530,"context_line":""}],"source_content_type":"text/x-python","patch_set":13,"id":"00d3d4df_15ff76b1","line":3527,"range":{"start_line":3527,"start_character":29,"end_line":3527,"end_character":42},"in_reply_to":"09cb4e12_f8ca565b","updated":"2024-06-05 10:16:43.000000000","message":"Created follow up patch, since release notes exist in this patch, not added there.\n\nhttps://review.opendev.org/c/openstack/manila/+/921369","commit_id":"7bcf51e962d68c6344662b9ccdc7c63dd2c847b6"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"c3f21af4aeb91851553d78d6aef8fa95d7d9b24e","unresolved":true,"context_lines":[{"line_number":3524,"context_line":""},{"line_number":3525,"context_line":"            # Snapshots are locked by clone(s), so split the clone(s)"},{"line_number":3526,"context_line":"            snapshot_children \u003d self.get_clone_children_for_snapshot("},{"line_number":3527,"context_line":"                volume_name, snapshot_name)"},{"line_number":3528,"context_line":"            for snapshot_child in snapshot_children:"},{"line_number":3529,"context_line":"                self.volume_clone_split_start(snapshot_child[\u0027name\u0027])"},{"line_number":3530,"context_line":""}],"source_content_type":"text/x-python","patch_set":13,"id":"09cb4e12_f8ca565b","line":3527,"range":{"start_line":3527,"start_character":29,"end_line":3527,"end_character":42},"in_reply_to":"9e6a9749_d1b0e0b6","updated":"2024-06-05 05:13:52.000000000","message":"@kinpaa@gmail.com","commit_id":"7bcf51e962d68c6344662b9ccdc7c63dd2c847b6"}],"manila/share/drivers/netapp/dataontap/client/client_cmode_rest.py":[{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"8141803c7971b53e9a87409be90d75879bedf026","unresolved":true,"context_lines":[{"line_number":2048,"context_line":"            LOG.info(msg, msg_args)"},{"line_number":2049,"context_line":""},{"line_number":2050,"context_line":"            # Snapshots are locked by clone(s), so split the clone(s)"},{"line_number":2051,"context_line":"            snapshot_children \u003d self.get_clone_children_for_snapshot("},{"line_number":2052,"context_line":"                volume_name, snapshot_name)"},{"line_number":2053,"context_line":"            for snapshot_child in snapshot_children:"},{"line_number":2054,"context_line":"                self.volume_clone_split_start(snapshot_child[\u0027name\u0027])"}],"source_content_type":"text/x-python","patch_set":12,"id":"b3527d9d_ac1e5152","line":2051,"updated":"2024-05-07 08:07:37.000000000","message":"NIT: Same set of statements are repeated in lib_base. Check this can be optimized.","commit_id":"2f952875e1f2761072db445d9c4599b5b4da14cb"}]}