)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":30615,"name":"Tushar Trambak Gite","email":"tushargite96@gmail.com","username":"tushargite96"},"change_message_id":"ebb05c34a3e6ea1ef9c56442dd6c8309a89a6840","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"47e12f42_a75aef34","updated":"2022-01-20 06:21:19.000000000","message":"LGTM","commit_id":"9bcf6229f7037681caa034b0360cc571cbec9a8d"},{"author":{"_account_id":33431,"name":"Fábio Oliveira","email":"fabioaurelio1269@gmail.com","username":"fabiooliveira1"},"change_message_id":"4f5dc3077b87c78fe86e5521aba18de83645086a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"bb1cf088_1be175f8","updated":"2022-01-20 14:45:25.000000000","message":"LGTM. Thank you, Felipe","commit_id":"9bcf6229f7037681caa034b0360cc571cbec9a8d"},{"author":{"_account_id":33648,"name":"Nahim Alves de Souza","email":"nahimsouza@outlook.com","username":"nahimsouza"},"change_message_id":"202fba4b45700cf803af619fe2e9d026f030acf7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d0856b11_5221a8d1","updated":"2022-01-13 14:38:45.000000000","message":"The code looks good to me, I also tested fix and it worked as expected. Thanks Felipe.","commit_id":"9bcf6229f7037681caa034b0360cc571cbec9a8d"},{"author":{"_account_id":33648,"name":"Nahim Alves de Souza","email":"nahimsouza@outlook.com","username":"nahimsouza"},"change_message_id":"5f5a2f65cbbeb98196ad6556455cfe47e13ca907","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1d0a99b0_b19e031e","updated":"2022-09-05 11:58:52.000000000","message":"run-NetApp CI","commit_id":"9bcf6229f7037681caa034b0360cc571cbec9a8d"},{"author":{"_account_id":33648,"name":"Nahim Alves de Souza","email":"nahimsouza@outlook.com","username":"nahimsouza"},"change_message_id":"82ccb759e49600a846050c238ef15c8686a8ff9a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"2d6c9636_2eb3c88c","updated":"2022-02-28 11:40:44.000000000","message":"run-NetApp CI","commit_id":"9bcf6229f7037681caa034b0360cc571cbec9a8d"},{"author":{"_account_id":33648,"name":"Nahim Alves de Souza","email":"nahimsouza@outlook.com","username":"nahimsouza"},"change_message_id":"646396195ae440e2eaf4bc08f1f7fb070c89edbe","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"46256846_b5d5fac0","updated":"2022-02-25 19:17:22.000000000","message":"run-NetApp CI","commit_id":"9bcf6229f7037681caa034b0360cc571cbec9a8d"},{"author":{"_account_id":31721,"name":"Felipe Rodrigues","email":"felipefuty01@gmail.com","username":"felipefutty"},"change_message_id":"23191fffa9e1feea891f6618e24465694049b62d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4b0da8c6_9a6ff1c8","updated":"2022-01-20 12:06:10.000000000","message":"run-NetApp CI","commit_id":"9bcf6229f7037681caa034b0360cc571cbec9a8d"},{"author":{"_account_id":33648,"name":"Nahim Alves de Souza","email":"nahimsouza@outlook.com","username":"nahimsouza"},"change_message_id":"0591006e93536ea99cf4aeca4fcf6a6a56f939c7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"29988c82_19d4f406","updated":"2023-06-06 18:51:44.000000000","message":"Hi, Eric, thanks for the review, I uploaded a new patch set with the changes.","commit_id":"b1e5c6686e25b3455f01151a01cf0bd077590cad"},{"author":{"_account_id":33648,"name":"Nahim Alves de Souza","email":"nahimsouza@outlook.com","username":"nahimsouza"},"change_message_id":"5b60e4752a914ff007e63e538fc8ecf99185b159","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"85c6b4b4_36cf3edc","updated":"2023-06-07 20:28:31.000000000","message":"recheck - failure not related to the patch","commit_id":"b35128078ca580c3bccf21f25ad302d81cd9bf20"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"59f4fe59cc9fe2e4c5f759a24b033b0fa26f060e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"3965226f_bce4f7c0","updated":"2023-07-18 10:33:19.000000000","message":"Code change looks good. Zuul and NetApp-CI have passed. So +1","commit_id":"0dc8d75229fec860c6fd82ecd772c242839e4e87"},{"author":{"_account_id":33301,"name":"Caique Mello","email":"caiquemellosbo@gmail.com","username":"MelloCaique"},"change_message_id":"028bfe04194b4fb669cee1bd46e68072db903cab","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"c735df21_62ecf3ec","updated":"2023-07-26 17:50:03.000000000","message":"LGTM thank you Felipe and Nahim.","commit_id":"0dc8d75229fec860c6fd82ecd772c242839e4e87"},{"author":{"_account_id":35063,"name":"Tony Saad","email":"tony.saad@dell.com","username":"tony-saad"},"change_message_id":"348ca40e68928e8fe59d687bf85d313fa9cb6ccf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"df7e06aa_45f03622","updated":"2023-07-27 12:59:32.000000000","message":"LGTM!","commit_id":"0dc8d75229fec860c6fd82ecd772c242839e4e87"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"ae129766ffd707436e0857a7518db2d83df1f7ef","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"f4ce9563_a0db269b","updated":"2023-08-30 17:57:54.000000000","message":"NetApp CI is passing. The commit message could have been more clear. code and test changes looks good. LGTM.","commit_id":"0dc8d75229fec860c6fd82ecd772c242839e4e87"},{"author":{"_account_id":30615,"name":"Tushar Trambak Gite","email":"tushargite96@gmail.com","username":"tushargite96"},"change_message_id":"a2945518c889728c75e0b70b56bf0930125cdc37","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"476ca15b_4be9dfe8","updated":"2023-07-05 03:56:37.000000000","message":"looks good","commit_id":"0dc8d75229fec860c6fd82ecd772c242839e4e87"},{"author":{"_account_id":36178,"name":"Saravanan Manickam","display_name":"msaravan","email":"manicsaran@gmail.com","username":"msaravan"},"change_message_id":"1718aa6f45637ab94e8ecd0f61e7d5303ebd3c37","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"6b9e5446_f9cfa7c0","updated":"2023-08-31 05:43:24.000000000","message":"recheck - unrelated IPV6 related failure.","commit_id":"0dc8d75229fec860c6fd82ecd772c242839e4e87"},{"author":{"_account_id":33301,"name":"Caique Mello","email":"caiquemellosbo@gmail.com","username":"MelloCaique"},"change_message_id":"64c0e4bd2b92454afeb1b7773b77ea38b73631ab","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"0613f36a_c8cb562e","updated":"2023-08-31 12:40:34.000000000","message":"recheck tempest-integrated-storage failed not related to the patch","commit_id":"0dc8d75229fec860c6fd82ecd772c242839e4e87"},{"author":{"_account_id":33301,"name":"Caique Mello","email":"caiquemellosbo@gmail.com","username":"MelloCaique"},"change_message_id":"9b873a7a40d361d1078b8d8cfde08ae63d933035","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"0db87881_c1c717c5","updated":"2023-08-31 17:01:44.000000000","message":"recheck unrelated error in plugin-lvm-lio-barbican","commit_id":"0dc8d75229fec860c6fd82ecd772c242839e4e87"},{"author":{"_account_id":35677,"name":"Thiago José de Andrade Alvoravel","display_name":"Thiago Alvoravel","email":"thiago.andrade@fit-tecnologia.org.br","username":"thiagoalvoravel"},"change_message_id":"d3d9618850042ad1b1f939e48b1809c1d9f0b159","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"2ad9bd17_0dc98536","updated":"2023-08-31 20:29:03.000000000","message":"recheck unrelated error in plugin-lvm-lio-barbican and tempest-integrated-storage","commit_id":"0dc8d75229fec860c6fd82ecd772c242839e4e87"}],"cinder/volume/drivers/netapp/dataontap/utils/data_motion.py":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"b4410e91ea7181bdb9a4a6ede7980f40bd18a3b8","unresolved":true,"context_lines":[{"line_number":472,"context_line":"        provisioning_options.pop(\u0027volume_type\u0027, None)"},{"line_number":473,"context_line":""},{"line_number":474,"context_line":"        source_aggregate \u003d provisioning_options.pop(\u0027aggregate\u0027)"},{"line_number":475,"context_line":"        if not isinstance(source_aggregate, list):"},{"line_number":476,"context_line":"            source_aggregate \u003d [source_aggregate]"},{"line_number":477,"context_line":""},{"line_number":478,"context_line":"        aggregate_map \u003d self._get_replication_aggregate_map("}],"source_content_type":"text/x-python","patch_set":2,"id":"515e09aa_652a2966","line":475,"updated":"2023-05-24 14:57:28.000000000","message":"Why not just have get_provisioning_options_from_flexvol() return a list for \u0027aggregate\u0027?  That seems like a cleaner way to fix this.\n\nChecking types to do it here is kind of messy.","commit_id":"b1e5c6686e25b3455f01151a01cf0bd077590cad"},{"author":{"_account_id":33648,"name":"Nahim Alves de Souza","email":"nahimsouza@outlook.com","username":"nahimsouza"},"change_message_id":"0591006e93536ea99cf4aeca4fcf6a6a56f939c7","unresolved":false,"context_lines":[{"line_number":472,"context_line":"        provisioning_options.pop(\u0027volume_type\u0027, None)"},{"line_number":473,"context_line":""},{"line_number":474,"context_line":"        source_aggregate \u003d provisioning_options.pop(\u0027aggregate\u0027)"},{"line_number":475,"context_line":"        if not isinstance(source_aggregate, list):"},{"line_number":476,"context_line":"            source_aggregate \u003d [source_aggregate]"},{"line_number":477,"context_line":""},{"line_number":478,"context_line":"        aggregate_map \u003d self._get_replication_aggregate_map("}],"source_content_type":"text/x-python","patch_set":2,"id":"f4ffe634_0f6bd974","line":475,"in_reply_to":"515e09aa_652a2966","updated":"2023-06-06 18:51:44.000000000","message":"This was done to avoid a bigger number of changes in other files.\n\nHowever, your suggestion makes sense, and after analyzing the code I concluded that it\u0027s unlikely that the changes will break anything. So I changed the code and will upload the change in the next patch set.","commit_id":"b1e5c6686e25b3455f01151a01cf0bd077590cad"}]}