)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":30615,"name":"Tushar Trambak Gite","email":"tushargite96@gmail.com","username":"tushargite96"},"change_message_id":"dd88157714d1439cd0bde263afbc0ae998fe6ed5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e8ce7bfa_b3b9283f","updated":"2022-09-29 04:39:00.000000000","message":"LGTM","commit_id":"5c03c4ef6ca8f9b5288c46b487e30741b27d2a0e"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"73a6d62b5a768675e5684cac0c2f49709b1a7713","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"71bb18e2_e3aacbfc","updated":"2022-11-18 15:35:33.000000000","message":"LGTM but one question inline.","commit_id":"5c03c4ef6ca8f9b5288c46b487e30741b27d2a0e"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"6fa202b4647b4de56dd4c4853bac5ff27920072e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"2e4be6b2_a1654fdd","updated":"2022-11-18 14:27:15.000000000","message":"Looks good to me","commit_id":"5c03c4ef6ca8f9b5288c46b487e30741b27d2a0e"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"e8dad046e934b773c5362a1c3733ce6876d187ee","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"35f9c5e6_f8a139f4","updated":"2022-09-22 17:43:10.000000000","message":"recheck\n\npy310 cinder.tests.unit.volume.drivers.ibm.test_ibm_flashsystem.FlashSystemDriverTestCase.test_flashsystem_map_vdisk_to_host failed","commit_id":"5c03c4ef6ca8f9b5288c46b487e30741b27d2a0e"}],"cinder/volume/manager.py":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"73a6d62b5a768675e5684cac0c2f49709b1a7713","unresolved":true,"context_lines":[{"line_number":3252,"context_line":""},{"line_number":3253,"context_line":"    def _get_my_volumes_summary("},{"line_number":3254,"context_line":"            self,"},{"line_number":3255,"context_line":"            ctxt: context.RequestContext) -\u003e tuple:"},{"line_number":3256,"context_line":"        filters \u003d self._get_cluster_or_host_filters()"},{"line_number":3257,"context_line":"        return objects.VolumeList.get_volume_summary(ctxt, False, filters)"},{"line_number":3258,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"318fac81_169ee34a","line":3255,"range":{"start_line":3255,"start_character":45,"end_line":3255,"end_character":50},"updated":"2022-11-18 15:35:33.000000000","message":"this is correct now[1] but I\u0027m thinking why we didn\u0027t get any error when the original change was merged?\n\n[1] https://github.com/openstack/cinder/blob/3f0f49bd8e9670da3588527329f35bf5dc40948f/cinder/db/sqlalchemy/api.py#L2809","commit_id":"5c03c4ef6ca8f9b5288c46b487e30741b27d2a0e"}]}
