)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":33634,"name":"Jan Hartkopf","email":"j@hartkopf.io","username":"jhartkopf"},"change_message_id":"b38b7997ed96fe77f0717c004675eb77b3ad97b9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"66d8b602_7b6fe815","updated":"2023-11-15 09:26:40.000000000","message":"Even a more fundamental problem than I initially thought. LGTM.","commit_id":"d1c54e39a3ad2f832386129ee2d6530db01c7576"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"a779a13bd7143805b3f62c7f50223e37487e1255","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"eee83741_7b5844e0","updated":"2023-11-14 23:37:48.000000000","message":"LGTM","commit_id":"d1c54e39a3ad2f832386129ee2d6530db01c7576"}],"designate_tempest_plugin/tests/api/v2/test_zones.py":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"83a484923e9900e4f474ee719a615a2af81a9763","unresolved":true,"context_lines":[{"line_number":587,"context_line":"                 \u0027(using \"x-auth-sudo-project-id\" HTTP header) to show \u0027"},{"line_number":588,"context_line":"                 \u0027a Primary tenant zone.\u0027)"},{"line_number":589,"context_line":"        body \u003d self.admin_client.show_zone("},{"line_number":590,"context_line":"            uuid\u003dNone, headers\u003d{"},{"line_number":591,"context_line":"                \u0027x-auth-sudo-project-id\u0027: zone[\u0027project_id\u0027]})[1]"},{"line_number":592,"context_line":""},{"line_number":593,"context_line":"        LOG.info(\u0027Ensure the fetched response matches the impersonated\u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"0ebd1b54_afdce454","line":590,"updated":"2023-11-14 22:39:34.000000000","message":"I would argue that the bug in this test is here, it should specify the UUID of the zone. Otherwise it is testing zone_list.\nI would access the zone with the zone[\u0027id\u0027] and change the check below to access the zone root tag.","commit_id":"6c045e31c704391ffd8f86f1b621c86119048df3"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"924dfbea19860a308b940e24b414974184a07b2c","unresolved":true,"context_lines":[{"line_number":592,"context_line":""},{"line_number":593,"context_line":"        LOG.info(\u0027Ensure the fetched response matches the impersonated\u0027"},{"line_number":594,"context_line":"                 \u0027 project, it means the ID of a zone \"A\"\u0027)"},{"line_number":595,"context_line":"        self.assertExpected(zone, body[\u0027zone\u0027], self.excluded_keys)"},{"line_number":596,"context_line":""},{"line_number":597,"context_line":"    @decorators.idempotent_id(\u0027e1cf7104-8b06-11eb-a861-74e5f9e2a801\u0027)"},{"line_number":598,"context_line":"    def test_list_all_projects_zones(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"a227a8b7_30ac71da","line":595,"updated":"2023-11-14 22:49:22.000000000","message":"Opps, zone show doesn\u0027t have a root tag (inconsistent API, sigh). Fixing.","commit_id":"207fcaa9af48751a1b6164e2e52a5a4b76731dd7"}]}