Project

General

Profile

Actions

Bug #5979

closed

librados: imposes internal tooling expectations on users

Added by Greg Farnum over 10 years ago. Updated over 10 years ago.

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

0%

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

Description

gregf@kai:~/ceph/examples/librados [(no branch)]$ ./librados_hello_world -c ../../src/ceph.conf
we just set up a rados cluster object
unable to find any monitors in conf. please specify monitors via -m monaddr or -c ceph.conf
couldn't connect to cluster! error -2

This is from the program I'm working on for #5905, and while I wrote "we just set up a rados cluster object" and "couldn't connect to cluster! error -2", the output "unable to find any monitors in conf. please specify monitors via -m monaddr or -c ceph.conf" is generated in the depths of our init code from a simple call to rados.connect(). That's not really appropriate for other applications which might use totally different option syntax.

I'm not sure if this is best handled as a single error string that needs to be dealt with, or if we need to re-examine the librados interfaces systematically for suitability with users outside of the Ceph core projects.

Actions #1

Updated by Sage Weil over 10 years ago

  • Assignee set to Greg Farnum

on second thought, i think we should just drop the mention of the -m and just say 'no monitors specified' or something generic. problem solved!

Actions #2

Updated by Greg Farnum over 10 years ago

  • Status changed from New to Fix Under Review

wip-5988, f5636be742bffb19f16fdb832891fd1a43679ccf. Pull request 512.

Actions #3

Updated by Greg Farnum over 10 years ago

  • Status changed from Fix Under Review to Resolved

Merged into master.

Actions

Also available in: Atom PDF