)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"7388628a704e41f70fe2c84d72a94107a9af8c4f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"9f10d93e_dc3ac724","updated":"2023-12-06 15:02:02.000000000","message":"thanks for the fixup!","commit_id":"7dadc5cac1e47605def54c78ebb10744e0239620"}],"test/unit/proxy/controllers/test_base.py":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"31c33b97eef235bf034f5568e297c3dd1eebcbc6","unresolved":true,"context_lines":[{"line_number":1670,"context_line":"        src.resp.nuke_from_orbit.assert_called_once_with()"},{"line_number":1671,"context_line":""},{"line_number":1672,"context_line":""},{"line_number":1673,"context_line":"@patch_policies([StoragePolicy(0, \u0027zero\u0027, True, object_ring\u003dFakeRing())])"},{"line_number":1674,"context_line":"class TestGetOrHeadHandler(BaseTest):"},{"line_number":1675,"context_line":"    def test_disconnected_logging(self):"},{"line_number":1676,"context_line":"        self.app.logger \u003d mock.Mock()"}],"source_content_type":"text/x-python","patch_set":1,"id":"a46f3283_9495f3d9","side":"PARENT","line":1673,"updated":"2023-12-06 10:35:04.000000000","message":"On zuul and OMM there\u0027s no default object ring file so this is needed.\n\nIt could go on BaseTest but I figure that makes TestBase brittle to being used with other policies (although right that isn\u0027t the case).","commit_id":"36ce42f54e48cdbf6edfc63dcb7a20a08b4e89a4"},{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"7388628a704e41f70fe2c84d72a94107a9af8c4f","unresolved":false,"context_lines":[{"line_number":1670,"context_line":"        src.resp.nuke_from_orbit.assert_called_once_with()"},{"line_number":1671,"context_line":""},{"line_number":1672,"context_line":""},{"line_number":1673,"context_line":"@patch_policies([StoragePolicy(0, \u0027zero\u0027, True, object_ring\u003dFakeRing())])"},{"line_number":1674,"context_line":"class TestGetOrHeadHandler(BaseTest):"},{"line_number":1675,"context_line":"    def test_disconnected_logging(self):"},{"line_number":1676,"context_line":"        self.app.logger \u003d mock.Mock()"}],"source_content_type":"text/x-python","patch_set":1,"id":"1d56b782_89b9d921","side":"PARENT","line":1673,"in_reply_to":"a46f3283_9495f3d9","updated":"2023-12-06 15:02:02.000000000","message":"Acknowledged","commit_id":"36ce42f54e48cdbf6edfc63dcb7a20a08b4e89a4"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"31c33b97eef235bf034f5568e297c3dd1eebcbc6","unresolved":true,"context_lines":[{"line_number":1690,"context_line":"            resp \u003d handler.get_working_response(req)"},{"line_number":1691,"context_line":"            resp.app_iter.close()"},{"line_number":1692,"context_line":"        self.assertEqual([\"Client disconnected on read of \u0027some-path\u0027\"],"},{"line_number":1693,"context_line":"                         self.logger.get_lines_for_level(\u0027info\u0027))"},{"line_number":1694,"context_line":""},{"line_number":1695,"context_line":"        self.logger.clear()"},{"line_number":1696,"context_line":"        node \u003d {\u0027ip\u0027: \u00271.2.3.4\u0027, \u0027port\u0027: 6200, \u0027device\u0027: \u0027sda\u0027}"}],"source_content_type":"text/x-python","patch_set":1,"id":"5dd7fe7e_769cc75b","line":1693,"updated":"2023-12-06 10:35:04.000000000","message":"yes, this is more than reasonable given that there is already a debug_logger installed in the app!","commit_id":"a3e983482c769e7eda6e050dd9dd32f7c1e7b7c7"}]}