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

osd: LibRadosTwoPools[EC]PP.PromoteSnap failure #5050

Merged
2 commits merged into from Oct 20, 2015

Conversation

Projects
None yet
3 participants
@smithfarm
Contributor

smithfarm commented Jun 22, 2015

@smithfarm smithfarm added this to the firefly milestone Jun 22, 2015

@smithfarm smithfarm changed the title from LibRadosTwoPools[EC]PP.PromoteSnap failure to [DNM] LibRadosTwoPools[EC]PP.PromoteSnap failure Jun 22, 2015

@smithfarm

This comment has been minimized.

Contributor

smithfarm commented Jun 23, 2015

I see now that 53cbd79 depends on a03f85a , which deprecates ObjectIterator in favor of NObjectIterator.

@smithfarm

This comment has been minimized.

Contributor

smithfarm commented Jun 24, 2015

cherry-picked d7a8148 to expose RadostTest::nspace

@smithfarm

This comment has been minimized.

Contributor

smithfarm commented Jun 25, 2015

@liewegas I have gotten quite bogged down in this attempt to backport http://tracker.ceph.com/issues/10052 and am beginning to think #2926 uses too much post-firefly stuff. What do you think?

@ghost ghost assigned smithfarm Aug 29, 2015

liewegas added some commits Nov 14, 2014

osd/ReplicatedPG: allow whiteout deletion with IGNORE_CACHE flag
If the client specifies IGNORE_CACHE, allow a regular DELETE to zap a
whiteout.  Expand test case to verify this works.

Signed-off-by: Sage Weil <sage@redhat.com>
(cherry picked from commit 34e4d24)

    Conflicts:
        src/test/librados/tier.cc
            replaced NObjectIterator -> ObjectIterator
            replaced cache_ioctx.nobjects_begin -> cache_ioctx.objects_begin
            replaced cache_ioctx.nobjects_end -> cache_ioctx.objects_end
            replace it->get_oid() with it->first for it of type ObjectIterator
ceph_test_rados_api_tier: fix cleanup of whiteouts
Doing a normal DELETE on a whiteout returns ENOENT.  Use the IGNORE_CACHE
flag to make sure it removes the whiteout, too.

Fixes: #10052
Signed-off-by: Sage Weil <sage@redhat.com>
(cherry picked from commit 8dde6a6)

Conflicts:
	src/test/librados/TestCase.cc
            for it of type ObjectIterator:
            - use it->first instead of it->get_oid()
            - use it->second instead of it->get_locator()

@smithfarm smithfarm changed the title from [DNM] LibRadosTwoPools[EC]PP.PromoteSnap failure to LibRadosTwoPools[EC]PP.PromoteSnap failure Sep 29, 2015

@liewegas

This comment has been minimized.

Member

liewegas commented Sep 30, 2015

lgtm

@smithfarm

This comment has been minimized.

Contributor

smithfarm commented Sep 30, 2015

\o/

ghost pushed a commit that referenced this pull request Oct 20, 2015

Merge pull request #5050: LibRadosTwoPools[EC]PP.PromoteSnap failure
Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Oct 20, 2015

Loic Dachary
Merge pull request #5050 from SUSE/wip-12015-firefly
LibRadosTwoPools[EC]PP.PromoteSnap failure

Reviewed-by: Sage Weil <sage@redhat.com>

@ghost ghost merged commit 21452ed into ceph:firefly Oct 20, 2015

@smithfarm smithfarm deleted the SUSE:wip-12015-firefly branch Oct 20, 2015

@ghost ghost changed the title from LibRadosTwoPools[EC]PP.PromoteSnap failure to osd: LibRadosTwoPools[EC]PP.PromoteSnap failure Oct 24, 2015

This issue was closed.

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