Bug #41716: LibRadosTwoPoolsPP.ManifestUnset fails - RADOS - Ceph

Actions

Copy link

Bug #41716

closed

LibRadosTwoPoolsPP.ManifestUnset fails

Added by Sage Weil over 4 years ago. Updated over 4 years ago.

Status:

Resolved

Priority:

High

Assignee:

Myoungwon Oh

Category:

Target version:

% Done:

Source:

Tags:

Backport:

Regression:

Severity:

3 - minor

Reviewed:

Affected Versions:

ceph-qa-suite:

Component(RADOS):

Pull request ID:

29985

Crash signature (v1):

Crash signature (v2):

Description

2019-09-07T08:35:10.542 INFO:tasks.workunit.client.0.smithi130.stdout:              api_tier_pp: [ RUN      ] LibRadosTwoPoolsPP.ManifestUnset
2019-09-07T08:35:10.542 INFO:tasks.workunit.client.0.smithi130.stdout:              api_tier_pp: require_osd_release = octopus
2019-09-07T08:35:10.543 INFO:tasks.workunit.client.0.smithi130.stdout:              api_tier_pp: /build/ceph-15.0.0-4676-g4777952/src/test/librados/tier_cxx.cc:3294: Failure
2019-09-07T08:35:10.543 INFO:tasks.workunit.client.0.smithi130.stdout:              api_tier_pp: Expected equality of these values:
2019-09-07T08:35:10.543 INFO:tasks.workunit.client.0.smithi130.stdout:              api_tier_pp:   0
2019-09-07T08:35:10.543 INFO:tasks.workunit.client.0.smithi130.stdout:              api_tier_pp:   completion->get_return_value()

/a/sage-2019-09-06_05:32:36-rados-wip-sage-testing-2019-09-05-2132-distro-basic-smithi/4281946

Actions

Copy link

Updated by Sage Weil over 4 years ago

Project changed from mgr to RADOS

Actions

Copy link

Updated by Myoungwon Oh over 4 years ago

Assignee set to Myoungwon Oh

Actions

Copy link

Updated by Myoungwon Oh over 4 years ago

This issue is fixed by https://github.com/ceph/ceph/pull/29985
When the error occurs, the following ops are executed in order.

1. set-redirect and increase the reference count

at this time, set-redirect(retry) is received.
so,

1. set-redirect and increment the reference count
2. set-redirect(retry) increment the reference count
3. finish 1
4. finish 2 (but manifest info is already updated, so osd return EINVAL)

To avoid this situation, start_block() and stop_block() were added.

1. set-redirect and increment the reference count
start_block
2. finish 1
stop_block

Actions

Copy link