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

jewel: osd: scrub_to specifies clone ver, but transaction include head write ver #16405

Merged
merged 1 commit into from Sep 5, 2017

Conversation

Projects
None yet
3 participants
@dzafman
Member

dzafman commented Jul 18, 2017

@dzafman dzafman added this to the jewel milestone Jul 18, 2017

@smithfarm

This comment has been minimized.

Show comment
Hide comment
@smithfarm

smithfarm Jul 19, 2017

Contributor

@dzafman I gather this bug is already fixed in master?

Contributor

smithfarm commented Jul 19, 2017

@dzafman I gather this bug is already fixed in master?

@dzafman

This comment has been minimized.

Show comment
Hide comment
@dzafman

dzafman Jul 24, 2017

Member

@smithfarm No, we are waiting for the this Jewel version to be tested. Then we'll merge master version #16404 before this one.

Member

dzafman commented Jul 24, 2017

@smithfarm No, we are waiting for the this Jewel version to be tested. Then we'll merge master version #16404 before this one.

@dzafman dzafman changed the title from DNM: Jewel: Wip 20041 to DNM: Jewel: osd: scrub_to specifies clone ver, but transaction include head write ver Jul 24, 2017

@dzafman dzafman changed the title from DNM: Jewel: osd: scrub_to specifies clone ver, but transaction include head write ver to Jewel: osd: scrub_to specifies clone ver, but transaction include head write ver Jul 26, 2017

osd: scrub_to specifies clone ver, but transaction include head write…
… ver

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

Signed-off-by: David Zafman <dzafman@redhat.com>
(cherry picked from commit fd598a0)

Conflicts:
	src/osd/ReplicatedPG.cc (trivial)
@smithfarm

This comment has been minimized.

Show comment
Hide comment
@smithfarm

smithfarm Jul 31, 2017

Contributor

@yuriw Can you test this jewel PR? I'm working on kraken atm.

Contributor

smithfarm commented Jul 31, 2017

@yuriw Can you test this jewel PR? I'm working on kraken atm.

@smithfarm smithfarm changed the title from Jewel: osd: scrub_to specifies clone ver, but transaction include head write ver to jewel: osd: scrub_to specifies clone ver, but transaction include head write ver Aug 1, 2017

@smithfarm

This comment has been minimized.

Show comment
Hide comment
@smithfarm

smithfarm Aug 1, 2017

Contributor

master PR has been merged.

Contributor

smithfarm commented Aug 1, 2017

master PR has been merged.

@smithfarm smithfarm requested a review from jdurgin Sep 3, 2017

@smithfarm

This comment has been minimized.

Show comment
Hide comment
@smithfarm

smithfarm Sep 3, 2017

Contributor

@jdurgin This passed a rados run at http://tracker.ceph.com/issues/20613#note-40 with some failures, all of which I tried to reproduce. The only one that is reliably reproducible is http://tracker.ceph.com/issues/20981

Please review.

Contributor

smithfarm commented Sep 3, 2017

@jdurgin This passed a rados run at http://tracker.ceph.com/issues/20613#note-40 with some failures, all of which I tried to reproduce. The only one that is reliably reproducible is http://tracker.ceph.com/issues/20981

Please review.

@jdurgin

jdurgin approved these changes Sep 5, 2017

@jdurgin jdurgin merged commit e9eda5a into ceph:jewel Sep 5, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
make check make check succeeded
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment