Project

General

Profile

Actions

Bug #12828

closed

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 8 years ago. Updated over 8 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 1 (0 open1 closed)

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)ResolvedLoïc DacharyActions
Actions #1

Updated by Samuel Just over 8 years ago

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

I'm out of town until 9/14

Actions #2

Updated by Samuel Just over 8 years ago

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

Actions #3

Updated by Samuel Just over 8 years ago

  • Assignee deleted (Samuel Just)
Actions #4

Updated by Sage Weil over 8 years ago

  • Status changed from 7 to Resolved
Actions #5

Updated by Sage Weil over 8 years ago

  • Status changed from Resolved to Pending Backport
  • Backport set to hammer
Actions #7

Updated by Loïc Dachary over 8 years ago

  • Status changed from Pending Backport to Resolved
Actions

Also available in: Atom PDF