rgw-multisite: metadata conflict not computed correctly

The former logic with a conditional based on `++i == 0` would never
execute. So this uses a boolean to differentiate the first from other
iterations and tries to clarify the code through commenting and an
explicit declaration. Additionally a warning is eliminated by
initializing a variable.

Signed-off-by: J. Eric Ivancich <ivancich@redhat.com>

Merge pull request #42792 from tchaikov/wip-librbd-cleanups

librbd: trade a map<> for a plain array

Reviewed-by: Ilya Dryomov <idryomov@gmail.com>

Merge pull request #42855 from josephsawaya/rook-label

.github/labeler: add rook label to PRs related to Rook

Reviewed-by: Kefu Chai <kchai@redhat.com>
Reviewed-by: Sebastian Wagner <sewagner@redhat.com>

Merge pull request #42813 from wjwithagen/wjw-fix-hexdump-output

blk: start 1st line of hexdump() on a new line

Reviewed-by: Kefu Chai <tchaikov@gmail.com>

Merge pull request #42856 from dang/wip-dang-zipper-manifest

RGW - Zipper - Remove RGWManifest from API

Reviewed-by: Casey Bodley <cbodley@redhat.com>

Merge pull request #41522 from Dreamyu20xx/wip-50967

rgw: Get the correct initial value(pos) in raw_obj_to_obj function when zoneid contains ‘_ ’character.

Reviewed-by: J. Eric Ivancich <ivancich@redhat.com>

Merge pull request #42723 from liewegas/fix-rgw-warning-2

rgw: default auth_client_required=cephx

Reviewed-by: Matt Benjamin <mbenjamin@redhat.com>
Reviewed-by: Casey Bodley <cbodley@redhat.com>

Merge pull request #42803 from joke-lee/sts-memory-leak

rgw: fix sts memory leak

Reviewed-by: Casey Bodley <cbodley@redhat.com>

Merge pull request #42683 from cbodley/wip-52027

rgw: change order of xml elements in ListRoles response

Reviewed-by: Pritha Srivastava <prsrivas@redhat.com>

Merge pull request #42740 from cbodley/wip-52128

cls/cmpomap: empty values are 0 in U64 comparisons

Reviewed-by: Adam C. Emerson <aemerson@redhat.com>
Reviewed-by: Yuval Lifshitz <ylifshit@redhat.com>

.github/labeler: add rook label to PRs related to Rook

To make it easier to look for PRs directly related to the Rook orchestrator.

Signed-off-by: Joseph Sawaya <jsawaya@redhat.com>

Merge pull request #42863 from sebastian-philipp/centos_8.2_container_tools_3.0.yaml

qa/distro: Add centos_8.2_container_tools_3.0.yaml

Reviewed-by: Patrick Seidensal <pseidensal@suse.com>

qa/distro: Add centos_8.2_container_tools_3.0.yaml

Let's avoid latest kubic stable

Fixes: https://tracker.ceph.com/issues/52279
Signed-off-by: Sebastian Wagner <sewagner@redhat.com>

Merge pull request #42842 from ideepika/wip-werror-testing

run-make-check.sh: fix run-make-check.sh skipping cmake options

Signed-off-by: Deepika Upadhyay <dupadhya@redhat.com>

blk: start 1st line of hexdump() on a new line

Otherwise the fist line looks rather strange

```
2021-08-19T23:42:33.604+0200 4f60700 40 bdev:kd:965(0x4fab180 /usr/local/src/wip.bluestore-test/build/dev/osd0/block.db) data: 00000000  56 f9 b8 f8 1c 00 01 01  1a 6c 65 76 65 6c 64 62  |V........leveldb|
00000010  2e 42 79 74 65 77 69 73  65 43 6f 6d 70 61 72 61  |.BytewiseCompara|
00000020  74 6f 72 98 af 58 a6 02  00 01 02 00 d2 c7 3c 95  |tor..X........<.|
00000030  06 00 01 09 00 03 04 04  00 6b 93 6d c5 2b 00 01  |.........k.m.+..|
00000040  01 1a 6c 65 76 65 6c 64  62 2e 42 79 74 65 77 69  |..leveldb.Bytewi|
.....
```

