Skip to content
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

rbd-mirror: deleting a snapshot during sync can result in read errors #13568

Merged
merged 3 commits into from Feb 23, 2017

Conversation

Projects
None yet
2 participants
@dillaman
Copy link
Contributor

dillaman commented Feb 21, 2017

No description provided.

@dillaman dillaman changed the title rbd-mirror: deleting a snapshot during sync can result in read errors [DNM] rbd-mirror: deleting a snapshot during sync can result in read errors Feb 21, 2017

dillaman added some commits Feb 21, 2017

rbd-mirror: replace complex object op tuple with struct
Signed-off-by: Jason Dillaman <dillaman@redhat.com>
rbd-mirror: object copy should always reference valid snapshots
If a remote snapshot is deleted while an image sync is in-progress,
associate the read request against the most recent, valid remote
snapshot for a given snapshot object clone.

Signed-off-by: Jason Dillaman <dillaman@redhat.com>
rbd-mirror: retry object copy after -ENOENT error
Fixes: http://tracker.ceph.com/issues/18990
Signed-off-by: Jason Dillaman <dillaman@redhat.com>

@dillaman dillaman force-pushed the dillaman:wip-18990 branch from 876afaa to b4f36d5 Feb 21, 2017

@dillaman dillaman changed the title [DNM] rbd-mirror: deleting a snapshot during sync can result in read errors rbd-mirror: deleting a snapshot during sync can result in read errors Feb 22, 2017

@trociny trociny self-assigned this Feb 22, 2017

@dillaman

This comment has been minimized.

Copy link
Contributor Author

dillaman commented Feb 22, 2017

@trociny
Copy link
Contributor

trociny left a comment

lgtm

@trociny trociny merged commit 7a8e718 into ceph:master Feb 23, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.