mon: OSDMonitor: return immediately if 'osd tier cache-mode' is a no-op

Signed-off-by: Joao Eduardo Luis <joao.luis@inktank.com>
crush/mapper.c: fix printf format for unsigned variable

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
KeyValueStore: rename s/logger/perf_logger/

[src/os/KeyValueStore.h:368] -> [src/os/ObjectStore.h:100]: (warning) The class
 'KeyValueStore' defines member variable with name 'logger' also defined in its
 parent class 'ObjectStore'.

KeyValueStore logger is of type PerfCounters while ObjectStore logger is Logger.
Change the name in KeyValueStore to more obvious name perf_logger.

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

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

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
hitset.cc: fix format string to unsigned int

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
SimpleLock.h: fix bool comparison using relational operator

[src/mds/SimpleLock.h:287]: (warning) Comparison of a boolean value
 using relational operator (<, >, <= or >=).

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
buffer.cc: catch exception by reference

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

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
CDentry.cc: fix bool comparison using relational operator

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
ObjectStore.h: pass const string parameter by reference

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
rgw_main.cc: use static_cast instead of c-style

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

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

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

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

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

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

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
rgw_bucket.cc: return error if update_containers_stats() fails

In case need_stats is set on rgw_read_user_buckets() and the
update_containers_stats() call fails with !-ENOENT, not only
print out a message but also return the error to the function
caller.

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
Dumper::dump_entries(): remove not needed variable

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