versus new:
```
2021-08-19T23:42:33.604+0200 4f60700 40 bdev:kd:965(0x4fab180 /usr/local/src/wip.bluestore-test/build/dev/osd0/block.db) data:
00000000  56 f9 b8 f8 1c 00 01 01  1a 6c 65 76 65 6c 64 62  |V........leveldb|
00000010  2e 42 79 74 65 77 69 73  65 43 6f 6d 70 61 72 61  |.BytewiseCompara|
00000020  74 6f 72 98 af 58 a6 02  00 01 02 00 d2 c7 3c 95  |tor..X........<.|
00000030  06 00 01 09 00 03 04 04  00 6b 93 6d c5 2b 00 01  |.........k.m.+..|
00000040  01 1a 6c 65 76 65 6c 64  62 2e 42 79 74 65 77 69  |..leveldb.Bytewi|
.....
```

Signed-off-by: Willem Jan Withagen <wjw@digiware.nl>

Merge pull request #42852 from sebastian-philipp/centos_8.3_container_tools_3.0.yaml

qa/cephadm: centos_8.3_container_tools_3.0.yaml

Reviewed-by: Josh Durgin <jdurgin@redhat.com>

Merge pull request #41950 from zenomri/rgw-crypt-dpp

src/rgw : ldout and lderr calls were replaced by ldpp_dout calls

Reviewed-by: Ali Maredia <amaredia@redhat.com>

RGW - Zipper - Remove StatOp

As part of removing RGWObjManifest from the Zipper API, we need to
remove StatOp.  It's only used by orphan list, which is part of the
RadosStore.

Signed-off-by: Daniel Gryniewicz <dang@redhat.com>

RGW - Zipper - Remove WriteOp

As part of removing RGWObjManifest from the Zipper API, we need to remove
WriteOp.  Fortunately, with the multipart upload changes, it's no longer
needed outside the RadosStore.

Signed-off-by: Daniel Gryniewicz <dang@redhat.com>

Merge pull request #42851 from dang/wip-dang-zipper-raw_obj

RGW - Zipper remove unused files

Reviewed-by: Casey Bodley <cbodley@redhat.com>

Merge pull request #42834 from josephsawaya/rook-orch-qa

qa/tasks/rook: add `ceph orch device ls` command to the rook qa task

Merge pull request #42757 from josephsawaya/wip-mgr-rook-osd-creation

mgr/rook: Rook orchestrator OSD creation using `ceph orch apply osd`

qa/cephadm: centos_8.3_container_tools_3.0.yaml

Let's avoid latest stable

Fixes: https://tracker.ceph.com/issues/52279
Signed-off-by: Sebastian Wagner <sewagner@redhat.com>

RGW - Zipper remove unused files

Signed-off-by: Daniel Gryniewicz <dang@redhat.com>

Merge pull request #42796 from dang/wip-dang-zipper-raw_obj

RGW - Remove last rgw_raw_obj from zipper

Reviewed-by: Soumya Koduri <skoduri@redhat.com>
Reviewed-by: Casey Bodley <cbodley@redhat.com>

run-make-check: fix do_cmake not consuming run-make-check opts

run-make-check.sh uses run-make.sh to `prepare`(install dependencies)
and `configure` cmake options, without quotes these options containing
special characters(hypens mostly) are skipped, hence we see not all
options supplied at cmake configure step.

Resolves(focused to solve issues in jenkins build env):
 - missing cmake options:
    cmake_opts+=" -DCMAKE_CXX_COMPILER=$cxx_compiler -DCMAKE_C_COMPILER=$c_compiler"
    cmake_opts+=" -DCMAKE_CXX_FLAGS_DEBUG=\-Werror"
 - Ninja not being used as cmake generator

Signed-off-by: Deepika Upadhyay <dupadhya@redhat.com>

Merge pull request #42793 from sebastian-philipp/dashboard-fix-standby-redirect

pybind/mgr: Fix IPv6 url generation

Merge pull request #42302 from myoungwon/wip-51000

osd: fix to allow inc manifest leaked

Reviewed-by: Samuel Just <sjust@redhat.com>

Merge pull request #42820 from ronen-fr/wip-ronenf-mgr-argvvec

common: create a faster & cleaner alternative to argv_to_vec()

Reviewed-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
Reviewed-by: Kefu Chai <kchai@redhat.com>

Merge pull request #42823 from mgfritch/ceph-volume-py3-exec

cmake: s/Python_EXECUTABLE/Python3_EXECUTABLE/

