)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":33502,"name":"Oleg","display_name":"Oleg Nesterenkov","email":"Oleg_Nesterenkov@Dell.com","username":"olegnest"},"change_message_id":"0899eb06e24799c31f8bf63689370c768a4f0c48","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"bcae9ab9_106c90a3","updated":"2022-08-08 00:08:45.000000000","message":"recheck","commit_id":"89c6db1d570cf587227e5cb1584e8cf47b21b778"},{"author":{"_account_id":33807,"name":"Jacob Wang","email":"jacob_wang1@dell.com","username":"jacob0522"},"change_message_id":"b844602345ca0ec3d509b31bbbe5d70a80301568","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c82464b7_ef3a0227","updated":"2022-08-08 13:28:03.000000000","message":"run-DellEMC PowerStore CI","commit_id":"e21bb2ab8740ef3e3fc59186d68a3a3e6d35d67c"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"c05074bdd2d13089c44b6a62cedbd0eb505579ab","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"4931d4db_454b6872","updated":"2022-08-19 15:59:40.000000000","message":"-DellEMC PowerStore CI passed","commit_id":"ee8e87cf1770eb63b3ccccacb1724468784b3bf1"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"323367ddc5ed3c9de35c5658bb200d5d27997038","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"e9f72dad_f742d764","updated":"2022-09-05 10:52:46.000000000","message":"Code looks good, and it\u0027s a very good idea to use the new connection info format, but I\u0027m surprised that not a single unit test needs to be updated.\n\nPlease, at least add 1 unit tests for the _get_connection_properties method of the adapter.","commit_id":"ee8e87cf1770eb63b3ccccacb1724468784b3bf1"},{"author":{"_account_id":30615,"name":"Tushar Trambak Gite","email":"tushargite96@gmail.com","username":"tushargite96"},"change_message_id":"688928a70b39aec4568fa73a15535a3d351ccb26","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"e11821e4_a1de7631","updated":"2022-08-23 09:13:56.000000000","message":"LGTM","commit_id":"ee8e87cf1770eb63b3ccccacb1724468784b3bf1"},{"author":{"_account_id":34988,"name":"Andrey Pakhomov","email":"a.pakhomov@dell.com"},"change_message_id":"49756cf718d1886a3174a5c2970a91ba22d9dd27","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"21bdd312_3b43f489","updated":"2022-08-11 06:57:43.000000000","message":"LGTM, PowerStore CI passed","commit_id":"ee8e87cf1770eb63b3ccccacb1724468784b3bf1"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"aec5bc3e3d4ad12b50750dcaed9fcafac33bf356","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"f8152ffa_73d1eb37","updated":"2022-09-21 11:57:50.000000000","message":"Thanks Gorka for the review. LGTM as well.","commit_id":"ee8e87cf1770eb63b3ccccacb1724468784b3bf1"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"bf6a14ddf5f89fddd3cecfd6d091c0123de29a2d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"0405a83f_dd655cd5","updated":"2022-09-21 11:54:06.000000000","message":"We should always include the unit tests in the same patch, but I\u0027ll look another way this time.","commit_id":"ee8e87cf1770eb63b3ccccacb1724468784b3bf1"},{"author":{"_account_id":33502,"name":"Oleg","display_name":"Oleg Nesterenkov","email":"Oleg_Nesterenkov@Dell.com","username":"olegnest"},"change_message_id":"84a4c0fff77a22a8e883bd95f5cc4585a5bd1656","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"89982286_da8efff2","updated":"2022-08-10 15:01:17.000000000","message":"run-DellEMC PowerStore CI","commit_id":"ee8e87cf1770eb63b3ccccacb1724468784b3bf1"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"8cf4a45ab56b6e147606a3ee7f9e60a76d2ad0e7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"7b1565f1_39a38eab","in_reply_to":"e9f72dad_f742d764","updated":"2022-09-21 11:40:53.000000000","message":"Hi Gorka,\n\nSince there were no tests for the NVMEoF adapter class, I\u0027ve proposed a followup here[1] which adds tests for whole adapter class.\n\n[1] https://review.opendev.org/c/openstack/cinder/+/858678/1","commit_id":"ee8e87cf1770eb63b3ccccacb1724468784b3bf1"}]}