)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"e49b11513a3e28f61f9a6b54179804bf3b349e79","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Improvements in snapshot data integrity tests"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch addresses the comments in the snapshot data integrity tests[1]."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"[1] https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/790899"},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"62b7ca0d_53a38603","line":9,"updated":"2022-05-09 14:03:20.000000000","message":"nit: It would have been good to spell them out.","commit_id":"027295a69b9f092c2981db43338d25bb033b027c"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"03785bef4276224897dd4cba20ea2f7f092f2f57","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"92f74fd5_ef2efb54","updated":"2022-03-18 15:00:27.000000000","message":"Gorka, this addresses a few comment from yours","commit_id":"027295a69b9f092c2981db43338d25bb033b027c"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"5ef2b242bdfe903424f5e66f760c57050c7f51b9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"357e3b69_4c11a799","updated":"2022-04-27 14:41:49.000000000","message":"Gorka?","commit_id":"027295a69b9f092c2981db43338d25bb033b027c"},{"author":{"_account_id":30615,"name":"Tushar Trambak Gite","email":"tushargite96@gmail.com","username":"tushargite96"},"change_message_id":"230a000935a2670d4fa5e6721ace9e16eba55a09","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"cb29a15d_30595dc5","updated":"2022-01-10 09:32:10.000000000","message":"LGTM","commit_id":"027295a69b9f092c2981db43338d25bb033b027c"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"e49b11513a3e28f61f9a6b54179804bf3b349e79","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"83bec604_a41eaf10","updated":"2022-05-09 14:03:20.000000000","message":"Thanks for addressing my concerns.","commit_id":"027295a69b9f092c2981db43338d25bb033b027c"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"4fef55395873bea641864e90f47e49779ac03f47","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"95e29f8b_86a1e700","updated":"2021-12-30 06:29:46.000000000","message":"recheck","commit_id":"027295a69b9f092c2981db43338d25bb033b027c"}],"cinder_tempest_plugin/scenario/test_snapshots.py":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"e49b11513a3e28f61f9a6b54179804bf3b349e79","unresolved":true,"context_lines":[{"line_number":93,"context_line":"        # Detach the volume"},{"line_number":94,"context_line":"        self.nova_volume_detach(server, volume)"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"        snap_map \u003d {1: snapshot1, 2: snapshot2, 3: snapshot3}"},{"line_number":97,"context_line":"        file_map \u003d {1: file1_md5, 2: file2_md5, 3: file3_md5}"},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"        # Loop over 3 times to check the data integrity of all 3 snapshots"},{"line_number":100,"context_line":"        for i in range(1, 4):"}],"source_content_type":"text/x-python","patch_set":4,"id":"0c12ee0d_95795bf3","line":97,"range":{"start_line":96,"start_character":8,"end_line":97,"end_character":61},"updated":"2022-05-09 14:03:20.000000000","message":"nit: In my opinion these should have used a tuple instead, and then iterate through it, but it\u0027s not the end of the world.\n\n  snap_files \u003d ((snapshot1, file1_md5), (snapshot2, file2_md5), (snapshot3, file3_md5))\n\n  for i, (snap, filename) in enumerate(snap_files, 1):\n    ....","commit_id":"027295a69b9f092c2981db43338d25bb033b027c"}]}