Project

General

Profile

Actions

Bug #65130

open

crimson: crimson-rados did not detect reintroduction of https://tracker.ceph.com/issues/61875

Added by Samuel Just about 2 months ago. Updated 27 days ago.

Status:
Fix Under Review
Priority:
Urgent
Category:
-
Target version:
-
% Done:

0%

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

Description

https://github.com/ceph/ceph/pull/56376 would have reintroduced https://tracker.ceph.com/issues/61875 as it puts the snap mapper keys back into the pg meta object. Oddly, a teuthology run on that branch which seems to have included tests with both snapshots and osd restarts did not show crashes associated with this regression and at least one case that seems like it should have exercised the relevant code passed. A quick glance over PGLog.cc::FuturizedShardStoreReader doesn't show any changes, so it should have crashed in the final else branch of FuturizedShardStoreLogReader::process_entry at e.decode_with_checksum.

Tasks:
- Confirm that the crimson-rados suite actually combines snapshots with OSD restarts
- Work out why the existing suite didn't fail the above PR
- Amend the tests to cover the gap


Related issues 1 (1 open0 closed)

Related to crimson - Bug #65247: ObjectContext::drop_recovery_read(): Assertion `recovery_read_marker' failed.Need More InfoMatan Breizman

Actions
Actions

Also available in: Atom PDF