)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5572,"name":"Don Kehn","display_name":"DEKehn","email":"dekehn@gmail.com","username":"dekehn"},"change_message_id":"c19a8a5980cf73626ed2dda6128cf19cb7987c8f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"8b2358ac_e646d311","updated":"2022-03-17 20:19:22.000000000","message":"LGTM","commit_id":"b1f2940631f1e474069134c6b89b71b1eba940fb"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"ae8ed4572188a63db6b604e6773037c3832f4a55","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"16c6e7ce_65e85a20","updated":"2022-06-07 00:40:40.000000000","message":"LGTM","commit_id":"0fbbccdfd4d004e49aa41e5f2d3ebb2b710e03f9"},{"author":{"_account_id":5572,"name":"Don Kehn","display_name":"DEKehn","email":"dekehn@gmail.com","username":"dekehn"},"change_message_id":"9528b96093d30dee163dcdc7cce980b1cf431698","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"cd122104_edba03db","updated":"2022-05-25 20:51:11.000000000","message":"LGTM","commit_id":"0fbbccdfd4d004e49aa41e5f2d3ebb2b710e03f9"}],"designate_tempest_plugin/tests/api/v2/test_tld.py":[{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"72e0af2b993df72a8a54c4b4be4ae85afff1482c","unresolved":true,"context_lines":[{"line_number":30,"context_line":""},{"line_number":31,"context_line":"class TldAdminTest(BaseTldTest):"},{"line_number":32,"context_line":"    credentials \u003d [\"admin\", \"system_admin\", \"system_reader\","},{"line_number":33,"context_line":"                   \"primary\", \"alt\", \"project_reader\", \"project_member\"]"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    # Use a TLD suffix unique to this test class."},{"line_number":36,"context_line":"    local_tld_suffix \u003d \u0027.\u0027.join([\"tldadmintest\", CONF.dns.tld_suffix])"}],"source_content_type":"text/x-python","patch_set":7,"id":"37065804_e7828647","line":33,"range":{"start_line":33,"start_character":37,"end_line":33,"end_character":71},"updated":"2022-06-01 13:20:59.000000000","message":"Shouldn\u0027t those be added to credentials only if CONF.dns_feature_enabled.enforce_new_defaults ?","commit_id":"0fbbccdfd4d004e49aa41e5f2d3ebb2b710e03f9"},{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"ea32918b4daa5946c44e149d3f162023771bc12f","unresolved":false,"context_lines":[{"line_number":30,"context_line":""},{"line_number":31,"context_line":"class TldAdminTest(BaseTldTest):"},{"line_number":32,"context_line":"    credentials \u003d [\"admin\", \"system_admin\", \"system_reader\","},{"line_number":33,"context_line":"                   \"primary\", \"alt\", \"project_reader\", \"project_member\"]"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    # Use a TLD suffix unique to this test class."},{"line_number":36,"context_line":"    local_tld_suffix \u003d \u0027.\u0027.join([\"tldadmintest\", CONF.dns.tld_suffix])"}],"source_content_type":"text/x-python","patch_set":7,"id":"a60e76cd_17b5d112","line":33,"range":{"start_line":33,"start_character":37,"end_line":33,"end_character":71},"in_reply_to":"23687945_55af3e3c","updated":"2022-06-03 12:23:29.000000000","message":"Ack","commit_id":"0fbbccdfd4d004e49aa41e5f2d3ebb2b710e03f9"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"333b5c57474bf398849f8d556cba93ecd2fc1fc2","unresolved":true,"context_lines":[{"line_number":30,"context_line":""},{"line_number":31,"context_line":"class TldAdminTest(BaseTldTest):"},{"line_number":32,"context_line":"    credentials \u003d [\"admin\", \"system_admin\", \"system_reader\","},{"line_number":33,"context_line":"                   \"primary\", \"alt\", \"project_reader\", \"project_member\"]"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    # Use a TLD suffix unique to this test class."},{"line_number":36,"context_line":"    local_tld_suffix \u003d \u0027.\u0027.join([\"tldadmintest\", CONF.dns.tld_suffix])"}],"source_content_type":"text/x-python","patch_set":7,"id":"23687945_55af3e3c","line":33,"range":{"start_line":33,"start_character":37,"end_line":33,"end_character":71},"in_reply_to":"37065804_e7828647","updated":"2022-06-01 15:05:28.000000000","message":"This credential property is used by tempest to setup credentials in the test suite. I have opted to have them here so the negative tests, when \"enforce_new_defaults\" is False, to test the API with these extra credentials that should not be allowed access.","commit_id":"0fbbccdfd4d004e49aa41e5f2d3ebb2b710e03f9"}]}