Merge pull request #1338 from dachary/wip-mailmap

mailmap updates

Reviewed-by: Christophe Courtaut <christophe.courtaut@gmail.com>
crush: revise JSON format for 'item' type

Commit a7e9a7b648 changed the JSON format of CRUSH rules
such that the 'item' attribute on a step was sometimes
an integer and sometimes a string.

This commit separates the integer and string representations
so that tools which rely on a 'item' consistently being an
integer ID will work.

Signed-off-by: John Spray <john.spray@inktank.com>
Merge pull request #1373 from ceph/wip-crush-json

crush: revise JSON format for 'item' type

Reviewed-by: Sage Weil <sage@inktank.com>
test: Use non-default out/ dev/ paths in vstart

Currently, running "make check" nukes any existing vstart
setup in the tree.  This change makes it use test_dev/ and test_out/
so that it can peacefully coexist.

Signed-off-by: John Spray <john.spray@inktank.com>
mon/PGMap: return empty stats if pool is not in sum

Greg was right!

When a pool is created, the PGs are not added to the PGMap until the *next*
proposal.  Weaken the assert here and return empty stats for non-existent
(new) pools so that a pool create + tier add sequence does not crash.

Signed-off-by: Sage Weil <sage@inktank.com>
Merge pull request #1375 from ceph/wip-pgmap-stat

mon/PGMap: return empty stats if pool is not in sum

Reviewed-by: Greg Farnum <greg@inktank.com>
mon: warn when pool nears target max objects/bytes

The cache pools will throttle when they reach the target max size, so it
is important to make the administrator aware when they approach that point.
Unfortunately it is not particularly easy to efficiently keep track of
which PGs have hit their limit and use that for reporting.  However, it
is easy to raise a flag when we start to approach the target for the
entire pool, and that sort of early warning is arguably more useful
anyway.

Trigger the warning based on the target full ratio.  Not when we hit the
target, but when we are 2/3 between it and completely full.

Implements: #7442
Signed-off-by: Sage Weil <sage@inktank.com>
test: Fix tiering test cases to use ---force-nonempty

Fixes: #7608

Signed-off-by: David Zafman <david.zafman@inktank.com>
Merge pull request #1376 from ceph/wip-7608

test: Fix tiering test cases to use ---force-nonempty

Reviewed-by: Sage Weil <sage@inktank.com>
qa/workunits/mon/crush_ops: use expect_false

Signed-off-by: Sage Weil <sage@inktank.com>
FileStore::_collection_move_rename: propogate EEXIST

Previously, an EEXIST would get masked by the subsequent clone
operation.

Signed-off-by: Samuel Just <sam.just@inktank.com>
PGBackend::rollback_stash: remove the correct shard

Fixes: #7616
Signed-off-by: Samuel Just <sam.just@inktank.com>
mon/OSDMonitor: disallow crush buckets of type 0

Prevent creation of buckets of type 0 ('osd', 'device', etc.), as they
will confusing the mapping algorithm.

Signed-off-by: Sage Weil <sage@inktank.com>
test: merge unittest_crushwrapper and unittest_crush_wrapper

Signed-off-by: Sage Weil <sage@inktank.com>
Merge pull request #1351 from ceph/wip-7248

osd: OSD: limit the value of 'size' and 'count' on 'osd bench'

Reviewed-by: Sage Weil <sage@inktank.com>
Merge pull request #1372 from ceph/wip-7607

wip 7607

Reviewed-by: Sage Weil <sage@inktank.com>
Merge pull request #1343 from ceph/wip-cache-warn-full

mon: warn when cache tier is full

Reviewed-by: Loic Dachary <loic@dachary.org>
Reviewed-by: Greg Farnum <greg@inktank.com>
mon: make quorum list (by name) be in quorum order

Fixes: #7562
Signed-off-by: Sage Weil <sage@inktank.com>
Revert "c_read_operations.cc: fix resource leak"

This reverts commit 3cd751b0a280909510c3e633cc8cd4b9f5e3b2d9.

A rados_release_read_op() has already been performed, but coverity
didn't recognize that as releasing memory.

Fixes: #7621

Signed-off-by: David Zafman <david.zafman@inktank.com>
Reviewed-by: Samuel Just <sam.just@inktank.com>
ReplicatedPG::recover_replicas: do not assume that missing objects are unfound

Fixes: #7618
Signed-off-by: Samuel Just <sam.just@inktank.com>