| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
| |
Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>
|
|
|
|
| |
Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>
|
|
|
|
| |
Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>
|
|
|
|
| |
Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>
|
|
|
|
| |
Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>
|
|
|
|
| |
Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>
|
|
|
|
| |
Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>
|
|
|
|
| |
Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>
|
|
|
|
| |
Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>
|
|
|
|
|
|
|
|
|
| |
1) change python to the virtualenv version, which means
2) we can avoid setting sys.path directly
Also:
3) remove the "DEVELOPER MODE" message
Signed-off-by: Dan Mick <dan.mick@inktank.com>
|
|
|
|
| |
Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>
|
|
|
|
| |
Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>
|
|
|
|
| |
Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>
|
|
|
|
| |
Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>
|
|
|
|
| |
Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>
|
|
|
|
| |
Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>
|
|
|
|
| |
Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>
|
|
|
|
| |
Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>
|
|
|
|
| |
Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>
|
|
|
|
| |
Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>
|
|
|
|
| |
Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>
|
|
|
|
| |
Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>
|
|
|
|
| |
Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>
|
|
|
|
| |
Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>
|
|
|
|
| |
Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>
|
|
|
|
| |
Signed-off-by: Alfredo Deza <alfredo@deza.pe>
|
|
|
|
| |
Signed-off-by: Alfredo Deza <alfredo@deza.pe>
|
|
|
|
| |
Signed-off-by: Alfredo Deza <alfredo@deza.pe>
|
|\
| |
| |
| | |
Reviewed-by: Sage Weil <sage@inktank.com>
|
| |
| |
| |
| |
| |
| | |
Fixes: #5848
Signed-off-by: David Zafman <david.zafman@inktank.com>
|
|\ \ |
|
| |\ \
| | | |
| | | | |
Reviewed-by: Sage Weil <sage@inktank.com>
|
| |/ /
| | |
| | |
| | |
| | |
| | |
| | | |
Fix by restructuring code to hoist common code and have only one
place where admin_socket is actually called.
Signed-off-by: Dan Mick <dan.mick@inktank.com>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The std::copy method leaves a trailing separator.
Signed-off-by: Sage Weil <sage@inktank.com>
Reviewed-by: Dan Mick <dan.mick@inktank.com>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The std::copy construct leaves a trailing separator character, which breaks
parsing for booleans (among other things) and probably mangles everything
else too.
Backport: dumpling
Signed-off-by: Sage Weil <sage@inktank.com>
Reviewed-by: Samuel Just <sam.just@inktank.com>
|
| | |
| | |
| | |
| | |
| | | |
Signed-off-by: Sage Weil <sage@inktank.com>
Reviewed-by: Dan Mick <dan.mick@inktank.com>
|
| | |
| | |
| | |
| | |
| | |
| | | |
This reverts commit fefe0c602f78e66d35fd5806da4c2e4e154a267c.
I have a cleaner cleanup.
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The std::copy construct leaves a trailing separator character, which breaks
parsing for booleans (among other things) and probably mangles everything
else too.
Backport: dumpling
Signed-off-by: Sage Weil <sage@inktank.com>
Reviewed-by: Samuel Just <sam.just@inktank.com>
|
| |\ \
| | | |
| | | | |
tools: ceph-monstore-tool: copy a store's contents to another store
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Usage:
ceph-monstore-tool --mon-store-path <in-path> --out <out-path> \
--command store-copy
Signed-off-by: Joao Eduardo Luis <joao.luis@inktank.com>
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Check new parameters and check that rados_id is not None again to
catch the empty string.
Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
Reviewed-by: Sage Weil <sage.weil@inktank.com>
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Previously it had no name parameter, so the default will be used by
old clients. However, if an old client set rados_id, a new check that
both rados_id and name are set would result in an error. Fix this by
only applying the default names after the check, and add tests of this
behavior.
This was introduced in 783b7ec847c7f987ac1814c9c41c91921cac4eba,
so it does not affect cuttlefish.
Fixes: #5970
Reported-by: Michael Morgan <mmorgan@dca.net>
Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
Reviewed-by: Sage Weil <sage.weil@inktank.com>
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
For aio flush, we register a wait on the most recent write. The write
completion code, however, was *only* waking the waiter if they were waiting
on that write, without regard to previous writes (completed or not).
For example, we might have 6 and 7 outstanding and wait on 7. If they
finish in order all is well, but if 7 finishes first we do the flush
completion early. Similarly, if we
- start 6
- start 7
- finish 7
- flush; wait on 7
- finish 6
we can hang forever.
Fix by doing any completions that are prior to the oldest pending write in
the aio write completion handler.
Refs: #5919
Signed-off-by: Sage Weil <sage@inktank.com>
Reviewed-by: Josh Durgin <josh.durgin@inktank.com>
Tested-by: Oliver Francke <Oliver.Francke@filoo.de>
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Add an already-locked helper so that C_Aio{Safe,Complete} can
increment the reference count when their caller holds the
lock. C_AioCompleteAndSafe's caller is not holding the lock, so call
regular get() to ensure no racing updates can occur.
This eliminates all direct manipulations of AioCompletionImpl->ref,
and makes the necessary locking clear.
The only place C_AioCompleteAndSafe is used is in handling
aio_flush_async(). This could cause a missing completion.
Refs: #5919
Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
Reviewed-by: Sage Weil <sage@inktank.com>
Tested-by: Oliver Francke <Oliver.Francke@filoo.de>
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
Bunch of tidying up on monitor services & fix #4635
Reviewed-by: Sage Weil <sage@inktank.com>
|
| | | | |
| | | | |
| | | | |
| | | | | |
Signed-off-by: Joao Eduardo Luis <joao.luis@inktank.com>
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
'osd crush set' should only be used to update already existing items on
the map whereas 'osd crush add' should be able to 'add and update' items.
Considering at that point we are effectively adding a new item to the
crush map, use 'add' instead of 'set'.
Signed-off-by: Joao Eduardo Luis <joao.luis@inktank.com>
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Fixes: #4635
Signed-off-by: Joao Eduardo Luis <joao.luis@inktank.com>
|
| | | | |
| | | | |
| | | | |
| | | | | |
Signed-off-by: Joao Eduardo Luis <joao.luis@inktank.com>
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Fixes: #4635
Signed-off-by: Sage Weil <sage@inktank.com>
Reviewed-by: Joao Eduardo Luis <joao.luis@inktank.com>
|