Merge pull request #17024 from cbodley/wip-20906

cls/log: cls_log_list always returns next marker

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

Merge pull request #17048 from theanalyst/rgw-fix-get-obj-tag-attr

rgw: rgw_op: exit early if object has no attrs in GetObjectTagging

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

Merge pull request #17050 from theanalyst/rgw-iam-get-bucket-acl

rgw: evaluate the correct bucket action for GetACL bucket operation

Reviewed-by: Casey Bodley <cbodley@redhat.com>
Reviewed-by: Adam C. Emerson <aemerson@redhat.com>

Merge pull request #17098 from dzafman/wip-runs-core

tests: run-standalone.sh skip core_pattern if already set

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

Merge pull request #17044 from baruza/wip-doc-21007

doc: Improve the ceph fs set max_mds command

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

doc: Improve the ceph fs set max_mds command

Fixes: http://tracker.ceph.com/issues/21007
Signed-off-by: Bara Ancincova <bara@redhat.com>

Merge pull request #17112 from optimistyzy/821_spdk

NVMDevice: fix issued caused by #17002

Merge pull request #17107 from Songweibin/wip-spell

misc fix spell errors in osd/OSD and doc

Reviewed-by: Jos Collin <jcollin@redhat.com>
Reviewed-by: Alfredo Deza <adeza@redhat.com>

Merge pull request #17105 from iliul/add-mailmap-affiliation

.mailmap, .organizationmap: Add Liu Lei's mailmap and affiliation

Reviewed-By: Abhishek Lekshmanan <abhishek@suse.com>
Reviewed-By: Jos Collin <jcollin@redhat.com>

Merge pull request #17109 from ashishkumsingh/add-mailmap-affiliation

organizationmap: Add Ashish Singh affiliation

Reviewed-by: Jos Collin <jcollin@redhat.com>
Reviewed-by: Abhishek Lekshmanan <abhishek@suse.com>

organizationmap: Ashish Singh affiliation

Signed-off-by: Ashish Singh <assingh@redhat.com>

NVMDevice: fix issued caused by #17002

Patch #17002 breaks running osd when using spdk, and
this patch can fix this.

Signed-off-by: Ziye Yang <optimistyzy@gmail.com>

Merge pull request #17078 from dillaman/wip-21029

test/librbd: TestMirroringWatcher unit tests should ignore duplicates

Merge pull request #16647 from tchaikov/wip-osd-exit-directly-on-failure

osd: exit(1) directly without lock if init fails

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

doc: fix a spell error and modify layout

Signed-off-by: songweibin <song.weibin@zte.com.cn>

osd/OSD: fix spell errors

Signed-off-by: songweibin <song.weibin@zte.com.cn>

.mailmap, .organizationmap: Add Liu Lei's mailmap and affiliation

Signed-off-by: iliul <liul124@chinaunicom.cn>

Merge pull request #17086 from optimistyzy/818_write

bluestore/NVMEDevice: fix the bug in write function

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

Merge pull request #17104 from iliul/fix-duplicate-words

doc: delete duplicate words

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

doc: delete duplicate words

Signed-off-by: iliul <liul124@chinaunicom.cn>

Merge pull request #17060 from cyx1231st/fix_reqid_assignment_in_oprequest

osd: fix reqid assignment for reply messages in OpRequest

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

Merge pull request #16687 from Songweibin/wip-xml-format

common: adding line break at end of some cli results

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

Merge pull request #15118 from squidboylan/fix_15872

debian: lock ceph user during purge

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

Merge pull request #17059 from tchaikov/wip-optional-lz4

compressor: conditionalize on HAVE_LZ4

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

Merge pull request #17015 from lixiaoy1/lat_fix

os/bluestore: Add lat record of deferred_queued and deferred_aio_wait

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

Merge pull request #16955 from xiexingguo/wip-app-ls

mon/OSDMonitor: "osd pool application get" support

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

Merge pull request #17064 from xiexingguo/wip-20970

mon/PGMap: reweight::by_utilization - skip DNE osds

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

Merge pull request #17067 from renhwztetecs/renhw-wip-mon-cleanup

mon: some cleanup

Reviewed-by: Joao Eduardo Luis <joao@suse.de>

Merge pull request #17052 from dzafman/wip-pidfile

tests: Increase sleep in test_pidfile.sh

Reviewed-by: Loic Dachary <ldachary@redhat.com>

Merge pull request #15691 from vshankar/mirror-ha-policy

rbd-mirror: introduce basic image mapping policy

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

tests: run-standalone.sh skip core_pattern if already set

If sudo requires a password it is nice to be able to avoid
sudo sysctl by setting core_pattern ahead of time.

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

Merge pull request #17076 from ceph/toc-sidebar-fix

doc/templates update toctree call to include hidden entries

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

Merge pull request #17093 from dzafman/wip-runs-message

