Delete Object Operation doesn't delete segments when it is multipart object

Bug #1420144 reported by SeongSoo Cho
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Swift3
Fix Committed
Undecided
Naoto Nishizono

Bug Description

when delete multipart uploaded object, it must be deleting segments. but swift3 doens't delete segments for multipart uploaded object.

To solve this bug(?), I suggest some idea.

Solution)
1. every delete object operation, check object metadata to ''X-Static-Large-Object' is in.
2. if object has 'X-Static-Large-Object' metadata, add 'multipart-manifest=delete' parameter in original request

The SLO returns 200 OK about 'multipart-manifest=delete' operation. So add HTTP_OK on expect success code map. (request.py)

description: updated
Revision history for this message
Naoto Nishizono (nishizono-naoto) wrote :

I have confirmed the bug details . I will address this .

Changed in swift3:
assignee: nobody → Naoto Nishizono (nishizono-naoto)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to swift3 (master)

Fix proposed to branch: master
Review: https://review.openstack.org/155199

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to swift3 (master)

Reviewed: https://review.openstack.org/155199
Committed: https://git.openstack.org/cgit/stackforge/swift3/commit/?id=14981b47c5d5904b8355a5b1bf9899dd0460b3c5
Submitter: Jenkins
Branch: master

commit 14981b47c5d5904b8355a5b1bf9899dd0460b3c5
Author: Naoto Nishizono <email address hidden>
Date: Thu Feb 12 15:22:37 2015 +0900

    Fix DELETE Object to delete segments when it is multipart object

    When deleting an object created via Multipart Upload, delete both the
    manifest file and the segments by adding "multipart-manifest=delete"
    to the query string.

    This requires an additional HEAD before each DELETE, which adds a good
    bit of overhead. There is a new config option, "allow_multipart_uploads"
    which operators may turn off to avoid this overhead if they know their
    use-case does not require Multipart Uploads.

    Co-Authored-By: Tim Burke <email address hidden>
    Change-Id: Ie1889750b0e6fbe48af0da40596f09ed504b9099
    Closes-Bug: #1420144

Changed in swift3:
status: In Progress → Fix Committed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.