Merge PR #22147 into master

* refs/pull/22147/head:
test: silence unused variable warning

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

Merge pull request #22187 from jdurgin/wip-pg-hard-limit

osd: increase default hard pg limit

Reviewed-by: Sage Weil <sage@redhat.com>

Merge pull request #22233 from neha-ojha/wip-fix-2580436

qa/workunits/suites/pjd.sh: use correct dir name

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

Merge pull request #21381 from liewegas/wip-seastore

doc/dev/seastore.rst: initial draft notes

Merge PR #22230 into master

* refs/pull/22230/head:
test/cli-integration/rbd: fixed unit formatting
qa/workunits/rbd: fix unit formatting tests
librbd: clones should not inherit implicit feature bits from parent

Reviewed-by: Mykola Golub <mgolub@suse.com>

Merge pull request #22250 from falcon78921/wip-doc-falcon78921-cloudstack

doc: updated reference link for creating new disk offerings in cloudstack

Merge pull request #22178 from aclamk/wip-bug-24225-fix

common/crc/aarch64: Added cpu feature pmull and make aarch64 specific…

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

Merge pull request #22236 from tchaikov/wip-rocksdb

rocksdb: sync with upstream

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

Merge pull request #22185 from ceph/wip-yuri-powercycle

qa/tests:  added supported distro for powercycle suite

Reviewed-by: Sage Weil <sage@redhat.com>

doc: updated reference link for creating new disk offerings in cloudstack

Signed-off-by: James McClune <jmcclune@mcclunetechnologies.net>

Merge pull request #22204 from tchaikov/wip-cmake-silence-bjam

cmake: always turn off bjam debugging output

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

Merge pull request #22126 from cbodley/wip-qa-rgw-no-ec-cache

qa/rgw: disable testing on ec-cache pools

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

Merge pull request #22124 from cbodley/wip-rgw-test-zonegroup-checkpoint

test/rgw: fix for bucket checkpoints

Reviewed-by: Yehuda Sadeh <yehuda@redhat.com>

Merge pull request #22246 from tchaikov/wip-mkfs-EINVAL

qa: use $TESTDIR for testing mkfs

Reviewed-by: Sage Weil <sage@redhat.com>

qa: use $TESTDIR for testing mkfs

as tmpfs does not support O_DIRECT, but bluefs use this flag for
accessing the underlying file.

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

Merge PR #22163 into wip-sage-testing-20180525.131829

* refs/pull/22163/head:
CodingStyle: update my_type_t sample
CodingStyle: allow #pragma once
CodingStyle: a few updates

Reviewed-by: Kefu Chai <kchai@redhat.com>
Reviewed-by: Josh Durgin <jdurgin@redhat.com>
Reviewed-by: Jesse Williamson <jwilliamson@suse.de>

Merge pull request #22237 from a2batic/link

mgr/dashboard: Fixes documentation link- to open in new tab

Reviewed-by: Ricardo Marques <rimarques@suse.com>

Merge pull request #22182 from tspmelo/wip-unit-test-pipe

mgr/dashboard: Add unit tests for all frontend pipes

Reviewed-by: Ricardo Marques <rimarques@suse.com>
Reviewed-by: Volker Theile <vtheile@suse.com>

Merge pull request #22206 from theanalyst/wip-doc-tag-fix

doc: rgw: fix tagging support status

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

Merge pull request #22207 from tchaikov/wip-dpp-constexpr

dout: declare dpp using `decltype(auto)` instead of `auto`

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

mgr/dashboard: Fixes documentation link- to open in new tab
Adds 'target' attribute to open link in new tab.
Fixes : https://tracker.ceph.com/issues/24288

Signed-off-by: a2batic <murarkakanika@gmail.com>

rocksdb: sync with upstream

since
https://github.com/facebook/rocksdb/pull/3862 has been merged by
upstream.

