)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"487cc49bc57ed0b69c2f5a86314889f47b32f572","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"2ceb50ce_ef40ef67","updated":"2026-02-13 16:38:06.000000000","message":"one nit! thanks for all the straightening out @Tim!","commit_id":"1047ae3f44c58cde3104ac645dd9b524ab9d6b2b"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"a06e294afa6fa566e4f7a0adc6986aa8510367fb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"1918a797_65ae62dc","updated":"2026-02-13 16:45:40.000000000","message":"I checked the compat test change is ok:\n```\n(swift-3.8.15) (acoles) ~/0dev/openstack/swift{review/tim_burke/merge-master} % SWIFT_TEST_AWS_CONFIG_FILE\u003d~/.aws/credentials pytest ./test/s3api/test_input_errors.py -s\n\n\u003csnip\u003e\n\n\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d 464 passed, 28 warnings in 165.53s (0:02:45) \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n```","commit_id":"7026d3939c605ce5c7a5e34470ef4504f5dc6e04"}],"test/probe/test_container_merge_policy_index.py":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"487cc49bc57ed0b69c2f5a86314889f47b32f572","unresolved":true,"context_lines":[{"line_number":572,"context_line":"                                    self.container_name,"},{"line_number":573,"context_line":"                                    self.object_name)"},{"line_number":574,"context_line":"        self.assertFalse(any(unexpected in name for name in misplaced),"},{"line_number":575,"context_line":"                         f\"Found {unexpected} record in {misplaced}\")"},{"line_number":576,"context_line":""},{"line_number":577,"context_line":"        # verify that the object data read by external client is correct"},{"line_number":578,"context_line":"        headers, data \u003d self._get_object_patiently(int(new_policy))"}],"source_content_type":"text/x-python","patch_set":4,"id":"d5ec5b6c_e9935c01","line":575,"updated":"2026-02-13 16:38:06.000000000","message":"@Tim do these changes need to go on master?","commit_id":"1047ae3f44c58cde3104ac645dd9b524ab9d6b2b"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"48a60c9f001a45df73fb20eac8e82f14b4265cf7","unresolved":true,"context_lines":[{"line_number":572,"context_line":"                                    self.container_name,"},{"line_number":573,"context_line":"                                    self.object_name)"},{"line_number":574,"context_line":"        self.assertFalse(any(unexpected in name for name in misplaced),"},{"line_number":575,"context_line":"                         f\"Found {unexpected} record in {misplaced}\")"},{"line_number":576,"context_line":""},{"line_number":577,"context_line":"        # verify that the object data read by external client is correct"},{"line_number":578,"context_line":"        headers, data \u003d self._get_object_patiently(int(new_policy))"}],"source_content_type":"text/x-python","patch_set":4,"id":"e38c3d5c_dbd34390","line":575,"in_reply_to":"d5ec5b6c_e9935c01","updated":"2026-02-13 16:50:23.000000000","message":"Certainly couldn\u0027t hurt, but AFAICT they\u0027re only necessary because of the new `\\x00history` container mpu introduces.","commit_id":"1047ae3f44c58cde3104ac645dd9b524ab9d6b2b"}],"test/s3api/test_input_errors.py":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"487cc49bc57ed0b69c2f5a86314889f47b32f572","unresolved":true,"context_lines":[{"line_number":267,"context_line":"                         if k not in self.ignored_auth_query_params}"},{"line_number":268,"context_line":"        if query_to_sign:"},{"line_number":269,"context_line":"            resource +\u003d \u0027?\u0027 + self._query_string(query_to_sign)"},{"line_number":270,"context_line":""},{"line_number":271,"context_line":"        string_to_sign_lines.append(resource)"},{"line_number":272,"context_line":""},{"line_number":273,"context_line":"        signature \u003d base64.b64encode(_hmac("}],"source_content_type":"text/x-python","patch_set":4,"id":"61b2e690_051e6c22","line":270,"updated":"2026-02-13 16:38:06.000000000","message":"nit: whitespace added w.r.t master","commit_id":"1047ae3f44c58cde3104ac645dd9b524ab9d6b2b"}]}
