)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":6968,"name":"Christian Schwede","email":"cschwede@nvidia.com","username":"cschwede"},"change_message_id":"0d511a06eb7a07e6010d18bcc4636860176a09ce","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c02ad183_c81949d4","updated":"2026-07-30 11:46:11.000000000","message":"LGTM.\n\nJust a minor nit, no reason for an extra round and cherry-picking from feature/mpu - pls see my inline comment.","commit_id":"f9fbe5589b8abff6252e2d25546cbf91eda83932"}],"test/s3api/test_versioning.py":[{"author":{"_account_id":6968,"name":"Christian Schwede","email":"cschwede@nvidia.com","username":"cschwede"},"change_message_id":"0d511a06eb7a07e6010d18bcc4636860176a09ce","unresolved":true,"context_lines":[{"line_number":265,"context_line":"        # we can delete a specific version"},{"line_number":266,"context_line":"        resp \u003d self.client.delete_object(Bucket\u003dself.bucket_name,"},{"line_number":267,"context_line":"                                         Key\u003dobj_name,"},{"line_number":268,"context_line":"                                         VersionId\u003dversions[1])"},{"line_number":269,"context_line":""},{"line_number":270,"context_line":"        # and that just pulls it out of the versions listing"},{"line_number":271,"context_line":"        resp \u003d self.client.list_object_versions(Bucket\u003dself.bucket_name)"}],"source_content_type":"text/x-python","patch_set":1,"id":"b8a4910a_da03d62f","line":268,"updated":"2026-07-30 11:46:11.000000000","message":"nit: every other `delete_object` below gets a `self.assertEqual(204, resp[\u0027ResponseMetadata\u0027][\u0027HTTPStatusCode\u0027])` with this patch - except this one. Might want to add that to feature/mpu later on as well?","commit_id":"f9fbe5589b8abff6252e2d25546cbf91eda83932"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"b1f254ec4a6a49a0dd7ed2463c2f9df80d621092","unresolved":false,"context_lines":[{"line_number":265,"context_line":"        # we can delete a specific version"},{"line_number":266,"context_line":"        resp \u003d self.client.delete_object(Bucket\u003dself.bucket_name,"},{"line_number":267,"context_line":"                                         Key\u003dobj_name,"},{"line_number":268,"context_line":"                                         VersionId\u003dversions[1])"},{"line_number":269,"context_line":""},{"line_number":270,"context_line":"        # and that just pulls it out of the versions listing"},{"line_number":271,"context_line":"        resp \u003d self.client.list_object_versions(Bucket\u003dself.bucket_name)"}],"source_content_type":"text/x-python","patch_set":1,"id":"db97d79a_3db8b032","line":268,"in_reply_to":"b8a4910a_da03d62f","updated":"2026-07-30 13:54:35.000000000","message":"Done - see 999291: s3api: add assertion to versioning compat test | https://review.opendev.org/c/openstack/swift/+/999291","commit_id":"f9fbe5589b8abff6252e2d25546cbf91eda83932"}]}