Reviewed-by: Kefu Chai <kchai@redhat.com>

Merge pull request #42799 from varshar16/wip-update-email-id

.githubmap: update mail address

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>

Merge pull request #42835 from ceph/wip-ctest-fail

run-make-check.sh: Increase failure output log size

Reviewed-by: Kefu Chai <kchai@redhat.com>

Merge pull request #42828 from tchaikov/wip-doc-dev-config

doc/dev/config: update to in sync with latest .yaml.in changes

Reviewed-by: Neha Ojha <nojha@redhat.com>

run-make-check.sh: Increase failure output log size

Fixes: https://tracker.ceph.com/issues/43566
Signed-off-by: David Galloway <dgallowa@redhat.com>

qa/tasks/rook: add device ls command to rook task

This commit adds the device ls command to the rook qa task
since that command should be working from now on.

Signed-off-by: Joseph Sawaya <jsawaya@redhat.com>

RGW - Zipper - remove last rgw_raw_obj from API

Remove the remaining rgw_raw_obj from the API.  This consists of:

- move get/delete raw_obj from Store to RadosStore; it's only needed by
  RadosStore code
- Add a layout formatter to the API
- remove head_obj and get_manifest() from ReadOp
- Remove GCChain (no longer needed)

Signed-off-by: Daniel Gryniewicz <dang@redhat.com>

Fix uninitialized warnings

Signed-off-by: Daniel Gryniewicz <dang@redhat.com>

Merge pull request #42827 from ronen-fr/wip-ronenf-std-fix2

key_value_store: fix missing std

Reviewed-by: Kefu Chai <kchai@redhat.com>

Merge pull request #42825 from hualongfeng/fix_blk_pmem

blk/pmem: Add namespace std for map,string

Reviewed-by: Kefu Chai <kchai@redhat.com>

mgr/cephadm: fix generation of wrong IPv6 urls

Signed-off-by: Sebastian Wagner <sewagner@redhat.com>

mgr/restful: Fix url generation for IPv6 hosts

Signed-off-by: Sebastian Wagner <sewagner@redhat.com>

mgr/prometheus: Fix url generation for IPv6 hosts

Signed-off-by: Sebastian Wagner <sewagner@redhat.com>

mgr/dashboard: Fix redirect to active MGR

Fixes: https://tracker.ceph.com/issues/52117
Signed-off-by: Sebastian Wagner <sewagner@redhat.com>

pybind/mgr: move build_url from dashboard to mgr_util

Signed-off-by: Sebastian Wagner <sewagner@redhat.com>

Merge pull request #42690 from adk3798/offline2

mgr/cephadm: make scheduler able to accomodate offline/maintenance hosts

Reviewed-by: Sebastian Wagner <sewagner@redhat.com>
Reviewed-by: Cory Snyder <csnyder@iland.com>

doc/dev/config: update to in sync with latest .yaml.in changes

we migrated to a build process which translate the options from .yaml.in
files. so update the document to reflect this change.

Signed-off-by: Kefu Chai <kchai@redhat.com>

Merge pull request #42592 from vfreex/cephadm-better-port-stripping

cephadm: better port stripping

Reviewed-by: Sebastian Wagner <sewagner@redhat.com>

Merge pull request #42590 from vfreex/cephadm-propagate-env-vars

cephadm: propagate environment variables to subprocesses

Merge pull request #42589 from vfreex/fix-cephadm-ipv6

cephadm: Fix bootstrap error with IPv6 mon-ip

Reviewed-by: Sebastian Wagner <sewagner@redhat.com>

Merge pull request #42816 from sebastian-philipp/cephadm-upgrade-status-message

mgr/cephadm/upgrade: Upgrade status: We also upgrade non-ceph daemons

Reviewed-by: Adam King <adking@redhat.com>
Reviewed-by: Dimitri Savineau <dsavinea@redhat.com>

key_value_store: fix missing std

following the removal of 'using std' in PR #42742, and missed
in PR #42790.

Signed-off-by: Ronen Friedman <rfriedma@redhat.com>

Merge pull request #41968 from CongMinYin/fix-ssd-bug

librbd/cache/pwl/ssd: fix m_bytes_allocated exceeding m_bytes_allocated_cap

Reviewed-by: Ilya Dryomov <idryomov@gmail.com>

