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: need conversion tool to handle fixes following #11974 #5384

Merged
3 commits merged into from Aug 4, 2015

Conversation

Projects
None yet
2 participants
@ghost

ghost commented Jul 28, 2015

yehudasa added some commits Apr 22, 2015

rgw_admin: add --remove-bad flag to bucket check
Add this flag so that the bad object will be removed (should be called
only after user has verified that objects content is correct).

Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
(cherry picked from commit 06d67d9)
rgw: only scan for objects not in namespace
Fixes: #11984
The tool should only work on the head objects, and these are not inside
any namespace.

Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
(cherry picked from commit 8103908)
rgw: conversion tool to fix broken multipart objects
Fixes: #12079

Broken multipart objects: multipart objects that created on 0.94.2
and that start with underscore have a bad locator on their tail objects.
This extends the tool that was needed for older issue we've had with
hammer multipart objects (that start with underscore). The same usage
applies:

$ radosgw-admin bucket check --check-head-obj-locator \
                             --bucket=<bucket> [--fix]

Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
(cherry picked from commit f02ca61)

@ghost ghost self-assigned this Jul 28, 2015

@ghost ghost added this to the hammer milestone Jul 28, 2015

@ghost ghost added bug fix rgw labels Jul 28, 2015

ghost pushed a commit that referenced this pull request Jul 29, 2015

@ghost

This comment has been minimized.

ghost commented Aug 1, 2015

pushed wip-12502-hammer in http://ceph.com/gitbuilder.cgi

@ghost

This comment has been minimized.

ghost commented Aug 3, 2015

@yehudasa although it had to be scheduled three times, it passes a rgw suite ( http://tracker.ceph.com/issues/12502#note-6 ). With one problem which is unrelated, I think ( http://tracker.ceph.com/issues/12470 ). Do you think it is good to merge in hammer ?

@yehudasa

This comment has been minimized.

Member

yehudasa commented Aug 3, 2015

@dachary yes

ghost pushed a commit that referenced this pull request Aug 4, 2015

Loic Dachary
Merge pull request #5384 from dachary/wip-12502-hammer
rgw: need conversion tool to handle fixes following #11974

Reviewed-by: Yehuda Sadeh <ysadehwe@redhat.com>

@ghost ghost merged commit bb12f92 into ceph:hammer Aug 4, 2015

This issue was closed.

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