doc: add-or-rm-mons.rst: better explanation on number of monitors

Signed-off-by: Joao Eduardo Luis <joao.luis@inktank.com>
doc: troubleshooting-mons.rst: better document how to troubleshoot mons

Signed-off-by: Joao Eduardo Luis <joao.luis@inktank.com>
rados.py: add aio_remove

Add aio_remove to the Python wrapper.

Signed-off-by: Dan van der Ster <daniel.vanderster@cern.ch>
Formatter::new_formatter(): pass const function parameter by reference

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
close file descriptor in error case

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
test_o_trunc.c: reduce scope of some variables

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
SyntheticClient::play_trace: reduce scope of local variable

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
ErasureCodePlugin.cc: reduce scope of local variable

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
test_o_trunc.c: fix indentation to kernel style (tabs, 8 spaces)

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
PG::generate_past_intervals: reduce scope of variable

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
PG.h: use !empty() instead of size() > 0

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
KeyValueStore: use !empty() instead of size()

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
ceph_argparse.cc: prefer prefix ++operator for non-trivial iterator

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
ECBackend.cc: use !empty() instead of size()

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
librados.cc: remove unused variable cpp_category

[src/librados/librados.cc:3083]: (style) Variable 'cpp_category' is
 assigned a value that is never used.

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
osd: bracket role changes with scrub reg/unreg

Make sure the reg/unreg calls for putting ourselves in the scrub schedule
bracket any PG role changes, or that initial reg happens after the role
has been initialized.

Signed-off-by: Sage Weil <sage@inktank.com>
osd: only register for scrub if we are the PG primary

There is no reason to put non-primary PGs on the scrub schedule as they
will be ignored when the worker comes around.  This reduces lock
contention and is generally cleaner.

Signed-off-by: Guang Yang <yguang@yahoo-inc.com>
Signed-off-by: Sage Weil <sage@inktank.com>
osd_types.cc: add missing break in operator<< case handling

CID 1086831 (#1 of 1): Missing break in switch (MISSING_BREAK)
 unterminated_case: This case (value 4635) is not terminated by a
 'break' statement.

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
MDSMonitor::prepare_command: fix "mds set"

CID 1160828 (#1 of 1): Operands don't affect result (CONSTANT_EXPRESSION_RESULT)
 result_independent_of_operands: !cmd_getval(g_ceph_context, cmdmap,
 std::string("var", std::allocator<char>()), var) < 0 is always false
 regardless of the values of its operands. This occurs as the logical
 operand of if.

Remove '< 0' check, replaced with check if returned string is empty.

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
erasure-code: remove jerasure version 1

http://tracker.ceph.com/issues/7599 refs #7599

Signed-off-by: Loic Dachary <loic@dachary.org>