Merge pull request #42768 from s0nea/wip-dashboard-suppressed-alerts

mgr/dashboard: don't notify for suppressed alerts

blk/pmem: Add namespace std for map,string

Due to not use namespace std for map,string, it leads to
`map,string` not be declared.

Signed-off-by: Feng Hualong <hualong.feng@intel.com>

Merge pull request #42758 from liewegas/gh-template

.github/pull_request_template: fix dashboard e2e test command

Reviewed-by: Kefu Chai <kchai@redhat.com>

Merge pull request #42801 from ivancich/wip-update-rados-rm-help

doc: clarify use of `rados rm` command

Reviewed-by: Kefu Chai <kchai@redhat.com>
Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
Reviewed-by: Neha Ojha <nojha@redhat.com>

Merge pull request #42822 from tchaikov/wip-drop-gdbm

ceph.spec.in: drop gdbm from build deps

Reviewed-by: Ken Dreyer <kdreyer@redhat.com>
Reviewed-by: Ilya Dryomov <idryomov@gmail.com>

tools/setup-virtualenv: verify the `--python` param

when given an invalid or empty (PYTHON_BINARY)
fixup for 494fea50ced7

Fixes: https://tracker.ceph.com/issues/52304
Signed-off-by: Michael Fritch <mfritch@suse.com>

cmake: s/Python_EXECUTABLE/Python3_EXECUTABLE/

pass the python3 exec when creating the ceph-volume build venv
fixup for 5fc657b40dc7

Fixes: https://tracker.ceph.com/issues/52304
Signed-off-by: Michael Fritch <mfritch@suse.com>

mgr/rook: make use of SizeMatcher in Creator classes

This commit uses the SizeMatcher Class in the Creator functions
to parse and filter devices according to the size specified in a
drive_group.

Signed-off-by: Joseph Sawaya <jsawaya@redhat.com>

common: create a faster & cleaner alternative to argv_to_vec()

New function signature follows current C++ guidance regarding
returning values from functions. It is also faster
- 1.3X for a sample test of 10 arguments.

Signed-off-by: Ronen Friedman <rfriedma@redhat.com>
Co-authored-by: Kefu Chai <tchaikov@gmail.com>

ceph.spec.in: drop gdbm from build deps

neither gdbm nor libgdbm is used when building ceph, so let's drop it.

Signed-off-by: Kefu Chai <kchai@redhat.com>

mgr/rook: fix convert_size method in DefaultFetcher

This commit fixes the convert_size method by getting it
to use the re python module to split the digits and letters
to support all units a PV could be expressed in.

Signed-off-by: Joseph Sawaya <jsawaya@redhat.com>

mgr/rook: create get_storage_class method on RookCluster

this commit creates the get_storage_class method on the RookCluster
class used to get the storage class matching the name provided
in the ceph config.

Signed-off-by: Joseph Sawaya <jsawaya@redhat.com>

mgr/rook: apply_drivegroups fix and coding style fixes

This commit fixes the apply_drivegroups method in RookOrchestrator
to process the entire list of drive groups passed.

This commit also fixes some coding style errors in RookCluster.

Signed-off-by: Joseph Sawaya <jsawaya@redhat.com>

mgr/rook: keep drive groups updated and clean up cluster CR

This commit creates a new threaded methods on the RookCluster class
that keeps the cluster updated by re-applying drive groups in a
loop.

Signed-off-by: Joseph Sawaya <jsawaya@redhat.com>

mgr/rook: OSD creation using Creator classes

This commit implements the apply_drivegroups method in the
RookOrchestrator class and creates the DefaultCreator and
LSOCreator classes that handle creating OSDs. The add_osds
method in RookCluster will use each creator based on what
storage class the user provided in the ceph config.

Signed-off-by: Joseph Sawaya <jsawaya@redhat.com>

mgr/rook: add supplemental information to device object in LSOFetcher

This commit adds the PV name, node name, vendor and model to the Device object
created in the LSOFetcher device method, this information is useful for adding
OSDs.

Signed-off-by: Joseph Sawaya <jsawaya@redhat.com>

Merge pull request #42738 from sebastian-philipp/mypy-3.7

pybind/mgr: tox: also test mypy with python 3.7

Reviewed-by: Kefu Chai <kchai@redhat.com>

test: allowing >= the real refcount to avoid false alarm

