Bug #13429: bufferlist::iterator no longer ABI compatible - Ceph - Ceph

Actions

Copy link

Bug #13429

closed

bufferlist::iterator no longer ABI compatible

Added by Jason Dillaman over 8 years ago. Updated over 8 years ago.

Status:

Resolved

Priority:

High

Assignee:

Jason Dillaman

Category:

Target version:

% Done:

Source:

other

Tags:

Backport:

Regression:

Severity:

3 - minor

Reviewed:

Affected Versions:

ceph-qa-suite:

Pull request ID:

Crash signature (v1):

Crash signature (v2):

Description

See http://qa-proxy.ceph.com/teuthology/jdillaman-2015-10-08_22:41:27-upgrade:client-upgrade-infernalis---basic-multi/1095689/teuthology.log

2015-10-08T20:29:52.062 INFO:tasks.workunit.client.0.plana67.stdout:[ RUN      ] TestLibRBD.TestIOPP
2015-10-08T20:29:52.062 INFO:tasks.workunit.client.0.plana67.stdout:using new format!
2015-10-08T20:29:52.269 INFO:tasks.workunit.client.0.plana67.stderr:ceph_test_librbd_api: symbol lookup error: ceph_test_librbd_api: undefined symbol: _ZN4ceph6buffer4list8iterator7advanceEi

The majority of public API methods were moved to a base class, which breaks compatibility.

Related issues 1 (0 open — 1 closed)

Actions

Copy link

Updated by Jason Dillaman over 8 years ago

ABI broken via commit 6504e3733, 1b43d811d, and d34e0415d

Actions

Copy link

Updated by Jason Dillaman over 8 years ago

librados API compatibility also broken via 4649ba506 and 6613b4b67

Actions

Copy link

Updated by Jason Dillaman over 8 years ago

Status changed from In Progress to Fix Under Review

infernalis PR: https://github.com/ceph/ceph/pull/6215

Actions

Copy link

Updated by Jason Dillaman over 8 years ago

Verified API compatibility via:

diff <(nm -Cg /path/to/infernalis/librados.so | grep " T " | cut -d ' ' -f 3- | sort) <(nm -Cg /path/to/hammer/librados.so | grep " T " |cut -d ' ' -f 3- | sort)

Actions

Copy link