)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"cdf97bded58128fd9db95690c43ab1df33983390","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"db1e6038_eda85523","updated":"2022-08-09 17:04:34.000000000","message":"LGTM","commit_id":"fdcea400c6f591485fcb116cb47dc07c5229898d"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"eaecf311ab706b0dc7017dd4dee4d6752c36fdbe","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"c489b55f_f574913b","updated":"2022-08-09 17:01:30.000000000","message":"The API should never return a 500, I think we need to look deeper into what is happening there.","commit_id":"fdcea400c6f591485fcb116cb47dc07c5229898d"}],"octavia/tests/functional/api/v2/test_pool.py":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"eaecf311ab706b0dc7017dd4dee4d6752c36fdbe","unresolved":true,"context_lines":[{"line_number":2721,"context_line":""},{"line_number":2722,"context_line":"        new_pool \u003d {\u0027name\u0027: \u0027foo\u0027}"},{"line_number":2723,"context_line":"        self.put(self.POOL_PATH.format(pool_id\u003dapi_pool.get(\u0027id\u0027)),"},{"line_number":2724,"context_line":"                 self._build_body(new_pool), status\u003d500)"},{"line_number":2725,"context_line":""},{"line_number":2726,"context_line":"        lb \u003d self.get(self.LB_PATH.format(lb_id\u003dself.lb_id)).json.get("},{"line_number":2727,"context_line":"            \"loadbalancer\")"}],"source_content_type":"text/x-python","patch_set":3,"id":"2256c8f1_35c1bc39","line":2724,"updated":"2022-08-09 17:01:30.000000000","message":"The API should never return a 500 under normal situations. I think we need to look into why it\u0027s throwing the 500.","commit_id":"fdcea400c6f591485fcb116cb47dc07c5229898d"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"c7961bcfcbe8c16873ea7a9a8af59222e5f1ed0c","unresolved":true,"context_lines":[{"line_number":2721,"context_line":""},{"line_number":2722,"context_line":"        new_pool \u003d {\u0027name\u0027: \u0027foo\u0027}"},{"line_number":2723,"context_line":"        self.put(self.POOL_PATH.format(pool_id\u003dapi_pool.get(\u0027id\u0027)),"},{"line_number":2724,"context_line":"                 self._build_body(new_pool), status\u003d500)"},{"line_number":2725,"context_line":""},{"line_number":2726,"context_line":"        lb \u003d self.get(self.LB_PATH.format(lb_id\u003dself.lb_id)).json.get("},{"line_number":2727,"context_line":"            \"loadbalancer\")"}],"source_content_type":"text/x-python","patch_set":3,"id":"fd4ed118_e9de1024","line":2724,"in_reply_to":"2256c8f1_35c1bc39","updated":"2022-08-09 17:03:09.000000000","message":"Oh, I forgot we assigned 500 for ProviderDriverError. Nevermind.","commit_id":"fdcea400c6f591485fcb116cb47dc07c5229898d"}]}