Fixes: http://tracker.ceph.com/issues/23653
Signed-off-by: Kefu Chai <kchai@redhat.com>

Merge PR #22026 into master

* refs/pull/22026/head:
mds: send cap export message when exporting non-auth caps to auth mds

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

Merge pull request #22232 from batrick/cephfs-release-notes-mimic

doc/releases: update CephFS mimic notes

Reviewed-by: Sage Weil <sage@redhat.com>

qa/workunits/suites/pjd.sh: use correct dir name

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

Merge pull request #22195 from neha-ojha/wip-minor-fixes

qa/suites/rados: minor fixes

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

doc/releases: update CephFS mimic notes

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

Merge pull request #22034 from mogeb/iostat-doc

doc: add documentation for iostat

Reviewed-by: Sage Weil <sage@redhat.com>

test/cli-integration/rbd: fixed unit formatting

Signed-off-by: Jason Dillaman <dillaman@redhat.com>

qa/workunits/rbd: fix unit formatting tests

Signed-off-by: Jason Dillaman <dillaman@redhat.com>

librbd: clones should not inherit implicit feature bits from parent

Signed-off-by: Jason Dillaman <dillaman@redhat.com>

CodingStyle: update my_type_t sample

Signed-off-by: Sage Weil <sage@redhat.com>

Merge pull request #22112 from szeestraten/fix_24199

common: fix typo in rados bench write JSON output

Reviewed-by: Sage Weil <sage@redhat.com>

Merge pull request #22093 from liewegas/wip-24162

msg/async/Event: clear time_events on shutdown

Reviewed-by: Hoamai Wang <haomai@xsky.com>

Merge pull request #21491 from majianpeng/kerneldevice

os/bluestore/KernelDevice: misc cleanup

Reviewed-by: Sage Weil <sage@redhat.com>

Merge pull request #22216 from votdev/fix_table_pagination_style

mgr/dashboard: Fix some datatable CSS issues

Approved-by: Ricardo Marques <rimarques@suse.com>

common/arch/arm: Improve CPU feature check.

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

mgr/dashboard: Fix some datatable CSS issues

Signed-off-by: Volker Theile <vtheile@suse.com>

Merge pull request #22214 from joscollin/wip-ceph-fuse-typo

doc: Fix typo in ceph-fuse(8)

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

doc: Fix typo in ceph-fuse

Fixes: https://github.com/ceph/ceph/pull/21616#pullrequestreview-122923127
Signed-off-by: Jos Collin <jcollin@redhat.com>

mgr/dashboard: Add unit tests for all frontend pipes

Signed-off-by: Tiago Melo <tmelo@suse.com>

Merge pull request #22205 from tchaikov/wip-librgw_admin_user

deb,rpm: package librgw_admin_user.{h,so.*}

Merge pull request #22181 from tchaikov/wip-24025

cmake/modules/BuildRocksDB.cmake: enable compressions for rocksdb

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

dout: declare dpp using `decltype(auto)` instead of `auto`

this makes `pdpp` an alias of `dpp`, guess this ensure GCC that the
returned `sub` is a constant.

In file included from /home/kefu/dev/ceph/src/kv/LevelDBStore.h:25,
                 from /home/kefu/dev/ceph/src/kv/KeyValueDB.cc:6:
/home/kefu/dev/ceph/src/osd/osd_types.h: In lambda function:
/home/kefu/dev/ceph/src/common/dout.h:101:75: error: the value of ‘pdpp’
is not usable in a constant expression
     dout_impl(pdpp->get_cct(),
ceph::dout::need_dynamic(pdpp->get_subsys()), v) \
                                                                           ^
/home/kefu/dev/ceph/src/common/dout.h:81:58: note: in definition of
macro ‘dout_impl’
       return (cctX->_conf->subsys.template should_gather<sub, v>()); \
                                                          ^~~
