)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"cd0cde85b8480931f0f2f29a8f73259c2c50ce28","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"877fa959_29aca8ce","updated":"2023-06-13 09:08:49.000000000","message":"Should it be picked up again? There is a  new proposed  may overlap with this, but this looks a bit bigger in scope:\nhttps://review.opendev.org/c/openstack/tempest/+/885841","commit_id":"db7e28a4d18c4a1e564ac25ff9650c464c8e0b5b"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"a4a6768d0f04c43a51ec0f813e630bfb436c0e56","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"2af7807f_a1aece22","updated":"2024-03-11 12:45:00.000000000","message":"recheck","commit_id":"db7e28a4d18c4a1e564ac25ff9650c464c8e0b5b"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"3806fc88a7972706b551be6f6c10b2d4dd5d7e25","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"2d831e0e_de459e79","updated":"2024-09-20 12:11:59.000000000","message":"recheck","commit_id":"db7e28a4d18c4a1e564ac25ff9650c464c8e0b5b"}],"cinder_tempest_plugin/api/volume/test_volume_backup.py":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"c22344815d7d545ae462bdc353e43089f8cfaae8","unresolved":false,"context_lines":[{"line_number":131,"context_line":"        self.assertTrue(is_incremental)"},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"        # Create a second full backup"},{"line_number":134,"context_line":"        # should have a diff basen_name"},{"line_number":135,"context_line":"        full_backup2 \u003d self.create_backup(volume_id\u003dvolume[\u0027id\u0027])"},{"line_number":136,"context_line":"        is_incremental \u003d self.backups_client.show_backup("},{"line_number":137,"context_line":"            full_backup2[\u0027id\u0027])[\u0027backup\u0027][\u0027base_name\u0027]"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fce034c_8a92b5f5","line":134,"range":{"start_line":134,"start_character":29,"end_line":134,"end_character":34},"updated":"2019-04-16 04:00:06.000000000","message":"base","commit_id":"3162a08c64e6f6f42728febf1d7af9f8ddb27453"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"c22344815d7d545ae462bdc353e43089f8cfaae8","unresolved":false,"context_lines":[{"line_number":134,"context_line":"        # should have a diff basen_name"},{"line_number":135,"context_line":"        full_backup2 \u003d self.create_backup(volume_id\u003dvolume[\u0027id\u0027])"},{"line_number":136,"context_line":"        is_incremental \u003d self.backups_client.show_backup("},{"line_number":137,"context_line":"            full_backup2[\u0027id\u0027])[\u0027backup\u0027][\u0027base_name\u0027]"},{"line_number":138,"context_line":"        self.assertFalse(is_incremental)"},{"line_number":139,"context_line":""},{"line_number":140,"context_line":"        # Create the first incremental backup from the second full"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fce034c_45ffde00","line":137,"range":{"start_line":137,"start_character":42,"end_line":137,"end_character":53},"updated":"2019-04-16 04:00:06.000000000","message":"\u0027is_incremental\u0027 ?","commit_id":"3162a08c64e6f6f42728febf1d7af9f8ddb27453"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"c22344815d7d545ae462bdc353e43089f8cfaae8","unresolved":false,"context_lines":[{"line_number":165,"context_line":"        with self.assertRaises(Exception.InvalidBackup):"},{"line_number":166,"context_line":"            self.backups_client.delete_backup(full_backup1[\u0027id\u0027])"},{"line_number":167,"context_line":""},{"line_number":168,"context_line":"        self.backups_client.delete_backup(backup_incremental3[\u0027id\u0027])"},{"line_number":169,"context_line":"        self.backups_client.delete_backup(full_backup1[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fce034c_0509d62a","line":168,"range":{"start_line":168,"start_character":60,"end_line":168,"end_character":61},"updated":"2019-04-16 04:00:06.000000000","message":"4","commit_id":"3162a08c64e6f6f42728febf1d7af9f8ddb27453"},{"author":{"_account_id":31055,"name":"Anastasiya Zhyrkevich","email":"Anastasiya.Zhyrkevich@gmail.com","username":"anastzhyr"},"change_message_id":"06331ab5504dfd479f779b91937b3ad606203fd9","unresolved":false,"context_lines":[{"line_number":165,"context_line":"        with self.assertRaises(Exception.InvalidBackup):"},{"line_number":166,"context_line":"            self.backups_client.delete_backup(full_backup1[\u0027id\u0027])"},{"line_number":167,"context_line":""},{"line_number":168,"context_line":"        self.backups_client.delete_backup(backup_incremental4[\u0027id\u0027])"},{"line_number":169,"context_line":"        self.backups_client.delete_backup(full_backup1[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":5,"id":"3fa7e38b_b21f1959","line":168,"updated":"2020-02-02 17:06:31.000000000","message":"Should not we delete backup_incremental1 as well here?","commit_id":"2074162fb3fd6c5193179c18773a84d9c5b1f247"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"222049cc23d3022e0e665452a7876dac1db8573c","unresolved":false,"context_lines":[{"line_number":165,"context_line":"        with self.assertRaises(Exception.InvalidBackup):"},{"line_number":166,"context_line":"            self.backups_client.delete_backup(full_backup1[\u0027id\u0027])"},{"line_number":167,"context_line":""},{"line_number":168,"context_line":"        self.backups_client.delete_backup(backup_incremental4[\u0027id\u0027])"},{"line_number":169,"context_line":"        self.backups_client.delete_backup(full_backup1[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":5,"id":"3fa7e38b_3d13f56a","line":168,"in_reply_to":"3fa7e38b_b21f1959","updated":"2020-02-03 15:01:10.000000000","message":"Yes, nice catch. \n\nI need to remove the backup_incremental1 before deleting full_backup1. \n\nAs this test isn\u0027t running in CI, Exception.InvalidBackup is not been raised.","commit_id":"2074162fb3fd6c5193179c18773a84d9c5b1f247"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"f4c60185a1e127bf2d2a52e88292e65e48ab1891","unresolved":false,"context_lines":[{"line_number":186,"context_line":""},{"line_number":187,"context_line":"        # Delete second full and third incremental"},{"line_number":188,"context_line":"        self.backups_client.delete_backup(backup_incremental3[\u0027id\u0027])"},{"line_number":189,"context_line":"        self.admin_backups_client.wait_for_resource_deletion("},{"line_number":190,"context_line":"            backup_incremental3[\u0027id\u0027])"},{"line_number":191,"context_line":""},{"line_number":192,"context_line":"        self.backups_client.delete_backup(full_backup2[\u0027id\u0027])"},{"line_number":193,"context_line":"        self.admin_backups_client.wait_for_resource_deletion("}],"source_content_type":"text/x-python","patch_set":10,"id":"bf51134e_1291729a","line":190,"range":{"start_line":189,"start_character":8,"end_line":190,"end_character":38},"updated":"2020-07-06 13:08:10.000000000","message":"AttributeError: \u0027VolumesBackupsTest\u0027 object has no attribute \u0027admin_backups_client\u0027\n\nZuul fails because admin_backups_client should be removed or replaced.","commit_id":"b2e944456ec51384cbd9da48e9dddc9e02ec0e42"}]}