)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"change_message_id":"b2a6e33c787fbc9bc0af261d32f7d915f13575e4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"1e12bcf1_f55060fe","updated":"2023-08-14 01:27:49.000000000","message":"yeah, nice use of the new fakestatsd client.","commit_id":"20dc888602f38f6c8aa289d35b308e101e4af334"}],"test/unit/common/middleware/s3api/test_s3api.py":[{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"033edf60de9e2bf52576a65f42f9a776d0b1fd45","unresolved":true,"context_lines":[{"line_number":234,"context_line":"        self.assertIsNot(s3api.logger.logger, proxy_logger)"},{"line_number":235,"context_line":"        self.assertEqual(\u0027proxy-server\u0027, s3api.logger.server)"},{"line_number":236,"context_line":"        self.assertEqual(\u0027s3api.\u0027, s3api.logger.logger.statsd_client._prefix)"},{"line_number":237,"context_line":"        client \u003d s3api.logger.logger.statsd_client"},{"line_number":238,"context_line":"        self.assertEqual({\u0027test-metric\u0027: 1}, client.get_increment_counts())"},{"line_number":239,"context_line":"        self.assertEqual(1, len(client.sendto_calls))"},{"line_number":240,"context_line":"        self.assertEqual(b\u0027s3api.test-metric:1|c\u0027, client.sendto_calls[0][0])"}],"source_content_type":"text/x-python","patch_set":2,"id":"e00ffe60_56137e92","line":237,"updated":"2023-08-21 22:39:42.000000000","message":"It\u0027s a bit of a funny string to pull -- definitely _not_ `s3api.logger.logger.logger.statsd_client` or `s3api.logger.statsd_client` yeah? Do either of those actually exist?","commit_id":"20dc888602f38f6c8aa289d35b308e101e4af334"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"f67c779aaa981a53885c3cf7045f441b8797f484","unresolved":true,"context_lines":[{"line_number":234,"context_line":"        self.assertIsNot(s3api.logger.logger, proxy_logger)"},{"line_number":235,"context_line":"        self.assertEqual(\u0027proxy-server\u0027, s3api.logger.server)"},{"line_number":236,"context_line":"        self.assertEqual(\u0027s3api.\u0027, s3api.logger.logger.statsd_client._prefix)"},{"line_number":237,"context_line":"        client \u003d s3api.logger.logger.statsd_client"},{"line_number":238,"context_line":"        self.assertEqual({\u0027test-metric\u0027: 1}, client.get_increment_counts())"},{"line_number":239,"context_line":"        self.assertEqual(1, len(client.sendto_calls))"},{"line_number":240,"context_line":"        self.assertEqual(b\u0027s3api.test-metric:1|c\u0027, client.sendto_calls[0][0])"}],"source_content_type":"text/x-python","patch_set":2,"id":"0d9e2b49_5675b199","line":237,"in_reply_to":"e00ffe60_56137e92","updated":"2023-08-22 09:44:24.000000000","message":"I\u0027m not sure what you\u0027re getting at, but no, s3api.logger.logger.logger.statsd_client or s3api.logger.statsd_client don\u0027t exist.\n\ns3api.logger is a LogAdapter that has a Logger instance names logger, which has a StatsdClient instance.","commit_id":"20dc888602f38f6c8aa289d35b308e101e4af334"}]}