)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"ab4425a079ad09bc82ae881e2d8dce9f97e95ab5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"45f9f8d9_1609c504","updated":"2022-05-26 20:04:41.000000000","message":"Looks good! Suggestion inline.","commit_id":"9470e9299399983505499f246acf05666660cf21"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"ee18d5cc3edc36a5c7b041bde18c77ea0a0cc712","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ad2560c5_48925b6b","updated":"2022-05-27 15:33:16.000000000","message":"lgtm and make sense. ","commit_id":"404b55cd2f07327ab2acc58295ed2fcd222e3bc2"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"70815888187c1412181f3287de343b14c68672c8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"9b70f703_aa3f6b46","updated":"2022-05-26 23:10:07.000000000","message":"recheck","commit_id":"404b55cd2f07327ab2acc58295ed2fcd222e3bc2"}],"tempest/scenario/manager.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"ab4425a079ad09bc82ae881e2d8dce9f97e95ab5","unresolved":true,"context_lines":[{"line_number":419,"context_line":""},{"line_number":420,"context_line":"        body \u003d self.backups_client.restore_backup(backup_id, **kwargs)"},{"line_number":421,"context_line":"        restore \u003d body[\u0027restore\u0027]"},{"line_number":422,"context_line":"        if not kwargs.get(\u0027volume_id\u0027, None):"},{"line_number":423,"context_line":"            self.addCleanup(self.volumes_client.delete_volume,"},{"line_number":424,"context_line":"                            restore[\u0027volume_id\u0027])"},{"line_number":425,"context_line":"        waiters.wait_for_volume_resource_status(self.backups_client,"}],"source_content_type":"text/x-python","patch_set":1,"id":"b33aac61_bdd3d179","line":422,"range":{"start_line":422,"start_character":0,"end_line":422,"end_character":45},"updated":"2022-05-26 20:04:41.000000000","message":"Your commit message explains clearly why you are doing this, but it would be a good idea to document it here too, because otherwise this looks kind of mysterious.  I\u0027d suggest either adding a comment, or you could try to make the code self-documenting like this:\n\n    using_pre_existing_volume \u003d kwargs.get(\u0027volume_id\u0027, False)\n    if not using_pre_existing_volume:\n        self.addCleanup(self.volumes_client.delete_volume,\n                                restore[\u0027volume_id\u0027])","commit_id":"9470e9299399983505499f246acf05666660cf21"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"8f359ea7a516e737368b66f7d8014e3387356dd2","unresolved":false,"context_lines":[{"line_number":419,"context_line":""},{"line_number":420,"context_line":"        body \u003d self.backups_client.restore_backup(backup_id, **kwargs)"},{"line_number":421,"context_line":"        restore \u003d body[\u0027restore\u0027]"},{"line_number":422,"context_line":"        if not kwargs.get(\u0027volume_id\u0027, None):"},{"line_number":423,"context_line":"            self.addCleanup(self.volumes_client.delete_volume,"},{"line_number":424,"context_line":"                            restore[\u0027volume_id\u0027])"},{"line_number":425,"context_line":"        waiters.wait_for_volume_resource_status(self.backups_client,"}],"source_content_type":"text/x-python","patch_set":1,"id":"1735faf2_2839447e","line":422,"range":{"start_line":422,"start_character":0,"end_line":422,"end_character":45},"in_reply_to":"b33aac61_bdd3d179","updated":"2022-05-26 21:04:05.000000000","message":"awesome! I\u0027ll update!","commit_id":"9470e9299399983505499f246acf05666660cf21"}]}