Paxos.h: fix dangerouse use of c_str()

No need to use c_str() in get_statename(), simply return a
std::strin instead.

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
Client.cc: don't pass c_str() if std::string is expected

Don't pass c_str() to _lookup(). The function expect a std::string
as second parameter.

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
ceph.spec.in: add new files

Add new files to spec file since they get installed.

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
debian: add new files

Add new (installed) files to debian install files.

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
Monitor.cc: fix -Wsign-compare

Fix -Wsign-compare, make 'i' unsigned int.

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
doc: Added verbiage to describe single host deadlocks.

fixes: #3076

Signed-off-by: John Wilkins <john.wilkins@inktank.com>
doc: Moved admonition to kernel mount.

fixes: #4146

Signed-off-by: John Wilkins <john.wilkins@inktank.com>
Merge pull request #70 from dalgaaf/wip-da-fix-spec

Add missing files to spec and debian files

Reviewed-by: Sage Weil <sage@inktank.com>
Merge pull request #72 from dalgaaf/wip-da-comp-sign-unsign

Monitor.cc: fix -Wsign-

Reviewed-by: Sage Weil <sage@inktank.com>
Merge pull request #71 from dalgaaf/wip-da-sca-c_str

fix some c_str() usage

Reviewed-by: Sage Weil <sage@inktank.com>
librbd: drop snap_lock before invalidating cache

Writeback will take the snap_lock, so read everything we need under it
before invalidating the cache. This avoids a recursive lock when writeback
uses snap_lock while snap_rollback() was holding it.

Remove a not-very-useful debugging message that depended on snap_lock being held.

Fixes: #4249
Backport: bobtail
Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
Merge branch 'wip-4249' into wip-4249-master

Make snap_rollback() only take a read lock on snap_lock, since
it does not modify snapshot-related fields.
Conflicts:
	src/librbd/internal.cc
librbd: remove unused internal method

get_snap_size() has been replaced by get_image_size(snap_id) everywhere.

Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
qa: output rados test names as they run

So we don't have to figure out which test is running from the output,
which can be difficult with the system tests.

Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
systest: fix race with pool deletion

The second test have pool deletion and object listing wait on the same
semaphore to connect and start. This led to errors sometimes when the
pool was deleted before it could be opened by the listing process. Add
another semaphore so the pool deletion happens only after the listing
has begun.

Fixes: #4147
Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
systest: restrict list error acceptance

Only ignore errors after the midway point if the midway_sem_post is
defined.

Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
doc: Updated osd configuration reference.

Signed-off-by: John Wilkins <john.wilkins@inktank.com>
doc: Added a small ref section for osd config reference.

Signed-off-by: John Wilkins <john.wilkins@inktank.com>
test_rbd: close image before removing it

This error was masked before by watch notify not differentiating
between watches from the same client with different cookies.
Reopen the image at the end of this test so teardown works.

Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
qa: enable watch-notify dependent test

This works now that watch-notify has been reworked a bit.

Signed-off-by: Josh Durgin <josh.durgin@inktank.com>