New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

rgw: fix the bug that part's index can't be removed after completing multipart upload #14500

Merged
merged 1 commit into from Jun 20, 2017

Conversation

Projects
None yet
3 participants
@zhangsw
Contributor

zhangsw commented Apr 13, 2017

If bucket versioning is enabled, complete multipart upload can't remove the part index.

Fixes: http://tracker.ceph.com/issues/19604

Signed-off-by: Zhang Shaowen zhangshaowen@cmss.chinamobile.com

rgw: fix the bug that part's index can't be removed after completing
multipart upload when the bucket versioning is enabled.

Fixes: http://tracker.ceph.com/issues/19604

Signed-off-by: Zhang Shaowen <zhangshaowen@cmss.chinamobile.com>

@zhangsw zhangsw changed the title from rgw: fix the bug that part's index can't be removed after completing multipart upload when the bucket versioning is enabled. to rgw: fix the bug that part's index can't be removed after completing multipart upload Apr 13, 2017

@zhangsw

This comment has been minimized.

Contributor

zhangsw commented Apr 21, 2017

@yehudasa @cbodley Please help to review this, thanks~

@zhangsw

This comment has been minimized.

Contributor

zhangsw commented Jun 6, 2017

@cbodley ping..

@cbodley

This comment has been minimized.

Contributor

cbodley commented Jun 6, 2017

@yehudasa can you take a look? how does this fit in with http://tracker.ceph.com/issues/17964 (rgw: multipart parts on versioned bucket create versioned bucket index entries)

@yehudasa

This comment has been minimized.

Member

yehudasa commented Jun 6, 2017

@zhangsw @cbodley looks good, probably duplicate as the other one. @zhangsw can you add a test to ragweed (https://github.com/ceph/ragweed) for this one?

@zhangsw

This comment has been minimized.

Contributor

zhangsw commented Jun 8, 2017

@yehudasa I've added a test to ragweed ceph/ragweed#1.

@zhangsw

This comment has been minimized.

Contributor

zhangsw commented Jun 20, 2017

jenkins test this please

1 similar comment
@cbodley

This comment has been minimized.

Contributor

cbodley commented Jun 20, 2017

jenkins test this please

@yehudasa yehudasa merged commit 8c85f84 into ceph:master Jun 20, 2017

5 of 6 checks passed

arm64 make check arm64 make check failed
Details
Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
Unmodified Submodules submodules for project are unmodified
Details
default Build finished.
Details
make check make check succeeded
Details

@zhangsw zhangsw deleted the zhangsw:fix-rgw-multipart-index-bug branch Jun 21, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment