Merge pull request #43262 from rzarzynski/wip-crimson-alienstore-fix-oncommit

crimson/os/alienstore: fix nullptr deref in OnCommit::finish().

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

crimson/os/alienstore: fix nullptr deref in OnCommit::finish().

`seastar::engine()` is available only for Seastar's threads;
it shouldn't be called outside of a reactor thread.
Unfortunately, this assumption is violated in `AlienStore`
where `OnCommit::finish()`, executed from a finisher thread
of `BlueStore`, calls `alien()` on `seastar::engine()`.
The net effect are crashes like the following one:

```
INFO  2021-09-22 14:26:33,214 [shard 0] osd - operator() writing superblock cluster_fsid 1d8f7908-2ebf-4a91-ae70-f445668c126b osd_fsid 4da9fe9a-1da5-4ea9-aa79-a1178165ede5         [381/1839]
Segmentation fault.
Backtrace:
 0# print_backtrace(std::basic_string_view<char, std::char_traits<char> >) at /home/rzarzynski/ceph1/build/../src/crimson/common/fatal_signal.cc:80
 1# FatalSignal::signaled(int, siginfo_t const&) at /opt/rh/gcc-toolset-9/root/usr/include/c++/9/ostream:570
 2# FatalSignal::install_oneshot_signal_handler<11>()::{lambda(int, siginfo_t*, void*)#1}::_FUN(int, siginfo_t*, void*) at /home/rzarzynski/ceph1/build/../src/crimson/common/fatal_signal.cc:
62
 3# 0x00007F16BBA13B30 in /lib64/libpthread.so.0
 4# (anonymous namespace)::OnCommit::finish(int) at /home/rzarzynski/ceph1/build/../src/crimson/os/alienstore/alien_store.cc:53
 5# Context::complete(int) at /home/rzarzynski/ceph1/build/../src/include/Context.h:100
 6# Finisher::finisher_thread_entry() at /home/rzarzynski/ceph1/build/../src/common/Finisher.cc:65
 7# 0x00007F16BBA0915A in /lib64/libpthread.so.0
 8# clone in /lib64/libc.so.6
Dump of siginfo:
  ...
  si_addr: 0x10
```

Signed-off-by: Radoslaw Zarzynski <rzarzyns@redhat.com>

Merge pull request #40728 from guits/doc_fix_pattern_rgw_nfs

doc/rgw-nfs: use same pattern for keyring name

Merge pull request #43254 from cyx1231st/wip-seastore-fix-onode-order

crimson/onode-staged-tree: convert hash to the reversed version

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

Merge pull request #43249 from cyx1231st/wip-seastore-fix-omap-hint

crimson/os/seastore: add missing hints in omap tree

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

Merge pull request #43055 from soumyakoduri/wip-skoduri-lua

rgw/lua: Install the packages only for RadosStore

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

Merge pull request #43054 from soumyakoduri/wip-skoduri-dbstore-vstart

rgw: Add option to configure backend store

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

Merge pull request #42911 from soumyakoduri/wip-skoduri-dbstore-object

rgw/dbstore object APIs

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

Merge pull request #43241 from sebastian-philipp/suites-orch-labelere

.github: fix path to cephadm suite

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

crimson/onode-staged-tree: consolidate laddr hint calculation

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

crimson/onode-staged-tree: convert hash to the reversed version

Store the reversed version of object hash to make sure that onodes in
the same PG are sorted together.

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

crimson/onode-staged-tree: print cursh-hash in hex mode

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

crimson/os/seastore: add missing hints in omap tree

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

Merge pull request #43247 from rzarzynski/wip-crimson-ertr-safe_then_unpack

crimson/common: add safe_then_unpack() to errorated futures

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

crimson/common: add safe_then_unpack() to errorated futures.

It was a prerequisite for another commit I finally thrown
away. However, this little bit can be still be useful even
for the sake of compliance with the interruptible variant.

Signed-off-by: Radoslaw Zarzynski <rzarzyns@redhat.com>

crimson/common: assert_moveable() doesn't depend on 3rd party's always_false<>.

Signed-off-by: Radoslaw Zarzynski <rzarzyns@redhat.com>

Merge pull request #43243 from rzarzynski/wip-crimson-net-dangling-bindvec2

crimson/net: fix dangling addrvec in bind(), the repeat_until_value() part

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

Merge pull request #43209 from rzarzynski/wip-crimson-silent-check-bot

tests/crimson: don't be so verbose when run by the 'make check' bot.

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

Merge PR #43219 into master

* refs/pull/43219/head:
doc: clarify MDS failed state

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

Merge PR #43156 into master

* refs/pull/43156/head:
qa/suites/orch/rook/smoke: add flannel back in

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

crimson/net: fix dangling addrvec in bind(), the repeat_until_value() part.

This is a follow-up to commit 480273082718230d64d2c068db1b22f44997bdc8.

`seastar::do_with()` doesn't extend the life-time of the callable's

```cpp
auto
do_with_impl(T1&& rv1, T2&& rv2, More&&... more) {
    // ...
    auto&& just_values = cherry_pick_tuple(idx(), std::move(all));
    auto&& just_func = std::move(std::get<nr>(std::move(all)));
    // ...
    auto task = std::apply(
        [](auto&&... x) {
            return std::make_unique<internal::do_with_state<value_tuple, ret_type>>(std::forward<decltype(x)>(x)...);
        },
        std::move(just_values));
    auto fut = std::apply(just_func, task->data());
    // ...
}
```

closure. This means the `addrs` needs to be captured by value also
by the lambda passed to `repeat_until_value()`.

Signed-off-by: Radoslaw Zarzynski <rzarzyns@redhat.com>

Merge pull request #43225 from guits/fix-iscsi-doc

doc/iscsi: fix some typos

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

.github: fix path to cepham suite

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

Merge pull request #43237 from javicacheiro/fix_upgrade_now_uses_quay.io

doc:  from v16.2.6 you have to point to the quay.io image

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

Merge pull request #43142 from sebastian-philipp/c-v-cephadm-osd-activate

doc/ceph-volume/activate: Users should call cephadm instead

Reviewed-by: Guillaume Abrioux <gabrioux@redhat.com>

Merge pull request #43162 from sebastian-philipp/cephadm-service-mgmt-reoder

doc/cephadm/service-management: Move "Updating Service Specifications"

Reviewed-by: Adam King <adking@redhat.com>
Reviewed-by: Michael Fritch <mfritch@suse.com>

Merge pull request #43210 from rhcs-dashboard/config-e2e-failure

mgr/dashboard: Fix failing config dashboard e2e check

Reviewed-by: Alfonso Martínez <almartin@redhat.com>
Reviewed-by: Avan Thakkar <athakkar@redhat.com>
Reviewed-by: Nizamudeen A <nia@redhat.com>
Reviewed-by: Pere Diaz Bou <pdiazbou@redhat.com>

doc: from v16.2.6 you have to point to the quay.io image if using docker

New images are no longer uploaded to the Docker Hub registry, so Docker users should point the upgrade
command to the quay.io image.

Fixes: https://tracker.ceph.com/issues/52683
Signed-off-by: Javier Cacheiro <javier.cacheiro.lopez@cesga.es>

Merge pull request #43233 from badone/wip-install-xmlstarlet

Revert "rpm, debian: drop xmlstarlet from build deps"

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

Revert "rpm, debian: drop xmlstarlet from build deps"

This reverts commit 707edc0b2d5283104aeb472131ee94dd59544290.

This commit was somewhat premature in removing the xmlstarlet
dependency.

Fixes: https://tracker.ceph.com/issues/52681
Signed-off-by: Brad Hubbard <bhubbard@redhat.com>

Merge PR #43179 into master

* refs/pull/43179/head:
qa: lengthen grace for fs map showing dead MDS

Reviewed-by: Venky Shankar <vshankar@redhat.com>
Reviewed-by: Jeff Layton <jlayton@redhat.com>

Merge PR #43164 into master

* refs/pull/43164/head:
test/libcephfs: put inodes after lookup

Reviewed-by: Xiubo Li <xiubli@redhat.com>
Reviewed-by: Venky Shankar <vshankar@redhat.com>

Merge PR #42763 into master

* refs/pull/42763/head:
mon/FSCommands: add 'recover' flag in `fs new` command

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

rgw/dbstore: Update README

Signed-off-by: Soumya Koduri <skoduri@redhat.com>

Merge pull request #43222 from sebastian-philipp/rook-ceph-client-before-3.8

mgr/rook: update rook-client-python to 2737c7467b7

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

Merge PR #43213 into master

* refs/pull/43213/head:
mon/OSDMonitor: make weight set create safer

Reviewed-by: xie xingguo <xie.xingguo@zte.com.cn>
Reviewed-by: Kefu Chai <kchai@redhat.com>

Merge PR #43211 into master

* refs/pull/43211/head:
qa/suites/upgrade/pacific-x: skip c-v install on pacific start
qa/packages: install ceph-volume

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

doc/iscsi: fix some typos

This fixes some typos in the iscsi documentation.

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>

rgw/dbstore: Object APIs

Support for simple Put, Get, Delete, List Ops of Regular Objects on dbstore.

Signed-off-by: Soumya Koduri <skoduri@redhat.com>

rgw/dbstore: Insert testid user on startup

Its temporary change to insert testid user while
starting up rgw server on dbstore.

Signed-off-by: Soumya Koduri <skoduri@redhat.com>

rgw: Add option to configure backend store

Signed-off-by: Soumya Koduri <skoduri@redhat.com>

rgw: Install the lua packages only for RadosStore

Installation, remove and listing of lua packages is supported
only on RadosStore. Verify the same.

Signed-off-by: Soumya Koduri <skoduri@redhat.com>

Merge pull request #43193 from dang/wip-dang-zipper-lifecycle

RGW Zipper - clean up and clarify lifecycle of s->object, s->bucket, and friends

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

qa/suites/upgrade/pacific-x: skip c-v install on pacific start

Signed-off-by: Sage Weil <sage@newdream.net>

mgr/dashboard: Fix failing config dashboard e2e check

Recently a new osd config has been added in
6ca32bde2e1d0dd58df168126582a570ac09aad6 and this is getting Modified.
So on our dashboard e2e config check which checks for the Modified
filter, this is also coming on the entry. So we need to increase the
count.

Fixes: https://tracker.ceph.com/issues/52649
Signed-off-by: Nizamudeen A <nia@redhat.com>

mgr/rook: update rook-client-python to 82673cd7c7a

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

qa/packages: install ceph-volume

Signed-off-by: Sage Weil <sage@newdream.net>

Merge pull request #43025 from TRYTOBE8TME/wip-rgw-amqp-timeout

Wip rgw amqp timeout

Merge pull request #42815 from tchaikov/wip-rocksdb

kv/rocksdb_cache: implement methods required by rocksdb v6.22.1

Reviewed-by: Mark Nelson <mnelson@redhat.com>

doc: clarify MDS failed state

Since 58eaa237b0a1, an MDS is only promoted if it is compatible with the
file system compat. The users may see persistent failed state even they
have enough standby daemons.

Signed-off-by: 胡玮文 <huww98@outlook.com>

Merge pull request #43195 from Thingee/update-rbd-lead

doc: updating RBD lead

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

Merge pull request #43208 from abcinje/librados-examples

examples/librados: avoid a memory leak

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

examples/librados: avoid a memory leak

Avoid a memory leak by deallocating the pre-allocated aio completion.

Signed-off-by: Injae Kang <abcinje@gmail.com>

Merge pull request #43212 from ceph/crn-span

src/script/ceph-release-notes: replace breaklink w span

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

Merge pull request #43206 from Matan-B/wip-matanb-installdeps-pip

install-deps.sh: install newer version of pip

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

Merge pull request #42991 from benhanokh/suppress_bogus_errors

BUG-FIX: NCB code was reporting Bogus error when we had an overlapped…

Reviewed-by: Adam Kupczyk <akupczyk@redhat.com>

mon/OSDMonitor: make weight set create safer

We cannot return early success based on uncommitted state.

Signed-off-by: Sage Weil <sage@newdream.net>

src/script/ceph-release-notes: replace breaklink w span

Not sure why but the <!-- breaklink > was getting rendered as plaintext.

Putting <span></span> breaks the auto hyperlinking.

Signed-off-by: David Galloway <dgallowa@redhat.com>

Merge PR #43144 into master

* refs/pull/43144/head:
doc/cephfs: fix typos

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

tests/crimson: don't be so verbose when run by the 'make check' bot.

This commit is a try to balance crimson's debug facilities with overwhelming
some hosts the `make check` bot runs on with huge number of log entries;
see https://jenkins.ceph.com/job/ceph-pull-requests-arm64/17127/ for an
exemplification of the problem.

This is achieved by differentiating debug levels depending upon the
`FOR_MAKE_CHECK` env variable. It's a hack / a makeshift solution
only. IMHO ultimately we should introduce a dedicated target for
the bot while leaving `ninja test` for humans which would allow for more
debugs and maybe also for running `vstart.sh`-dependent tests.

Signed-off-by: Radoslaw Zarzynski <rzarzyns@redhat.com>

Merge pull request #43071 from pritha-srivastava/wip-rgw-sts-op-logs

rgw/sts: add check for s->auth.identity pointer

Merge pull request #43165 from linuxbox2/wip-perl-semicolon

workunits/rgw: semicolon terminates perl statements

RGW Zipper - clean up setting bucket on s->object

s->object is always created wihtout a bucket, and the bucket is added
later.  The proper place for this is in rgw_build_bucket_policies(), in
the permissions handling callpaths.  Remove all the other places where
bucket is set, ensuring that s->object has a stable bucket pointer.

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

RGW Zipper - comment setting s->object in website code

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

RGW - Remove extra src_object from Copy OP

req_state has a src_object, and it's used from places that don't have
access to the Copy OP.  Remove the version from the Copy OP, and
consolidate on the one in req_state.

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

RGW - Zipper don't overwrite s->bucket

Undo an overwrite of s->bucket that was added as part of the zipper
work.  s->bucket should never be overwritten.

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

RGW - Zipper bucket/object lifecycle in librgw

s->bucket and s->object need to have a single lifecycle across an entire
op, never replaced, since lots of things use pointers to them.

Make sure s->bucket is created in read_permissions(), and that s->object
is created with the correct bucket.

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

Merge pull request #43154 from ceph/wip-yuriw-pacific-16.2.6-RN

doc: 16.2.6 Release Notes

install-deps.sh: install newer version of pip

although pip comes with virtualenv, having a recent version of pip matters to use wheel packages.

Signed-off-by: Matan Breizman <Matan.Brz@gmail.com>

Merge pull request #43197 from anthonyeleven/anthonyeleven/stale-read

doc: clarify stale_read and placement-groups wording

Reviewed-by: Zac Dover <zac.dover@gmail.com>

Merge PR #43122 into master

* refs/pull/43122/head:
qa: add test for standby-replay marking rank damaged
MDSMonitor: handle damaged from standby-replay
mds: add config to mark rank damaged in standby-replay
include: unset std::hex after printing CompatSet
mds: refactor iterator lookup
mds: harden rank lookup

Reviewed-by: Venky Shankar <vshankar@redhat.com>

doc: clarify stale_read and placement-groups wording

Signed-off-by: Anthony D'Atri <anthony.datri@gmail.com>

Merge pull request #43011 from rzarzynski/wip-crimson-opsequencer-ooo-lesser-footprint

crimson/osd: bring OpRegistry-based OpSequencer to handle out-of-order ClientRequests

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

Merge pull request #43192 from rzarzynski/wip-crimson-net-dangling-bindvec

crimson/net: fix dangling addrvec in SocketMessenger::bind().

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

doc: updating RBD lead

Signed-off-by: Mike Perez thingee@gmail.com

doc/releases/pacific.rst: Add note about Bullseye

crimson/net: fix dangling addrvec in SocketMessenger::bind().

`SocketMessenger::bind()` takes the address vector by `const&`.
while the callers in `OSD` are passing a temporary:

  ```cpp
  cluster_msgr->bind(pick_addresses(CEPH_PICK_ADDRESS_CLUSTER))
    // ...
  public_msgr->bind(pick_addresses(CEPH_PICK_ADDRESS_PUBLIC))
    // ...
  ```

  ```cpp
  entity_addrvec_t pick_addresses(int what) {
    // ...
    return addrs;
  }
  ```

The net result is a life-time mismatch and crashes like the following
one:

  ```
  WARN  2021-09-10 20:01:57,791 [shard 0] ms - [osd.0(client) v2:172.17.0.1:6800/3172806564@59988 >> mgr.? v2:172.17.0.7:6800/3846062660] waiting 3.2 seconds ...
  /opt/rh/gcc-toolset-9/root/usr/include/c++/9/bits/stl_iterator.h:820:17: runtime error: reference binding to misaligned address 0x000041b58ab3 for type 'const struct entity_addr_t', which requires 4 byte alignment
  0x000041b58ab3: note: pointer points here
  <memory cannot be printed>
  /opt/rh/gcc-toolset-9/root/usr/include/c++/9/bits/stl_vector.h:1132:16: runtime error: reference binding to misaligned address 0x000041b58ab3 for type 'const struct value_type', which requires 4 byte alignment
  0x000041b58ab3: note: pointer points here
  <memory cannot be printed>
  /home/jenkins-build/build/workspace/ceph-dev-new-build/ARCH/x86_64/AVAILABLE_ARCH/x86_64/AVAILABLE_DIST/centos8/DIST/centos8/MACHINE_SIZE/gigantic/release/17.0.0-7486-g27cb19ed/rpm/el8/BUILD/ceph-17.0.0-7486-g27cb19ed/src/msg/msg_types.h:561:22: runtime error: reference binding to misaligned address 0x000041b58ab3 for type 'const struct entity_addr_t', which requires 4 byte alignment
  0x000041b58ab3: note: pointer points here
  <memory cannot be printed>
  Segmentation fault on shard 0.
  ```

Signed-off-by: Radoslaw Zarzynski <rzarzyns@redhat.com>

doc/releases/pacific.rst: add core release notes

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

crimson/osd: rework OpSequencer for out-of-order op execution.

Signed-off-by: Radoslaw Zarzynski <rzarzyns@redhat.com>

Merge pull request #43180 from athanatos/sjust/wip-52630

crimson/net/SocketMessenger: include sleep.hh

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

Merge pull request #43178 from athanatos/sjust/wip-52623

crimson/os/seastore/cache: Cache::get_root check for invalid

Reviewed-by: Xuehan Xu <xxhdx1985126@gmail.com>
Reviewed-by: Yingxin Cheng <yingxin.cheng@intel.com>

Merge PR #43044 into master

* refs/pull/43044/head:
mgr/rook: add unit tests to Rook module
mgr/rook: translate placement spec to node selector and vice versa

Reviewed-by: Juan Miguel Olmo <jolmomar@redhat.com>

Merge PR #43172 into master

* refs/pull/43172/head:
qa/tasks/kubeadm: modify (do not clobber) daemon.json

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

crimson/net/SocketMessenger: include sleep.hh

4a00a145 added a call to seastar::sleep.

Fixes: https://tracker.ceph.com/issues/52630
Signed-off-by: Samuel Just <sjust@redhat.com>

Merge PR #43022 into master

* refs/pull/43022/head:
doc/cephfs: add mds_dir_max_entries option into mds-config-ref.rst
mds: add mds_dir_max_entries config option

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

Merge PR #42943 into master

* refs/pull/42943/head:
mon: do not quickly mark mds laggy when MON_DOWN
mon: refactor quorum age calculation

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

Merge PR #42846 into master

* refs/pull/42846/head:
mds: Add new flag to MClientSession

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

Merge PR #42737 into master

* refs/pull/42737/head:
mds: ensure that we send the btime in cap messages

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

Merge PR #42719 into master

* refs/pull/42719/head:
mgr/volumes: Fix permission during subvol creation with mode

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

crimson/os/seastore/cache: Cache::get_root check for invalid

As with Cache::get_extent, we need to check that the ref
is still valid after wait_io since it's not in the
read set yet.

Fixes: https://tracker.ceph.com/issues/52623
Signed-off-by: Samuel Just <sjust@redhat.com>

Merge PR #42584 into master

* refs/pull/42584/head:
doc: fix `daemon status` interface (exclude file system name)
test: adjust mirroring tests for `daemon status` change
mgr/mirroring: `daemon status` command does not require file system name

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

Merge PR #42541 into master

* refs/pull/42541/head:
client: switch to use ceph_assert instead of assert
mds: switch to use ceph_assert() instead of assert()

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

qa: lengthen grace for fs map showing dead MDS

Fixes: https://tracker.ceph.com/issues/52625
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>

crimson/os/seastore/cache: rename invalidate(Transaction&, CachedExtent&)

This particular overload is really only declaring the Transaction
conflicted -- it assumes the extent is already invalid.  Rename to
mark_transaction_conflicted instead to avoid confusion with
invalidate(CachedExtent&).

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

qa/tasks/kubeadm: modify (do not clobber) daemon.json

Otherwise we blow away the mirror config.

Signed-off-by: Sage Weil <sage@newdream.net>

doc: updated the change log with three additional PRs merged on top

Signed-off-by: Yuri Weinstein <yweinste@redhat.com>

doc: dashboard motd change

Signed-off-by: Ernesto Puerta <epuertat@redhat.com>

doc: note simpler MDS upgrade procedure for v16.2.6

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

doc: 16.2.6 Release Notes

Signed-off-by: Yuri Weinstein <yweinste@redhat.com>

Merge pull request #43146 from jianwei1216/fix-partial-recovery-become-whole-object-recovery-for-master

osd: fix partial recovery become whole object recovery after restart osd

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