)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5572,"name":"Don Kehn","display_name":"DEKehn","email":"dekehn@gmail.com","username":"dekehn"},"change_message_id":"8dbee98ebfc028399da6d5e4cfb0cedebadb43a0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"7e178d55_50b0ab9e","updated":"2022-07-12 23:40:21.000000000","message":"See note basically the usage of the \u0027_\u0027","commit_id":"1784ed2da5c12ac9b9e4e64f2c0eeae36ba0c10e"},{"author":{"_account_id":5572,"name":"Don Kehn","display_name":"DEKehn","email":"dekehn@gmail.com","username":"dekehn"},"change_message_id":"bdddf476a24e71f68b95ef5ef32de930047d73eb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"57d7f1eb_00ce3ed7","updated":"2022-07-13 14:06:26.000000000","message":"LGTM","commit_id":"a5db847f21e33d69c713c8d8074b675fbaa0b3db"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"cb93fa452846142da9bd85006795c221b3bb2a31","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"58eb3f86_dfb6f592","updated":"2022-07-14 20:17:48.000000000","message":"LGTM","commit_id":"a5db847f21e33d69c713c8d8074b675fbaa0b3db"}],"designate/tests/test_api/test_service.py":[{"author":{"_account_id":5572,"name":"Don Kehn","display_name":"DEKehn","email":"dekehn@gmail.com","username":"dekehn"},"change_message_id":"8dbee98ebfc028399da6d5e4cfb0cedebadb43a0","unresolved":true,"context_lines":[{"line_number":40,"context_line":"        self.assertIsInstance(self.service.wsgi_application, urlmap.URLMap)"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"    @mock.patch(\u0027designate.utils.find_config\u0027, return_value\u003dlist())"},{"line_number":43,"context_line":"    def test_unable_to_find_config(self, _):"},{"line_number":44,"context_line":"        with self.assertRaisesRegex("},{"line_number":45,"context_line":"                exceptions.ConfigurationError,"},{"line_number":46,"context_line":"                \u0027Unable to determine appropriate api-paste-config file\u0027):"}],"source_content_type":"text/x-python","patch_set":8,"id":"64e23106_7bcfe4b7","line":43,"updated":"2022-07-12 23:40:21.000000000","message":"The use of the \u0027_\u0027 as a parameter to a function  is a don\u0027t care: I believe we are trying to get away from using the \u0027_\u0027 given its other meanings in openstack, you might consider setting it to None, or if remove completely.","commit_id":"1784ed2da5c12ac9b9e4e64f2c0eeae36ba0c10e"},{"author":{"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"},"change_message_id":"e2fe29e0cab6e56c7086f494b38f8e93febecd91","unresolved":false,"context_lines":[{"line_number":40,"context_line":"        self.assertIsInstance(self.service.wsgi_application, urlmap.URLMap)"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"    @mock.patch(\u0027designate.utils.find_config\u0027, return_value\u003dlist())"},{"line_number":43,"context_line":"    def test_unable_to_find_config(self, _):"},{"line_number":44,"context_line":"        with self.assertRaisesRegex("},{"line_number":45,"context_line":"                exceptions.ConfigurationError,"},{"line_number":46,"context_line":"                \u0027Unable to determine appropriate api-paste-config file\u0027):"}],"source_content_type":"text/x-python","patch_set":8,"id":"6c68ffc0_ea481843","line":43,"in_reply_to":"64e23106_7bcfe4b7","updated":"2022-07-13 01:51:18.000000000","message":"Thanks. Should be fixed now.","commit_id":"1784ed2da5c12ac9b9e4e64f2c0eeae36ba0c10e"}]}
