)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":34930,"name":"Jianjian Huo","email":"jhuo@nvidia.com","username":"jhuo"},"change_message_id":"3c72f4d43bba78c5fa169365a187e020102961d3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"cf7e209e_fed38efb","updated":"2025-03-01 01:17:22.000000000","message":"the changes of internal client is not necessary anymore to fix the broken probe test, let\u0027s squash this patch and merge \"938631: py3: Fix unicode-header-name handling | https://review.opendev.org/c/openstack/swift/+/938631\"\n\nBTW, the test case change in this patch can be dropped.","commit_id":"847e5dd9965db9bd3a35bd2be5d87e0eaa797eff"}],"test/probe/test_reconstructor_rebuild.py":[{"author":{"_account_id":34930,"name":"Jianjian Huo","email":"jhuo@nvidia.com","username":"jhuo"},"change_message_id":"3c72f4d43bba78c5fa169365a187e020102961d3","unresolved":true,"context_lines":[{"line_number":433,"context_line":"        self.assertEqual(503, cm.exception.http_status)"},{"line_number":434,"context_line":"        # ... but client HEAD succeeds"},{"line_number":435,"context_line":""},{"line_number":436,"context_line":"        path \u003d self.int_client.make_path("},{"line_number":437,"context_line":"            self.account,"},{"line_number":438,"context_line":"            self.container_name.decode(\u0027utf-8\u0027),"},{"line_number":439,"context_line":"            self.object_name.decode(\u0027utf-8\u0027))"}],"source_content_type":"text/x-python","patch_set":1,"id":"2e75ac4f_1f647601","line":436,"updated":"2025-03-01 01:17:22.000000000","message":"I don\u0027t think the change into this test is needed. the test case still passes after I reverted the change.","commit_id":"847e5dd9965db9bd3a35bd2be5d87e0eaa797eff"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"ce71085f9d59f84f1819bd4039702d73824228c8","unresolved":true,"context_lines":[{"line_number":433,"context_line":"        self.assertEqual(503, cm.exception.http_status)"},{"line_number":434,"context_line":"        # ... but client HEAD succeeds"},{"line_number":435,"context_line":""},{"line_number":436,"context_line":"        path \u003d self.int_client.make_path("},{"line_number":437,"context_line":"            self.account,"},{"line_number":438,"context_line":"            self.container_name.decode(\u0027utf-8\u0027),"},{"line_number":439,"context_line":"            self.object_name.decode(\u0027utf-8\u0027))"}],"source_content_type":"text/x-python","patch_set":1,"id":"9d444a79_e028704d","line":436,"in_reply_to":"2e75ac4f_1f647601","updated":"2025-03-03 17:03:57.000000000","message":"It avoids the header-casing bug in internal_client.","commit_id":"847e5dd9965db9bd3a35bd2be5d87e0eaa797eff"},{"author":{"_account_id":34930,"name":"Jianjian Huo","email":"jhuo@nvidia.com","username":"jhuo"},"change_message_id":"e8b7c63b043fcefa9f5c1ff93e79c3decc0892ea","unresolved":true,"context_lines":[{"line_number":433,"context_line":"        self.assertEqual(503, cm.exception.http_status)"},{"line_number":434,"context_line":"        # ... but client HEAD succeeds"},{"line_number":435,"context_line":""},{"line_number":436,"context_line":"        path \u003d self.int_client.make_path("},{"line_number":437,"context_line":"            self.account,"},{"line_number":438,"context_line":"            self.container_name.decode(\u0027utf-8\u0027),"},{"line_number":439,"context_line":"            self.object_name.decode(\u0027utf-8\u0027))"}],"source_content_type":"text/x-python","patch_set":1,"id":"e5cc195f_a00531af","line":436,"in_reply_to":"9d444a79_e028704d","updated":"2025-03-03 19:30:33.000000000","message":"but this test case still passes without the internal client fix, I think it won\u0027t trigger the unicode header-casing bug.","commit_id":"847e5dd9965db9bd3a35bd2be5d87e0eaa797eff"}]}