Project

General

Profile

Actions

Bug #1821

closed

librados: rados_create_with_context is unusable

Added by Josh Durgin over 12 years ago. Updated about 12 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
librados
Target version:
-
% Done:

0%

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

Description

There's no way to get a CephContext using the C api, so you can't pass one to rados_create_with_context. Maybe a rados_create_with_config(rados_t *dest, rados_t src) would make more sense?

Actions #1

Updated by Sage Weil over 12 years ago

i think radosgw uses it. it creates a CephContext by linking directly the ceph internals...

Actions #2

Updated by Josh Durgin over 12 years ago

The C++ variant librados::Rados::init_with_context is used by librbd, radosgw, and some command line tools, but this bug is about the C version of the function, which doesn't make much sense since there's no C api for dealing with a CephContext. Nothing in ceph.git uses the C api.

Actions #3

Updated by Sage Weil over 12 years ago

Josh Durgin wrote:

The C++ variant librados::Rados::init_with_context is used by librbd, radosgw, and some command line tools, but this bug is about the C version of the function, which doesn't make much sense since there's no C api for dealing with a CephContext. Nothing in ceph.git uses the C api.

ah, ok!

Actions #4

Updated by Sage Weil about 12 years ago

see wip-1821

Actions #5

Updated by Sage Weil about 12 years ago

  • Status changed from New to Resolved

we could still add refcounting to the CephContext later.

Actions

Also available in: Atom PDF