/home/kefu/dev/ceph/src/osd/osd_types.h:2992:3: note: in expansion of
macro ‘ldpp_dout’
   ldpp_dout(dpp, 10) << "build_prior all_probe " << all_probe << dendl;
   ^~~~~~~~~
/home/kefu/dev/ceph/src/common/dout.h:100:12: note: ‘pdpp’ was not
declared ‘constexpr’
   if (auto pdpp = (dpp); pdpp) /* workaround -Wnonnull-compare for
'this' */ \
            ^~~~
/home/kefu/dev/ceph/src/common/dout.h:100:12: note: in definition of
macro ‘ldpp_dout’
   if (auto pdpp = (dpp); pdpp) /* workaround -Wnonnull-compare for
'this' */ \
            ^~~~

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

doc: rgw: fix tagging support status

As of mimic, policy and LC conditionals for tagging are merged, so let's drop
the unsupported status.

Fixes: http://tracker.ceph.com/issues/24164
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>

deb,rpm: package librgw_admin_user.{h,so.*}

* install and package librgw_admin_user.h, so developers can use it to
  create rgw user.
* package librgw_admin_user, so user can use it to create rgw user.

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

cmake: always turn off bjam debugging output

we have 'DH_VERBOSE=1' in debian/rules, which instructs debhelper to
pass -DCMAKE_VERBOSE_MAKEFILE=ON to cmake. but the verbose output for
building boost does not really help if something goes wrong while
building the deb packages. if we do want to enable the verbose output,
we can always enable it by passing `-d <N>' to it. see
https://boostorg.github.io/build/manual/develop/index.html#bbv2.overview.invocation.options
.

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

Merge pull request #22198 from dzafman/wip-minor

test: wait_for_pg_stats() should do another check after last 13 secon…

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

test: silence unused variable warning

Remove the unused variable 'poolid' in TEST(LibRadosMicsPool,
PoolCreationRace).

Signed-off-by: Mao Zhongyi <maozy.fnst@cn.fujitsu.com>

test: wait_for_pg_stats() should do another check after last 13 second sleep

Signed-off-by: David Zafman <dzafman@redhat.com>

Merge pull request #22194 from yehudasa/wip-fix-build

rgw: fix build

Reviewed-by: Sage Weil <sage@redhat.com>

qa/suites/rados/objectstore: move yaml files under a subdir

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

qa: whitelist SMALLER_PGP_NUM

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

rgw: fix build

Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>

Merge pull request #22190 from smithfarm/wip-google-style

doc/SubmittingPatches.rst: use Google style guide for doc patches

Reviewed-by: Sage Weil <sage@redhat.com>

doc/SubmittingPatches.rst: rv double spaces and Please

Signed-off-by: Nathan Cutler <ncutler@suse.com>

doc/SubmittingPatches.rst: use Google style guide for doc patches

Signed-off-by: Nathan Cutler <ncutler@suse.com>

Merge pull request #22186 from cbodley/wip-cmake-rgw-request

cmake: remove deleted rgw_request.cc from CMakeLists.txt

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

osd: increase default hard pg limit

The hard limit was too easy to hit during failure cases, especially
due to mismatched device sizes causing high variance in pgs/osd.

Fixes: https://tracker.ceph.com/issues/24243
Signed-off-by: Josh Durgin <jdurgin@redhat.com>

cmake/modules/BuildRocksDB.cmake: enable compressions for rocksdb

we should enable them if they are found.

currently, we don't have bzip2 compressor plugin, so it's not detected
in the cmake script. we can always enable it for rocksdb in future.

Fixes: http://tracker.ceph.com/issues/24025
Signed-off-by: Kefu Chai <kchai@redhat.com>

cmake: remove deleted rgw_request.cc from CMakeLists.txt

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

qa/tests:  added supported distor for powercycle suite

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

Merge pull request #22116 from alfredodeza/wip-cv-ansible-deps

ceph-volume tests.functional add notario dep for ceph-ansible

Reviewed-by: Andrew Schoen <aschoen@redhat.com>

Merge pull request #21439 from oritwas/wip-rgw-librgwadmin

new librgw_admin_us

common/crc/aarch64: Added cpu feature pmull and make aarch64 specific crc function to depend on it.

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

Merge pull request #22060 from ricardoasmarques/wip-perf-counter-breadcrumb

Add performance counters breadcrumb

Reviewed-by: Tiago Melo <tmelo@suse.com>

Merge pull request #22150 from ricardoasmarques/wip-improve-no-pools-message

mgr/dashboard: Improve 'no pool' message on rbd form

Reviewed-by: Kanika Murarka <murarkakanika@gmail.com>

ceph-volume tests.functional install new ceph-ansible dependencies

Make note that ceph-ansible's requirements.txt can't be used just yet

Signed-off-by: Alfredo Deza <adeza@redhat.com>

Merge pull request #22158 from liewegas/wip-24222

mon/MgrMonitor: change 'unresponsive' message to info level

Reviewed-by: David Zafman <dzafman@redhat.com>
Reviewed-by: John Spray <john.spray@redhat.com>

CodingStyle: allow #pragma once

Signed-off-by: Sage Weil <sage@redhat.com>

Merge pull request #22123 from liewegas/wip-24211

os/bluestore: simplify and fix SharedBlob::put()

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

Merge pull request #22127 from liewegas/wip-23386

crush: fix device_class_clone for unpopulated/empty weight-sets

Reviewed-by: xie xingguo <xie.xingguo@zte.com.cn>

Merge pull request #22110 from majianpeng/bluefs-flush-dirty-devices

os/bluestore/BlueFS: only flush dirty devices when do _fsync

Reviewed-by: Sage Weil <sage@redhat.com>
Reviewed-by: Igor Fedotov <ifedotov@suse.com>

Merge pull request #22129 from rzarzynski/wip-auth-fix-openssl-on-arm

auth, test: fix building on ARMs after the NSS -> OpenSSL transition

Reviewed-by: Sage Weil <sage@redhat.com>

Merge pull request #22164 from dzafman/wip-test-fixes

test: Whitelist corrections

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

CodingStyle: a few updates

The C++ style was originally adopted in 2011.  Most of the guidelines
have been followed but some have not.  Drop the ones we haven't been
following.

Signed-off-by: Sage Weil <sage@redhat.com>

Merge pull request #22156 from rzarzynski/wip-bug-24037

common: OpTracker doesn't visit TrackedOp when nref == 0.

Reviewed-by: Sage Weil <sage@redhat.com>

mon/MgrMonitor: change 'unresponsive' message to info level

We generate a MGR_DOWN health warning at the appropriate points; having
this at WRN level just triggers failed teuthology runs but doesn't much
value for the user.

Clear out teuthology whitelisting for this message.

Fixes: http://tracker.ceph.com/issues/24222
Signed-off-by: Sage Weil <sage@redhat.com>

test: Whitelist corrections

Signed-off-by: David Zafman <dzafman@redhat.com>

common: OpTracker doesn't visit TrackedOp when nref == 0.

The patch fixes a race condition that happens between
`unregister_inflight_op` and `visit_ops_in_flight` of
`OpTracker`. When a callable passed to the former one
turns the plain reference it gets into `TrackedOpRef`,
an almost-to-terminate `TrackedOp` (with `nref == 0`)
can be resurrected (`nref++`). This will be reflected
in extra call to `unregister_inflight_op` for same op
leading to e.g. use-after-free. For more details see:
https://tracker.ceph.com/issues/24037#note-5.

The fix deals with the problem by ensuring there will
be no call to the visitor for ops with zeroized `nref`.

Fixes: http://tracker.ceph.com/issues/24037
Signed-off-by: Radoslaw Zarzynski <rzarzyns@redhat.com>

Merge pull request #21700 from cbodley/wip-rgw-dpp

rgw: use DoutPrefixProvider to add more context to log output

Reviewed-by: Orit Wasserman <owasserm@redhat.com>
Reviewed-by: Matt Benjamin <mbenjamin@redhat.com>
Reviewed-by: Abhishek Lekshmanan <abhishek@suse.com>

Merge pull request #21755 from linuxbox2/wip-unordered-gc-delay

rgw: use partial-order bucket listing in RGWLC, add configurable processing delay

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

Merge pull request #21838 from yehudasa/wip-rgw-async-watch

rgw: async watch registration

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

Merge pull request #22024 from votdev/bug_24142

rgw: Do not modify email if argument is not set

Reviewed-by: Orit Wasserman <owasserm@redhat.com>
Reviewed-by: Casey Bodley <cbodley@redhat.com>
Reviewed-by: Matt Benjamin <mbenjamin@redhat.com>
Reviewed-by: Abhishek Lekshmanan <abhishek@suse.com>

Merge pull request #22100 from joscollin/wip-rgw-period

rgw: update --url in usage and doc

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

Merge pull request #22114 from zhangsw/cleanup-rgw-cors

rgw: remove the useless is_cors_op in RGWHandler_REST_Obj_S3.

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

Merge pull request #22083 from liewegas/wip-21480

os/bluestore: fix flush_commit locking

Reviewed-by: Igor Fedotov <ifedotov@suse.com>

Merge pull request #22062 from alfredodeza/wip-rm24031

ceph-volume dmcrypt and activate --all documentation updates

Reviewed-by: Andrew Schoen <aschoen@redhat.com>

Merge pull request #22065 from ricardoasmarques/wip-table-pagination-style

Improve table pagination style

Reviewed-by: Tiago Melo <tmelo@suse.com>

Merge pull request #22149 from b-ranto/wip-fix-metadata

prometheus: Fix order of occupation values

Reviewed-by: Jan Fajerski <jfajerski@suse.com>

os/bluestore: check input validtiy for aio_read.

Signed-off-by: Jianpeng Ma <jianpeng.ma@intel.com>

os/bluestore: using func is_valid_io to check input validity.

Signed-off-by: Jianpeng Ma <jianpeng.ma@intel.com>

os/bluestore: fix wrong messages.

Signed-off-by: Jianpeng Ma <jianpeng.ma@intel.com>

Merge pull request #22031 from tspmelo/wip-jest

mgr/dashboard: Add Jest Runner

Reviewed-by: Laura Paduano <lpaduano@suse.com>
Reviewed-by: Ricardo Marques <rimarques@suse.com>

Merge pull request #22088 from tspmelo/fix-rbd-task-metadata

mgr/dashboard: Fix RBD task metadata

Reviewed-by: Ricardo Dias <rdias@suse.com>
Reviewed-by: Ricardo Marques <rimarques@suse.com>

mgr/dashboard: Improve 'no pool' message on rbd form

Signed-off-by: Ricardo Marques <rimarques@suse.com>

prometheus: Fix order of occupation values

When we merged the PR to unify the metadata labels, we forgot to switch
the order of hostname and disk in ceph_disk_occupation metric.

Signed-off-by: Boris Ranto <branto@redhat.com>

os/bluefs: only flush dirty devices when do _fsync.

Now _fsync call flush_bdev make data safely. But flush_bdev flush all
devices which don't care whether has data for this sync.
So add new api flush_bdev(std::array<bool, MAX_BDEV>& dirty_bdevs)
which only flush dirty devices for this sync op.

Signed-off-by: Jianpeng Ma <jianpeng.ma@intel.com>

mgr/iostat: enable module by default on new and vstart clusters

Signed-off-by: Mohamad Gebai <mgebai@suse.com>

doc: add documentation for iostat

Signed-off-by: Mohamad Gebai <mgebai@suse.com>