Current dedup allow to contain multiple same sources using
multiset, which results in inconsistent situation as follow
(during set_chunk, but not confined in set_chunk).

1. User issues set_chunk
2. OSD receives the set_chunk, and sends increment message
to an object in the low tier (INPROGRESS).
3. OSD map is changed (841 → 843)
3.5. on_change() is called
4. the set_chunk op is reenqueued by requeue_op()
5. OSD handles the duplicated set_chunk, but it is not able to
 know the set_chunk is duplicated because it does not log on the disk yet.
6. OSD issues increment message again to the object
in the low tier. (increment operation is executed twice)

To fix this, this commit allows >= the real refcount in test cases

fixes: https://tracker.ceph.com/issues/51000

Signed-off-by: Myoungwon Oh <myoungwon.oh@samsung.com>

Merge pull request #42811 from tchaikov/wip-dashboard-test

mgr/dashboard/api: set a UTF-8 locale when running pip

Reviewed-by: Avan Thakkar <athakkar@redhat.com>

Merge pull request #42732 from rhcs-dashboard/bug-1950644

mgr/dashboard: disable create snapshot with subvolumes

Merge pull request #42717 from rhcs-dashboard/iscsi-target-refresh-btn

mgr/dashboard: Refresh button on the iscsi targets page

mgr/cephadm/upgrade: Upgrade status: We also upgrade non-ceph daemons

Signed-off-by: Sebastian Wagner <sewagner@redhat.com>

librbd/cache/pwl/ssd: ensure bdev and pool sizes match

m_log_pool_size should be multiple of 1M but, just in case, prevent
is_valid_io() assert in KernelDevice::aio_write().

Signed-off-by: Ilya Dryomov <idryomov@gmail.com>

Merge pull request #42786 from aaryanporwal/visual-tests

mgr/dashboard: visual tests: Add more ignore regions for dashboard component

mgr/dashboard: disable create snapshot with subvolumes

Signed-off-by: Pere Diaz Bou <pdiazbou@redhat.com>

Merge pull request #42808 from tchaikov/wip-crimson-test-cleanup

test/crimson: do not discard returned future<>

Reviewed-by: Samuel Just <sjust@redhat.com>
Reviewed-by: Radoslaw Zarzynski <rzarzyns@redhat.com>

mgr/dashboard/api: set a UTF-8 locale when running pip

ansible-core started to include files whose filenames are encoded in
non-ascii characters, so we have to use a more capable encoding for the
locale in order to install this package. otherwise we'd have following
error:

Collecting ansible-core<2.12,>=2.11.3
  Using cached ansible-core-2.11.4.tar.gz (6.8 MB)
ERROR: Exception:

Traceback (most recent call last):
  File "/tmp/tmp.fX76ASIrch/venv/lib/python3.8/site-packages/pip/_internal/cli/base_command.py", line 173, in _main
    status = self.run(options, args)
...
  File "/tmp/tmp.fX76ASIrch/venv/lib/python3.8/site-packages/pip/_internal/utils/unpacking.py", line 226, in untar_file
    with open(path, "wb") as destfp:
UnicodeEncodeError: 'latin-1' codec can't encode characters in position 137-140: ordinal not in range(256)

Signed-off-by: Kefu Chai <kchai@redhat.com>

Merge pull request #42807 from tchaikov/wip-crimson-test-more-chatty

test/crimson: re-enable logging output

Reviewed-by: Yingxin Cheng <yingxin.cheng@intel.com>
Reviewed-by: Xuehan Xu <xuxuehan@360.cn>

Merge pull request #42791 from ideepika/wip-50947-blk-cmake

src/blk: fix block_device_t return if no aio libs present

Reviewed-by: Kefu Chai <kchai@redhat.com>

Merge pull request #42800 from tchaikov/wip-cmake-debug-build

cmake: set CMAKE_BUILD_TYPE to Debug when .git exists

Reviewed-by: Deepika Upadhyay <dupadhya@redhat.com>
Reviewed-by: Xiubo Li <xiubli@redhat.com>

test/crimson: do not discard returned future<>

to silence following warning:

