)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"05b8e1663d55218898289b4c9fb49c2d4c67446c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ebee57d7_0c1b6bc2","updated":"2023-11-14 19:09:05.000000000","message":"recheck\n\nhttps://bugs.launchpad.net/swift/+bug/2028175","commit_id":"6db575ba1d93607f5d68e8bacd0724a21e8260ae"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"7427bce02a2863b1868e479850b23101ff92527c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"38be7071_3767a867","updated":"2024-06-06 14:13:26.000000000","message":"If we make progress with https://review.opendev.org/c/openstack/swift/+/921451 then I\u0027d suggest re-using the request helper that I introduce in that patch for setting log_info","commit_id":"75ed9eeaa0a4b5d123daadfe5ab8f48444bad3e9"}],"test/unit/common/middleware/s3api/test_multi_upload.py":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"7427bce02a2863b1868e479850b23101ff92527c","unresolved":true,"context_lines":[{"line_number":158,"context_line":"        self.assertEqual([], self.swift.calls)"},{"line_number":159,"context_line":"        self.assertNotIn(\u0027X-Backend-Storage-Policy-Index\u0027, headers)"},{"line_number":160,"context_line":"        self.assertIn(\u0027mpu:X\u0027, req.environ.get(\u0027swift.log_info\u0027, []))"},{"line_number":161,"context_line":"        self.assertIn(\u0027mpu:X\u0027, req.environ.get(\u0027swift.log_info\u0027, []))"},{"line_number":162,"context_line":""},{"line_number":163,"context_line":"    def test_object_multipart_uploads_list(self):"},{"line_number":164,"context_line":"        req \u003d Request.blank(\u0027/bucket/object?uploads\u0027,"}],"source_content_type":"text/x-python","patch_set":5,"id":"920661ca_3403eb7a","line":161,"updated":"2024-06-06 14:13:26.000000000","message":"duplicate but also not true - the request doesn\u0027t make it past the object_operation decorator on PartController.PUT\n\nwhich makes me wonder if we need another decorator that sets log_info early in the request handling if uploadId is given with the request?","commit_id":"75ed9eeaa0a4b5d123daadfe5ab8f48444bad3e9"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"7427bce02a2863b1868e479850b23101ff92527c","unresolved":true,"context_lines":[{"line_number":2220,"context_line":"        override_etag \u003d \u0027; s3_etag\u003d%s\u0027 % expected_etag.strip(\u0027\"\u0027)"},{"line_number":2221,"context_line":"        h \u003d \u0027X-Object-Sysmeta-Container-Update-Override-Etag\u0027"},{"line_number":2222,"context_line":"        self.assertEqual(headers.get(h), override_etag)"},{"line_number":2223,"context_line":"        self.assertIn(\u0027mpu:invalid\u0027, req.environ.get(\u0027swift.log_info\u0027, []))"},{"line_number":2224,"context_line":"        # We never even get around to parsing uploadId"},{"line_number":2225,"context_line":"        self.assertNotIn(\u0027mpu:X\u0027, req.environ.get(\u0027swift.log_info\u0027, []))"},{"line_number":2226,"context_line":"        self.assertIn(\u0027mpu:X\u0027, req.environ.get(\u0027swift.log_info\u0027, []))"}],"source_content_type":"text/x-python","patch_set":5,"id":"b29a7c08_61acae39","line":2223,"updated":"2024-06-06 14:13:26.000000000","message":"I couldn\u0027t see where \u0027invalid\u0027 gets set and the test fails with \u0027mpu:x\u0027 in log_info","commit_id":"75ed9eeaa0a4b5d123daadfe5ab8f48444bad3e9"}]}
