)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"8391c87a7ee76d75ea92ed29061d80bc384ade3f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"70b69137_c841e5a0","updated":"2022-07-18 22:59:13.000000000","message":"LGTM\nSome minor nits, but can be cleaned up later.","commit_id":"8abb3117afa5347e100887577eb29578371aac7e"},{"author":{"_account_id":5572,"name":"Don Kehn","display_name":"DEKehn","email":"dekehn@gmail.com","username":"dekehn"},"change_message_id":"388ac4c9421a63953865f0a01c6fea92bcd23dc1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"409026d8_4b8485df","updated":"2022-07-18 22:53:50.000000000","message":"LGTM - big","commit_id":"8abb3117afa5347e100887577eb29578371aac7e"}],"designate/objects/adapters/api_v2/service_status.py":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"8391c87a7ee76d75ea92ed29061d80bc384ade3f","unresolved":true,"context_lines":[{"line_number":43,"context_line":"        )"},{"line_number":44,"context_line":"        new_obj[\u0027links\u0027][\u0027self\u0027] \u003d ("},{"line_number":45,"context_line":"                \u0027%s/v2/%s/%s\u0027 % (cls._get_base_url(kwargs[\u0027request\u0027]),"},{"line_number":46,"context_line":"                                 \u0027service_statuses\u0027, new_obj[\u0027id\u0027])"},{"line_number":47,"context_line":"        )"},{"line_number":48,"context_line":"        return new_obj"},{"line_number":49,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"cc04d362_32188925","line":46,"updated":"2022-07-18 22:59:13.000000000","message":"nit: \u0027service_statuses\u0027 could move into the base string. No need to do a substitution with a static string.","commit_id":"8abb3117afa5347e100887577eb29578371aac7e"}],"designate/objects/adapters/api_v2/zone_import.py":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"8391c87a7ee76d75ea92ed29061d80bc384ade3f","unresolved":true,"context_lines":[{"line_number":46,"context_line":""},{"line_number":47,"context_line":"        if new_obj[\u0027zone_id\u0027] is not None:"},{"line_number":48,"context_line":"            new_obj[\u0027links\u0027][\u0027zone\u0027] \u003d ("},{"line_number":49,"context_line":"                \u0027%s/v2/%s/%s\u0027 % (cls._get_base_url(kwargs[\u0027request\u0027]), \u0027zones\u0027,"},{"line_number":50,"context_line":"                                 new_obj[\u0027zone_id\u0027])"},{"line_number":51,"context_line":"            )"},{"line_number":52,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"4cc93fd9_b6c8e6f6","line":49,"updated":"2022-07-18 22:59:13.000000000","message":"nit: Same here, \u0027zones\u0027 can go in the base string.","commit_id":"8abb3117afa5347e100887577eb29578371aac7e"}],"designate/objects/adapters/api_v2/zone_transfer_accept.py":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"8391c87a7ee76d75ea92ed29061d80bc384ade3f","unresolved":true,"context_lines":[{"line_number":48,"context_line":"        )"},{"line_number":49,"context_line":"        new_obj[\u0027links\u0027][\u0027zone\u0027] \u003d ("},{"line_number":50,"context_line":"                \u0027%s/v2/%s/%s\u0027 % (cls._get_base_url(kwargs[\u0027request\u0027]),"},{"line_number":51,"context_line":"                                 \u0027zones\u0027, new_obj[\u0027zone_id\u0027])"},{"line_number":52,"context_line":"        )"},{"line_number":53,"context_line":"        return new_obj"},{"line_number":54,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"203c3a63_723ec72c","line":51,"updated":"2022-07-18 22:59:13.000000000","message":"nit: Same here, \u0027zones\u0027 can be moved into the base string.","commit_id":"8abb3117afa5347e100887577eb29578371aac7e"}],"designate/tests/unit/objects/test_adapters.py":[{"author":{"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"},"change_message_id":"864b6415ee91ba277b6f2aad2153bc2765e95c8e","unresolved":false,"context_lines":[{"line_number":80,"context_line":"            test_dict[\u0027created_at\u0027], \u0027%Y-%m-%dT%H:%M:%S.%f\u0027"},{"line_number":81,"context_line":"        )"},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"        self.assertEqual(now, test_obj.created_at)"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"class RecordSetAPIv2AdapterTest(oslotest.base.BaseTestCase):"}],"source_content_type":"text/x-python","patch_set":5,"id":"eeccc5af_7c524970","line":83,"updated":"2022-07-18 23:20:03.000000000","message":"These tests are still largely useless and there is a mistake in the assert here I will fix in the future.","commit_id":"8abb3117afa5347e100887577eb29578371aac7e"}]}