tests: run-standalone.sh improve error message

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

doc update template call for toctree

Up until version 1.1.3 of Sphinx, this call would show hidden toctree
entries. Most of Ceph's toctree's are hidden, so when the version of
Sphinx was bumped to >1.6, the sidebar stopped showing entries

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

Merge pull request #17026 from dzafman/wip-add-trap

tests: qa/standalone: Add trap for signals to restore the kernel core pattern

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

os/bluestore: Add lat record of deferred_queued and deferred_aio_wait

Originally bluestore defines the two lat deferred_queued_lat and
deferred_aio_wait_lat, but doesn't use it.
This patch is to add the records that each TXC stays on deferred queue
deferred ios.

Signed-off-by: Xiaoyan Li xiaoyan.li@intel.com

rbd-mirror: load/update image map state machine

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

rbd-mirror: track on-disk image to instance map

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

tests: run-standalone.sh improve error message

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

NVMEDevice: fix the write bug

This patch can solve the write issue when we use spdk nvme driver
to handle I/Os from db bdev in the future.

Signed-off-by: Ziye Yang <optimistyzy@gmail.com>
Signed-off-by: Pan Liu <wanjun.lp@alibaba-inc.com>

Merge pull request #17070 from ktdreyer/rpm-macros-in-comments

rpm: rm macros in comments

Reviewed-By: Nathan Cutler <ncutler@suse.com>
Reviewed-by: Boris Ranto <branto@redhat.com>

mon/PGMap: reweight::by_utilization - skip DNE osds

EC could set one or more members of acting set to CRUSH_ITEM_NONE,
which as a result can cause pgs_by_osd.resize() attempt to apply
for a large amount of memory which we can not afford.

Fix the above problem by always excluding a current DNE osd from
acting set.

Fixes: http://tracker.ceph.com/issues/20970
Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>

Merge pull request #17043 from renhwztetecs/renhw-wip-pgmap-pool

mon/pgmap: update pool nearfull display

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

Merge pull request #17002 from mychoxin/yuanxin_adjust_blockdevice

os/bluestore: move aio_callback{,_priv} to base class BlockDevice

Reviewed-by: Pan Liu <liupan1111@gmail.com>
Reviewed-by: Kefu Chai <kchai@redhat.com>

Merge pull request #16940 from jcsp/wip-20955

mgr: add missing call to pick_addresses

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

Merge pull request #16407 from dzafman/wip-20243

osd: Improve size scrub error handling and ignore system attrs in xattr checking

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

common: add new line break when get command descriptions

Signed-off-by: songweibin <song.weibin@zte.com.cn>

rbd: adding newline operation when only '--format' used

Signed-off-by: songweibin <song.weibin@zte.com.cn>

test/librbd: TestMirroringWatcher unit tests should ignore duplicates

Fixes: http://tracker.ceph.com/issues/21029
Signed-off-by: Jason Dillaman <dillaman@redhat.com>

Merge pull request #17062 from trociny/wip-qa-rbd-mirror-data-pool

qa/workunits/rbd: test data pool is mirrored correctly

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

Merge pull request #16517 from gohighsec/master

rbd: check if an image is already mapped before rbd map

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

Merge pull request #16591 from PCzhangPC/fix7

rbd: introduce commands of "image-meta ls/rm"

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

Merge pull request #17073 from dillaman/wip-20567

rbd-mirror: Set the data pool correctly when creating images

Reviewed-by: Mykola Golub <mgolub@mirantis.com>
Reviewed-by: Jason Dillaman <dillaman@redhat.com>

doc: Update mirroring docs to mention data-pool configuration

Signed-off-by: Adam Wolfe Gordon <awg@digitalocean.com>
(cherry picked from commit 1e001c19e98ac1cd2428dd66f047ea4d6b2b01b1)

rbd-mirror: Set the data pool correctly when creating images

Previously the data-pool option was ignored by rbd-mirror, so when using
erasure coding data for the image on the secondary site would end up in
the metadata pool.

Configure the data-pool using the data-pool's name from the primary site
when creating images on the secondary site.

Fixes: http://tracker.ceph.com/issues/20567
Signed-off-by: Adam Wolfe Gordon <awg@digitalocean.com>
(cherry picked from commit dffaac83d19de544e279a530393b3998f1e1fbb4)

Merge pull request #17069 from yuriw/wip_add_luminous

Decreased amount of jobs on master, kraken, luminous runs

Merge pull request #17040 from mdw-at-linuxbox/rgw-master-fix-plus

Rgw master fix plus

rpm: rm macros in comments

rpm expands all macros in a .spec file, even those in comments. Drop the
percent signs so rpm will not expand these.

This change silences rpmlint's warning about macros in comments.

Signed-off-by: Ken Dreyer <kdreyer@redhat.com>

Merge pull request #17066 from amitkumar50/cov-os-1

os: Initializing uninitialized members aio_info

