)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"ddbdab1d9db74e87630958acbf23f747e4a28466","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"bc019828_37b78102","updated":"2025-01-07 17:18:04.000000000","message":"LGTM!  Since I\u0027m co-author I\u0027ll leave for someone else to +A","commit_id":"25c45f0ee3e5ccdc3236fd7b02f80d7a32d88bc6"},{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"4890e40c9429f1f4088b167a644557adb546009e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"40c09dfc_61c83232","updated":"2025-01-08 15:56:29.000000000","message":"great now we are authors!  let\u0027s get this merged.","commit_id":"1bc0507c13992395d6ec618a114622e0abdb7171"}],"test/unit/common/test_utils.py":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"468c8c6975e184b5e71b160e3d890ffec90d0d08","unresolved":true,"context_lines":[{"line_number":7520,"context_line":"        logger \u003d call_get_logger(None, None, None)"},{"line_number":7521,"context_line":"        self.assertEqual(\u0027swift.\u0027, logger.logger.statsd_client._prefix)"},{"line_number":7522,"context_line":"        self.assertEqual(\u0027swift\u0027, logger.name)"},{"line_number":7523,"context_line":"        self.assertEqual(\u0027swift\u0027, logger.server)"},{"line_number":7524,"context_line":""},{"line_number":7525,"context_line":"        # tail prefix defaults to swift, log_route is ignored for stats"},{"line_number":7526,"context_line":"        logger \u003d call_get_logger(None, None, None, log_route\u003d\u0027route\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"159a9e5f_72c03a07","line":7523,"updated":"2024-11-27 19:02:32.000000000","message":"these logger assertions are straying into the existing test.unit.common.utils.test_logs.TestUtilsLogs.test_get_swift_logger_name_and_route but perhaps interesting to have adjacent to the statsd assertions","commit_id":"af0c0608487bdf2fbad3f4f2e0c654b39fd2d68f"},{"author":{"_account_id":35790,"name":"Shreeya Deshpande","email":"shreeyad@nvidia.com","username":"shreeyad"},"change_message_id":"4ccbb77291cd63eb3bb64d69ffde7e41256091a6","unresolved":false,"context_lines":[{"line_number":7520,"context_line":"        logger \u003d call_get_logger(None, None, None)"},{"line_number":7521,"context_line":"        self.assertEqual(\u0027swift.\u0027, logger.logger.statsd_client._prefix)"},{"line_number":7522,"context_line":"        self.assertEqual(\u0027swift\u0027, logger.name)"},{"line_number":7523,"context_line":"        self.assertEqual(\u0027swift\u0027, logger.server)"},{"line_number":7524,"context_line":""},{"line_number":7525,"context_line":"        # tail prefix defaults to swift, log_route is ignored for stats"},{"line_number":7526,"context_line":"        logger \u003d call_get_logger(None, None, None, log_route\u003d\u0027route\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"2e01fd91_ba04cac4","line":7523,"in_reply_to":"159a9e5f_72c03a07","updated":"2025-01-07 15:11:43.000000000","message":"Acknowledged","commit_id":"af0c0608487bdf2fbad3f4f2e0c654b39fd2d68f"},{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"4890e40c9429f1f4088b167a644557adb546009e","unresolved":true,"context_lines":[{"line_number":7532,"context_line":"            swift_logger \u003d utils.get_logger("},{"line_number":7533,"context_line":"                conf, name\u003dname,"},{"line_number":7534,"context_line":"                log_route\u003dlog_route,"},{"line_number":7535,"context_line":"                statsd_tail_prefix\u003dstatsd_tail_prefix)"},{"line_number":7536,"context_line":"            self.assertTrue(hasattr(swift_logger.logger, \u0027statsd_client\u0027))"},{"line_number":7537,"context_line":"            self.assertIsInstance(swift_logger.logger.statsd_client,"},{"line_number":7538,"context_line":"                                  StatsdClient)"}],"source_content_type":"text/x-python","patch_set":6,"id":"1b7f53fa_29ed6faf","line":7535,"updated":"2025-01-08 15:56:29.000000000","message":"i think the ps4-\u003e6 change is drop the use of FakeSwiftClient patching here since we don\u0027t ever configure a statsd host or use the statsd_client...\n\nwhich seems also fine.","commit_id":"1bc0507c13992395d6ec618a114622e0abdb7171"}]}