| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
| |
This reverts commit 1e6c390a67b2500d5a934b6537b8c7e9e1f096c6.
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Stopping the flusher is essentially the shutdown step for the
ObjectCacher - the next thing is actually destroying it.
If we leave any reads outstanding, when they complete they will
attempt to use the now-destroyed ObjectCacher. This is particularly a
problem with rbd images, since an -ENOENT can instantly complete many
readers, so the upper layers don't wait for the other rados-level
reads of that object to finish before trying to shutdown the cache.
Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
|
|\
| |
| |
| | |
Reviewed-by: Samuel Just <sam.just@inktank.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
After Monitor::init_paxos() has loaded all of the PaxosService state,
we should then map creating pgs to osds. This ensures we do so after the
osdmap has been loaded and the pgs actually map somewhere meaningful.
Fixes: #4675
Signed-off-by: Sage Weil <sage@inktank.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This avoids calculating new pg creation mappings if the osdmap isn't
loaded yet, which currently happens when during Monitor::paxos_init()
on startup. Assuming osdmap epoch is nonzero, it should always be
safe to do this (although possibly unnecessary).
More cleanup here is certainly possible, but this is one step toward fixing
the bad behavior for #4675.
Signed-off-by: Sage Weil <sage@inktank.com>
|
|/
|
|
|
|
|
| |
Factor out the portion of the function that remaps creating pgs to osds
from the part that sends those pending creates out.
Signed-off-by: Sage Weil <sage@inktank.com>
|
|\
| |
| |
| |
| | |
Fixes: #4836
Reviewed-by: Sage Weil <sage@inktank.com>
|
| |
| |
| |
| |
| | |
Fixes: #4836
Signed-off-by: Samuel Just <sam.just@inktank.com>
|
| |
| |
| |
| |
| |
| |
| | |
Otherwise, we just duplicate the logic for generating the version
key names.
Signed-off-by: Samuel Just <sam.just@inktank.com>
|
| |
| |
| |
| | |
Signed-off-by: Samuel Just <sam.just@inktank.com>
|
| |
| |
| |
| |
| |
| |
| | |
Use Context::complete() to finish context, it frees the context
after executing Context::finish().
Signed-off-by: Yan, Zheng <zheng.z.yan@intel.com>
|
| |
| |
| |
| | |
Signed-off-by: John Wilkins <john.wilkins@inktank.com>
|
| |
| |
| |
| | |
Signed-off-by: John Wilkins <john.wilkins@inktank.com>
|
| |
| |
| |
| | |
Signed-off-by: John Wilkins <john.wilkins@inktank.com>
|
| |
| |
| |
| | |
Signed-off-by: John Wilkins <john.wilkins@inktank.com>
|
| |
| |
| |
| | |
Signed-off-by: John Wilkins <john.wilkins@inktank.com>
|
| |
| |
| |
| | |
Signed-off-by: John Wilkins <john.wilkins@inktank.com>
|
| |
| |
| |
| | |
Signed-off-by: John Wilkins <john.wilkins@inktank.com>
|
| |
| |
| |
| | |
Signed-off-by: John Wilkins <john.wilkins@inktank.com>
|
| |
| |
| |
| | |
Signed-off-by: John Wilkins <john.wilkins@inktank.com>
|
| |
| |
| |
| | |
Signed-off-by: John Wilkins <john.wilkins@inktank.com>
|
| |
| |
| |
| | |
Signed-off-by: John Wilkins <john.wilkins@inktank.com>
|
|/
|
|
|
|
|
|
|
|
|
| |
tools/ceph-filestore-dump.cc: In member function ‘int header::get_header()’:
warning: tools/ceph-filestore-dump.cc:454:19: comparison between signed and unsigned integer expressions [-Wsign-compare]
tools/ceph-filestore-dump.cc: In member function ‘int footer::get_footer()’:
warning: tools/ceph-filestore-dump.cc:471:19: comparison between signed and unsigned integer expressions [-Wsign-compare]
tools/ceph-filestore-dump.cc: In member function ‘int super_header::read_super()’:
warning: tools/ceph-filestore-dump.cc:697:30: comparison between signed and unsigned integer expressions [-Wsign-compare]
Signed-off-by: Sage Weil <sage@inktank.com>
|
|
|
|
|
|
| |
tcmalloc is available on all supported platforms now.
Signed-off-by: Gary Lowell <gary.lowell@inktank.com>
|
|
|
|
|
|
|
|
| |
Since all currently supported platforms have tcmalloc
available and it is now the default, remove broken check code
that turns it off if the package is not listed in build-depends.
Signed-off-by: Gary Lowell <gary.lowell@inktank.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The monitor may get a series of messages from the OSD that prompt it to
send incremental maps (pg_temp updates, failures, probably more). Avoid
sending the same incremental maps twice by keeping a cache of what epochs
we think the OSDs have.
This reduces monitor load, especially when the mon is a bit behind and is
getting a stream of delayed messages, and the work associated with sending
the inc maps prevents it from catching up.
Signed-off-by: Sage Weil <sage@inktank.com>
Reviewed-by: Greg Farnum <greg@inktank.com>
|
|
|
|
| |
Signed-off-by: Greg Farnum <greg@inktank.com>
|
|
|
|
|
| |
Signed-off-by: Greg Farnum <greg@inktank.com>
Reviewed-by: Sage Weil <sage@inktank.com>
|
|
|
|
| |
Signed-off-by: John Wilkins <john.wilkins@inktank.com>
|
|
|
|
| |
Signed-off-by: John Wilkins <john.wilkins@inktank.com>
|
|
|
|
| |
Signed-off-by: John Wilkins <john.wilkins@inktank.com>
|
|
|
|
| |
Signed-off-by: John Wilkins <john.wilkins@inktank.com>
|
|
|
|
| |
Signed-off-by: John Wilkins <john.wilkins@inktank.com>
|
|
|
|
| |
Signed-off-by: John Wilkins <john.wilkins@inktank.com>
|
|
|
|
| |
Signed-off-by: John Wilkins <john.wilkins@inktank.com>
|
|
|
|
| |
Signed-off-by: John Wilkins <john.wilkins@inktank.com>
|
|
|
|
| |
Signed-off-by: John Wilkins <john.wilkins@inktank.com>
|
|
|
|
| |
Signed-off-by: John Wilkins <john.wilkins@inktank.com>
|
|
|
|
| |
Signed-off-by: John Wilkins <john.wilkins@inktank.com>
|
|
|
|
| |
Signed-off-by: John Wilkins <john.wilkins@inktank.com>
|
|
|
|
| |
Signed-off-by: John Wilkins <john.wilkins@inktank.com>
|
|\
| |
| |
| | |
Reviewed-by: Sam Just <sam.just@inktank.com>
|
| |
| |
| |
| |
| |
| |
| |
| | |
Check for up OSDs as we walk up the crushmap hierarchy
fixes: #4822
Signed-off-by: David Zafman <david.zafman@inktank.com>
|
| |
| |
| |
| |
| |
| |
| | |
This is a safe range to do comparisons against, and we compare
against the signed rank inside the loop.
Signed-off-by: Greg Farnum <greg@inktank.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
We try and select a random monitor first, but if that fails we should
make sure that nobody's available before asserting.
Fixes #4812
Signed-off-by: Greg Farnum <greg@inktank.com>
Reviewed-by: Sage Weil <sage@inktank.com>
|
|\ \ |
|
| | |
| | |
| | |
| | |
| | |
| | | |
Casting.
Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
There was an issue when limit was being set, we didn't
break from the iterating loop if limit was reached. Also,
S3 does not enforce any limit, so keep that behavior.
Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
|
| | |
| | |
| | |
| | |
| | |
| | | |
Bucket listing was broken when limit was set.
Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
In order to keep compatibility with swift, if a plain formatter
is being used, we should return 204 when there are no containers.
Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
|