)]}'
{"designate_tempest_plugin/tests/api/v2/test_transfer_accepts.py":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"2e4b766825dfc4fb94547f5ee06a476126c0fa12","unresolved":true,"context_lines":[{"line_number":208,"context_line":"        LOG.info(\u0027Use Admin tenant to receive a list of all transfer accepts, \u0027"},{"line_number":209,"context_line":"                 \u0027for all projects\u0027)"},{"line_number":210,"context_line":"        accept_ids \u003d ["},{"line_number":211,"context_line":"            item[\u0027id\u0027] for item in"},{"line_number":212,"context_line":"            self.admin_client.list_transfer_accepts("},{"line_number":213,"context_line":"                headers\u003d{\u0027x-auth-all-project\u0027: True})]"},{"line_number":214,"context_line":"        LOG.info(\u0027Retrieved list (IDs) of accepted transfers is:{} \u0027.format("}],"source_content_type":"text/x-python","patch_set":2,"id":"1bd11213_8aae8065","line":211,"updated":"2021-04-20 23:07:46.000000000","message":"This is incorrect and will return an empty list.","commit_id":"a9030d343366256f2e36a89f258904ad8db05345"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"7ecc359cbabee659223d635081356145d8567a6a","unresolved":true,"context_lines":[{"line_number":144,"context_line":"        # Cleanup is supposed to be done with Alt tenant, as Alt"},{"line_number":145,"context_line":"        # is used to get the ownership for that zone"},{"line_number":146,"context_line":"        self.addCleanup("},{"line_number":147,"context_line":"            self.wait_zone_delete, self.alt_zone_client, zone[\u0027id\u0027])"},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"        LOG.info(\u0027Create a zone transfer_request as primary tenant\u0027)"},{"line_number":150,"context_line":"        transfer_request \u003d self.request_client.create_transfer_request("}],"source_content_type":"text/x-python","patch_set":3,"id":"64b3f091_aafc2e53","line":147,"range":{"start_line":147,"start_character":40,"end_line":147,"end_character":55},"updated":"2021-04-28 21:33:00.000000000","message":"It is probably best to use the admin client for cleanup.\nIf something fails before the alt client gets access to the zone, cleanup would fail.","commit_id":"dafdf888f20829ca86743876f97e173cd79b6a30"},{"author":{"_account_id":28609,"name":"Arkady Shtempler","email":"ashtempl@redhat.com","username":"ashtempl"},"change_message_id":"16e171adb8382959347d471fb8f272f7e03f437b","unresolved":false,"context_lines":[{"line_number":144,"context_line":"        # Cleanup is supposed to be done with Alt tenant, as Alt"},{"line_number":145,"context_line":"        # is used to get the ownership for that zone"},{"line_number":146,"context_line":"        self.addCleanup("},{"line_number":147,"context_line":"            self.wait_zone_delete, self.alt_zone_client, zone[\u0027id\u0027])"},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"        LOG.info(\u0027Create a zone transfer_request as primary tenant\u0027)"},{"line_number":150,"context_line":"        transfer_request \u003d self.request_client.create_transfer_request("}],"source_content_type":"text/x-python","patch_set":3,"id":"69144358_aebd7d19","line":147,"range":{"start_line":147,"start_character":40,"end_line":147,"end_character":55},"in_reply_to":"64b3f091_aafc2e53","updated":"2021-04-29 09:00:10.000000000","message":"Done","commit_id":"dafdf888f20829ca86743876f97e173cd79b6a30"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"0b733ccf6e6374d53a336c4725ad24f4b2e8e18f","unresolved":true,"context_lines":[{"line_number":154,"context_line":"            self.wait_zone_delete,"},{"line_number":155,"context_line":"            self.admin_zone_client, zone[\u0027id\u0027],"},{"line_number":156,"context_line":"            headers\u003d{"},{"line_number":157,"context_line":"                \u0027x-auth-sudo-project-id\u0027:"},{"line_number":158,"context_line":"                    self.os_primary.credentials.project_id},"},{"line_number":159,"context_line":"            ignore_errors\u003dlib_exc.NotFound)"},{"line_number":160,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"a97f7778_8ed301f5","line":157,"updated":"2021-04-29 18:17:45.000000000","message":"This should not be required. If you get an error without it, maybe we need x-auth-all-projects.\nThe idea here is to disconnect the delete from the project_id that owns the zone, since that project_id owner can change during this test.","commit_id":"d5195ffa4bf8b0c52e1b5d40f5f93de7ba96c5cb"},{"author":{"_account_id":28609,"name":"Arkady Shtempler","email":"ashtempl@redhat.com","username":"ashtempl"},"change_message_id":"32253ab5cbeb0252060b100d8b9b6137d43edfdd","unresolved":true,"context_lines":[{"line_number":154,"context_line":"            self.wait_zone_delete,"},{"line_number":155,"context_line":"            self.admin_zone_client, zone[\u0027id\u0027],"},{"line_number":156,"context_line":"            headers\u003d{"},{"line_number":157,"context_line":"                \u0027x-auth-sudo-project-id\u0027:"},{"line_number":158,"context_line":"                    self.os_primary.credentials.project_id},"},{"line_number":159,"context_line":"            ignore_errors\u003dlib_exc.NotFound)"},{"line_number":160,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"906686d1_857b1ea7","line":157,"in_reply_to":"a97f7778_8ed301f5","updated":"2021-04-29 19:05:41.000000000","message":"Yep, \"x-auth-all-projects\" works better here and this cleanup works as expected (without ignore_errors), when the second one line:#187 is disabled.","commit_id":"d5195ffa4bf8b0c52e1b5d40f5f93de7ba96c5cb"}]}