)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"ebcb2e94077b0b47ca7f5eb77448cc17629f7db1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"276fba55_57839213","updated":"2023-05-19 23:32:53.000000000","message":"Well now I\u0027m not sure which way to go 🤔","commit_id":"81cd2f6017866107747cbf630b4cfa0d53260538"}],"test/debug_logger.py":[{"author":{"_account_id":34930,"name":"Jianjian Huo","email":"jhuo@nvidia.com","username":"jhuo"},"change_message_id":"a15146640c5bc685c7ebeb5bce543d104e408298","unresolved":true,"context_lines":[{"line_number":217,"context_line":""},{"line_number":218,"context_line":"    def _send_to_logger(name):"},{"line_number":219,"context_line":"        def stub_fn(self, *args, **kwargs):"},{"line_number":220,"context_line":"            if name \u003d\u003d \u0027timing_since\u0027:"},{"line_number":221,"context_line":"                # Simulation the internal op of real timing_since"},{"line_number":222,"context_line":"                test \u003d (time.time() - args[1]) * 1000"},{"line_number":223,"context_line":"            return getattr(self.logger, name)(*args, **kwargs)"}],"source_content_type":"text/x-python","patch_set":1,"id":"4ea96678_b5c195c6","line":220,"updated":"2023-05-19 23:18:42.000000000","message":"this change was able to catch the traceback and verify the fix, but it\u0027s causing other unrelated test case to fail. still debugging.\nERROR: test_log_line_format (test.unit.container.test_server.TestNonLegacyDefaultStoragePolicy)\nTraceback (most recent call last):\n  File \"/vagrant/swift/test/unit/container/test_server.py\", line 5535, in test_log_line_format\n    req.get_response(self.controller)\n  File \"/vagrant/swift/swift/common/swob.py\", line 1165, in get_response\n    status, headers, app_iter \u003d self.call_application(application)\n  File \"/vagrant/swift/swift/common/swob.py\", line 1149, in call_application\n    app_iter \u003d application(self.environ, start_response)\n  File \"/vagrant/swift/swift/container/server.py\", line 916, in __call__\n    log_message \u003d get_log_line(req, res, trans_time, \u0027\u0027,\n  File \"/vagrant/swift/swift/common/utils/__init__.py\", line 636, in get_log_line\n    \u0027time\u0027: StrFormatTime(time.time()),\n  File \"/usr/local/lib/python3.8/dist-packages/mock/mock.py\", line 1149, in __call__\n    return _mock_self._mock_call(*args, **kwargs)\n  File \"/usr/local/lib/python3.8/dist-packages/mock/mock.py\", line 1153, in _mock_call\n    return _mock_self._execute_mock_call(*args, **kwargs)\n  File \"/usr/local/lib/python3.8/dist-packages/mock/mock.py\", line 1212, in _execute_mock_call\n    result \u003d next(effect)\nStopIteration","commit_id":"81cd2f6017866107747cbf630b4cfa0d53260538"}],"test/unit/common/test_utils.py":[{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"ebcb2e94077b0b47ca7f5eb77448cc17629f7db1","unresolved":true,"context_lines":[{"line_number":5505,"context_line":"        # Timestamp objects work too"},{"line_number":5506,"context_line":"        self.assertStatMatches(r\u0027pfx\\.another\\.op:\\d+\\.\\d+\\|ms|@0.972\u0027,"},{"line_number":5507,"context_line":"                               self.logger.timing_since, \u0027another.op\u0027,"},{"line_number":5508,"context_line":"                               utils.Timestamp.now(), sample_rate\u003d0.972)"},{"line_number":5509,"context_line":"        self.assertStat(\u0027pfx.another.counter:3|c|@0.972\u0027,"},{"line_number":5510,"context_line":"                        self.logger.update_stats, \u0027another.counter\u0027, 3,"},{"line_number":5511,"context_line":"                        sample_rate\u003d0.972)"}],"source_content_type":"text/x-python","patch_set":1,"id":"cf24aef9_fbb83b34","side":"PARENT","line":5508,"updated":"2023-05-19 23:32:53.000000000","message":"Oh! I didn\u0027t realize we had tests that claimed to exercise the `timing_since`-takes-`Timestamp` behavior from https://review.opendev.org/c/openstack/swift/+/883636 ... hmm...","commit_id":"909052c881efac96c487ca7d3f11e126cf86c10c"}]}