Project

General

Profile

Bug #12828

ReplicatedBackend: populate recovery_info.size for clone (bug symptom is size mismatch on replicated backend on a clone in scrub)

Added by Samuel Just over 7 years ago. Updated almost 7 years ago.

Status:
Resolved
Priority:
Urgent
Assignee:
-
Category:
-
Target version:
-
% Done:

0%

Source:
other
Tags:
Backport:
hammer
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

This apparently never got filled in. It wasn't actually a problem since we do populate the the copy and clone subsets and the size is really only used for that. However, we now use the size to truncate out to the right size for the case where the end of the file is a hole (b5381c22a7b0d8d4c14cd8feb3a115f7d80d5d81). This didn't actually cause a failure until wip-proxy-write merged with a bug in ceph-test-rados causing an empty write past the end of the file resulting in such a hole.


Related issues

Copied to Ceph - Backport #13205: ReplicatedBackend: populate recovery_info.size for clone (bug symptom is size mismatch on replicated backend on a clone in scrub) Resolved

Associated revisions

Revision 1d3e46be (diff)
Added by Samuel Just about 7 years ago

ReplicatedBackend::prepare_pull: set recover_info.size for clones

Fixes: #12828
Signed-off-by: Samuel Just <>

Revision 3e657304 (diff)
Added by Samuel Just about 7 years ago

ReplicatedBackend::prepare_pull: set recover_info.size for clones

Fixes: #12828
Signed-off-by: Samuel Just <>
(cherry picked from commit 1d3e46bedb9a8c0d2200e39f62f4e2c3337619f3)

History

#1 Updated by Samuel Just over 7 years ago

https://github.com/ceph/ceph/pull/5706

I'm out of town until 9/14

#2 Updated by Samuel Just over 7 years ago

/a/samuelj-2015-08-26_12:18:34-rados-wip-sam-testing-distro-basic-multi/1033873/remote

#3 Updated by Samuel Just over 7 years ago

  • Assignee deleted (Samuel Just)

#4 Updated by Sage Weil about 7 years ago

  • Status changed from 7 to Resolved

#5 Updated by Sage Weil about 7 years ago

  • Status changed from Resolved to Pending Backport
  • Backport set to hammer

#7 Updated by Loïc Dachary almost 7 years ago

  • Status changed from Pending Backport to Resolved

Also available in: Atom PDF