)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"21f76b7ac638f5a91da15f65483a69921d86d4f8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"95dccfcf_ae6afe18","updated":"2023-07-27 05:48:03.000000000","message":"We again ran into the same problem, hopefully we can check beforehand next time before working on the tests.","commit_id":"1c0a4fa26c08c77f1b6a6a6603413887e7c1d910"}],"cinder/tests/functional/api_sample_tests/test_backups_actions.py":[{"author":{"_account_id":35836,"name":"Toheeb Oyekola","display_name":"Toheeb","email":"toheeb.olawale.to23@gmail.com","username":"Toheeb"},"change_message_id":"4d6ada06c6aed1418aa511ed770f19cc2bdddb66","unresolved":true,"context_lines":[{"line_number":36,"context_line":"            self._poll_backup_while(backup_id, [\u0027creating\u0027])"},{"line_number":37,"context_line":"            return backup_id"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"    def test_backup_actions_force_delete(self):"},{"line_number":40,"context_line":"        backup_id \u003d self._create_backup()"},{"line_number":41,"context_line":"        subs \u003d {"},{"line_number":42,"context_line":"            \"backup_id\": backup_id,"},{"line_number":43,"context_line":"            \"os-force_delete\": \"True\""},{"line_number":44,"context_line":"        }"},{"line_number":45,"context_line":"        response \u003d self._do_post(\u0027backups/%s/action\u0027 % backup_id,"},{"line_number":46,"context_line":"                                 \u0027backup-force-delete-request\u0027,"},{"line_number":47,"context_line":"                                 subs)"},{"line_number":48,"context_line":"        self.assertEqual(202, response.status_code)"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"    def test_backup_actions_reset_status(self):"},{"line_number":51,"context_line":"        backup_id \u003d self._create_backup()"}],"source_content_type":"text/x-python","patch_set":3,"id":"8013dcd0_0ea83de4","line":48,"range":{"start_line":39,"start_character":4,"end_line":48,"end_character":51},"updated":"2023-07-24 11:11:13.000000000","message":"The server has either erred or is incapable of performing the requested operation.","commit_id":"1c0a4fa26c08c77f1b6a6a6603413887e7c1d910"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"21f76b7ac638f5a91da15f65483a69921d86d4f8","unresolved":true,"context_lines":[{"line_number":36,"context_line":"            self._poll_backup_while(backup_id, [\u0027creating\u0027])"},{"line_number":37,"context_line":"            return backup_id"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"    def test_backup_actions_force_delete(self):"},{"line_number":40,"context_line":"        backup_id \u003d self._create_backup()"},{"line_number":41,"context_line":"        subs \u003d {"},{"line_number":42,"context_line":"            \"backup_id\": backup_id,"},{"line_number":43,"context_line":"            \"os-force_delete\": \"True\""},{"line_number":44,"context_line":"        }"},{"line_number":45,"context_line":"        response \u003d self._do_post(\u0027backups/%s/action\u0027 % backup_id,"},{"line_number":46,"context_line":"                                 \u0027backup-force-delete-request\u0027,"},{"line_number":47,"context_line":"                                 subs)"},{"line_number":48,"context_line":"        self.assertEqual(202, response.status_code)"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"    def test_backup_actions_reset_status(self):"},{"line_number":51,"context_line":"        backup_id \u003d self._create_backup()"}],"source_content_type":"text/x-python","patch_set":3,"id":"53e74dae_dda3886f","line":48,"range":{"start_line":39,"start_character":4,"end_line":48,"end_character":51},"in_reply_to":"8013dcd0_0ea83de4","updated":"2023-07-27 05:48:03.000000000","message":"This code path includes a RPC call[1] to backup service[2] which is not available in functional tests so we can\u0027t test this\n\n[1] https://github.com/openstack/cinder/blob/26847ddeed98f4078908844ab86de3932bc49b3e/cinder/backup/api.py#L81-L82\n\n[2] https://github.com/openstack/cinder/blob/26847ddeed98f4078908844ab86de3932bc49b3e/cinder/backup/manager.py#L1133","commit_id":"1c0a4fa26c08c77f1b6a6a6603413887e7c1d910"},{"author":{"_account_id":35836,"name":"Toheeb Oyekola","display_name":"Toheeb","email":"toheeb.olawale.to23@gmail.com","username":"Toheeb"},"change_message_id":"4d6ada06c6aed1418aa511ed770f19cc2bdddb66","unresolved":true,"context_lines":[{"line_number":47,"context_line":"                                 subs)"},{"line_number":48,"context_line":"        self.assertEqual(202, response.status_code)"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"    def test_backup_actions_reset_status(self):"},{"line_number":51,"context_line":"        backup_id \u003d self._create_backup()"},{"line_number":52,"context_line":"        subs \u003d {"},{"line_number":53,"context_line":"            \"backup_id\": backup_id,"},{"line_number":54,"context_line":"            \"os-reset_status\": \"error\""},{"line_number":55,"context_line":"        }"},{"line_number":56,"context_line":"        response \u003d self._do_post(\u0027backups/%s/action\u0027 % backup_id,"},{"line_number":57,"context_line":"                                 \u0027backup-reset-status-request\u0027,"},{"line_number":58,"context_line":"                                 subs)"},{"line_number":59,"context_line":"        self.assertEqual(202, response.status_code)"},{"line_number":60,"context_line":"        self.assertEqual(\u0027\u0027, response.content)"}],"source_content_type":"text/x-python","patch_set":3,"id":"64b9fe3e_57c7c5fe","line":60,"range":{"start_line":50,"start_character":4,"end_line":60,"end_character":46},"updated":"2023-07-24 11:11:13.000000000","message":"Returned message is \"Service cinder-backup could not be found.\n\nthis is the same issue faced in this abandoned patch \nhttps://review.opendev.org/c/openstack/cinder/+/888620, view comments by Rajat for more details.","commit_id":"1c0a4fa26c08c77f1b6a6a6603413887e7c1d910"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"21f76b7ac638f5a91da15f65483a69921d86d4f8","unresolved":true,"context_lines":[{"line_number":47,"context_line":"                                 subs)"},{"line_number":48,"context_line":"        self.assertEqual(202, response.status_code)"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"    def test_backup_actions_reset_status(self):"},{"line_number":51,"context_line":"        backup_id \u003d self._create_backup()"},{"line_number":52,"context_line":"        subs \u003d {"},{"line_number":53,"context_line":"            \"backup_id\": backup_id,"},{"line_number":54,"context_line":"            \"os-reset_status\": \"error\""},{"line_number":55,"context_line":"        }"},{"line_number":56,"context_line":"        response \u003d self._do_post(\u0027backups/%s/action\u0027 % backup_id,"},{"line_number":57,"context_line":"                                 \u0027backup-reset-status-request\u0027,"},{"line_number":58,"context_line":"                                 subs)"},{"line_number":59,"context_line":"        self.assertEqual(202, response.status_code)"},{"line_number":60,"context_line":"        self.assertEqual(\u0027\u0027, response.content)"}],"source_content_type":"text/x-python","patch_set":3,"id":"254b6ba5_96a255a5","line":60,"range":{"start_line":50,"start_character":4,"end_line":60,"end_character":46},"in_reply_to":"64b9fe3e_57c7c5fe","updated":"2023-07-27 05:48:03.000000000","message":"This code path fetches the backup host[1] which requires backup manager (though i haven\u0027t traced down the whole code path) so again not possible\n[1] https://github.com/openstack/cinder/blob/26847ddeed98f4078908844ab86de3932bc49b3e/cinder/backup/api.py#L477-L478","commit_id":"1c0a4fa26c08c77f1b6a6a6603413887e7c1d910"}]}