../src/test/crimson/seastore/test_randomblock_manager.cc:122:7: warning: ignoring return value of function declared with 'nodiscard' attribute [-Wunused-result]
      rbm_manager->add_free_extent(t.allocated_blocks, r.first * DEFAULT_BLOCK_SIZE,
      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Signed-off-by: Kefu Chai <kchai@redhat.com>

test/crimson: use structured binding

for better readability

Signed-off-by: Kefu Chai <kchai@redhat.com>

Merge pull request #42798 from xxhdx1985126/wip-52275

crimson/common: allow the tls interrupt_cond to exist when a continuation starts to run

Reviewed-by: Samuel Just <sjust@redhat.com>
Reviewed-by: Chunmei Liu <chunmei.liu@intel.com>
Reviewed-by: Kefu Chai <kchai@redhat.com>

test/crimson: re-enable logging output

this change reverts 57877075c0b6885c7e35dc4c4a881b140c30cfb0

per Yingxin's request, we should enable it for better debugging
experience when the test fails.

Signed-off-by: Kefu Chai <kchai@redhat.com>

Merge pull request #42787 from rzarzynski/wip-crimson-fix-warn-storenbd

crimson/tools: silence [[maybe_unused]] warning in store-nbd.cc.

Reviewed-by: Samuel Just <sjust@redhat.com>
Reviewed-by: Kefu Chai <kchai@redhat.com>

.githubmap: update mail address

Signed-off-by: Varsha Rao <rvarsha016@gmail.com>

Revert "crimson/onode-staged-tree: force test work with invalidated transactions"

This reverts commit 8b6428c4ac08a7d3beabf64f549bfe11e5245978.

Signed-off-by: Yingxin Cheng <yingxin.cheng@intel.com>

test/crimson: avoid creating interrupt condition within an already interruptible continuation chain

Signed-off-by: Xuehan Xu <xxhdx1985126@gmail.com>

crimson/common: assert the existence of interrupt_cond even in release builds

Signed-off-by: Xuehan Xu <xxhdx1985126@gmail.com>

crimson/common: allow the tls interrupt_cond to exist when a continuation starts to run

Release build of seastar invokes continuations synchronously if their predecessor futures
are resolved at the time when those continuations are constructed, so we have to allow
these circumstances to happen

Fixes: https://tracker.ceph.com/issues/52275
Signed-off-by: Xuehan Xu <xxhdx1985126@gmail.com>

Merge pull request #42658 from tchaikov/wip-doc-build

admin/build-doc: rebuild venv if md5 does not match

Reviewed-by: Deepika Upadhyay <dupadhya@redhat.com>

rgw: fix sts memory leak

fix https://tracker.ceph.com/issues/52290

Signed-off-by: yuliyang_yewu <yuliyang_yewu@cmss.chinamobile.com>

Merge PR #42749 into master

* refs/pull/42749/head:
doc: document that NEARFULL conditions can result in slow writes

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>

Merge pull request #42797 from rzarzynski/wip-crimson-no-prometheus-by-default

crimson/osd: disable Premetheous by default.

Reviewed-by: Chunmei Liu <chunmei.liu@intel.com>
Reviewed-by: Kefu Chai <kchai@redhat.com>

cmake: set CMAKE_BUILD_TYPE to Debug when .git exists

if CMAKE_BUILD_TYPE is an empty string, we should also set it to Debug.

Signed-off-by: Kefu Chai <kchai@redhat.com>

erasure-code/clay: do not set unreferenced variable

plane_count was introduced by 2a3e56b22c5b52e4834a9fd5047b3cdff7e8859f,
and presumably it is only used for debugging. let's drop it as it is not
referenced in production.

this change silences following warning:

../src/erasure-code/clay/ErasureCodeClay.cc:502:7: error: variable 'plane_count' set but not used [-Werror,-Wunused-but-set-variable]
  int plane_count = 0;
      ^

Signed-off-by: Kefu Chai <kchai@redhat.com>

erasure-code/lrc: early return if to_string() fails

so far to_string() always return 0, but it is designed to return
an error code if it fails to parse a string.

this change silences following warning:

../src/erasure-code/lrc/ErasureCodeLrc.cc:399:7: error: variable 'err' set but not used [-Werror,-Wunused-but-set-variable]
  int err = 0;
      ^

Signed-off-by: Kefu Chai <kchai@redhat.com>

mds/CDir: do not set referenced variable

off is set by never get referenced, let's just drop it.

this variable was introduced by
0ec5f3a84aa29ec422e920fb5dc989ed90897e6b.

Signed-off-by: Kefu Chai <kchai@redhat.com>