Reviewed-by: Greg Farnum <gfarnum@redhat.com>

os: Initializing uninitialized members aio_info

Fixes the coverity issue:

** 717332 Uninitialized pointer field
2. uninit_member: Non-static class member field iocb.data is not
initialized in this constructor nor in any functions that it calls.
4. uninit_member: Non-static class member field iocb.key is not
initialized in this constructor nor in any functions that it calls.
6. uninit_member: Non-static class member field iocb.__pad2 is not
initialized in this constructor nor in any functions that it calls.
8. uninit_member: Non-static class member field iocb.aio_lio_opcode
is not initialized in this constructor nor in any functions that it calls.
10. uninit_member: Non-static class member field iocb.aio_reqprio
is not initialized in this constructor nor in any functions that it calls.

CID 717332 (#1 of 1): Uninitialized pointer field (UNINIT_CTOR)
12. uninit_member: Non-static class member field iocb.aio_fildes is not
initialized in this constructor nor in any functions that it calls.

Signed-off-by: Amit Kumar amitkuma@redhat.com

osd/OpRequest: fix reqid assignment for reply messages

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

Decreased amount of jobs on master, kraken, luminous runs

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

Merge pull request #16896 from amitkumar50/cov-librado-2

rados: Initializing members of librados/TestCase.h

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

Merge pull request #16939 from trociny/wip-nonlinux-fsx

test/librbd: make fsx build on non-linux platform

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

osd: exit(1) directly without lock if init fails

if init fails to connect to a monitor, we just exit(1), there is no need
to set the `r` or lock the grand osd_lock for this. this change also
silences the warning of clang analyzer:

Value stored to 'r' is never read

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

Merge pull request #17027 from joscollin/wip-cleanup-Wsign-compare

test/osd: silence warnings from -Wsign-compare

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

qa/workunits/rbd: test data pool is mirrored correctly

Signed-off-by: Mykola Golub <mgolub@mirantis.com>

cmake: require lz4 v1.7 for building lz4 compressor

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

compressor: conditionalize on HAVE_LZ4

do not enable lz4 compressor plugin unless HAVE_LZ4 is ON. so, for
example, when the compressor type is "random", the compressor factory
won't complain with:

load failed dlopen(): "/usr/lib64/ceph/compressor/libceph_lz4.so: cannot
open shared object file: No such file or directory"

anymore.

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

Merge PR #16891 into master

* refs/remotes/upstream/pull/16891/head:
mds/MDSDaemon: add 'is_valid=false' when failed to parse caps

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

Merge pull request #17022 from ceph/wip-doc-sans-gitbuilder

Update docs to remove gitbuilder and add shaman references

Reviewed-By: Nathan Cutler <ncutler@suse.com>
Reviewed-by: Ken Dreyer <kdreyer@redhat.com>
Reviewed-by: Greg Farnum <gfarnum@redhat.com>

tests: Increase sleep in test_pidfile.sh

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

doc/rados/configuration comment out '...' on ini blocks

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

doc/rados/api comment out '...' on ini blocks

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

doc/install describe shaman urls and how to retrieve list and repo files

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

doc/install update international for non-US

Use of "international" is ambiguous, for non-US users, international
could mean US.

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

doc/install remove repetitive dev release information for rpm distros

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

doc/install remove mention of autobuild key for dev packages

Dev packages are no longer signed, and the key is not used at all

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

doc/install remove Apache and FastCGI instructions for RGW

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

doc: update repo-access to point to ceph-ci.git

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

rgw: evaluate the correct bucket action for GetACL bucket operation

We currently evaluate s3:GetObjectAcl instead of s3:GetBucketAcl for
getting BucketACL

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

rgw: rgw_op: exit early if object has no attrs in GetObjectTagging

Check for the return value of get_obj_attrs and exit earlier in case of
errors

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

Merge pull request #17025 from ceph/wip-doc-warnings

fix sphinx build warnings and errors

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

doc: fix short underline on release-notes

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

Merge pull request #15597 from ceph/wip-19200

selinux: Allow nvme devices

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

doc exclude man_index when not building man pages

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

doc: create an index/toctree only for man pages

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

doc: exclude mgr from man pages, define a man index.rst

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

doc/release-notes use new ref label for erasure code profiles

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

doc/rados/operations use new ref labels in health-checks

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

doc/rados/operations add a ref label to erasure code profiles

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

doc/rados/operations add a ref label to pool application

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

doc/rados/operations use new ref label in health-checks

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

doc/rados/operations add ref label for placement group number

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

doc/rados/api/python document required_state manually

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

admin: bump sphinx to 1.6

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

doc/rados/operations use the new ref label for crush map tunables

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

doc/rados/operations add label for crush map tunables for ref linking

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

doc/cephfs add label to health messages for use in refs linking

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

admin: do not allow errors or warnings from doc build

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