Bug #50028: seastore_test_t.omap_test_simple aborts - crimson - Ceph

Actions

Copy link

Bug #50028

closed

seastore_test_t.omap_test_simple aborts

Added by Kefu Chai about 3 years ago. Updated about 3 years ago.

Status:

Resolved

Priority:

Normal

Assignee:

Yingxin Cheng

Category:

Target version:

% Done:

Source:

Tags:

Backport:

Regression:

Severity:

3 - minor

Reviewed:

Affected Versions:

ceph-qa-suite:

Pull request ID:

40462

Crash signature (v1):

Crash signature (v2):

Description

[ RUN      ] seastore_test_t.collection_create_list_remove
...
unittest-seastore: ../src/crimson/os/seastore/onode_manager/staged-fltree/node_delta_recorder.h:21: virtual crimson::os::seastore::onode::DeltaRecorder::~DeltaRecorder(): Assertion `is_empty()' failed.
Aborting on shard 0.

hi Sam, could you take a look?

the log is collected from the failure of a "make check" run of https://github.com/ceph/ceph/pull/40444 .

Files

seastore.log (418 KB) seastore.log

Kefu Chai, 03/27/2021 04:40 AM

Actions

Copy link

Updated by Samuel Just about 3 years ago

Subject changed from seastore_test_t.collection_create_list_remove aborts to seastore_test_t.omap_test_simple aborts

Actions

Copy link

Updated by Samuel Just about 3 years ago

Assignee changed from Samuel Just to Yingxin Cheng

I think the set_omap transaction hit an eagain due to a race with the GC, which is fine. In those cases, I think DeltaRecorder is supposed to be destroyed without being empty. That assert should, I think, be removed. Does that sound right?

Actions

Copy link