]> git.apps.os.sepia.ceph.com Git - ceph.git/log
ceph.git
4 years agoosd/ReplicatedBackend: handle tailing zeros of recovering objects nicer 38662/head
songweibin [Sat, 19 Dec 2020 10:50:31 +0000 (18:50 +0800)]
osd/ReplicatedBackend: handle tailing zeros of recovering objects nicer

E.g.,
(1) recovery_info.copy_subset = [0, 4194304]
(2) copy_subset(fiemap) = [0, 8192]
(3) out_op->data_included.span_of(copy_subset, 0, 4194304) = [0, 8192]

We'll set new_progress.data_recovered_to = 8192 as a result, and
fail to complete the push in one go.

Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
6 years agoMerge pull request #27694 from dillaman/wip-39386
Mykola Golub [Mon, 22 Apr 2019 15:13:02 +0000 (18:13 +0300)]
Merge pull request #27694 from dillaman/wip-39386

qa/suites/rbd: added writearound cache test permutations

Reviewed-by: Mykola Golub <mgolub@suse.com>
6 years agoMerge pull request #27682 from dillaman/wip-39031
Mykola Golub [Mon, 22 Apr 2019 15:12:30 +0000 (18:12 +0300)]
Merge pull request #27682 from dillaman/wip-39031

librbd: async open/close should free ImageCtx before issuing callback

Reviewed-by: Mykola Golub <mgolub@suse.com>
6 years agoMerge pull request #24744 from liewegas/wip-stale-prs
Sage Weil [Mon, 22 Apr 2019 14:58:24 +0000 (09:58 -0500)]
Merge pull request #24744 from liewegas/wip-stale-prs

.github/stale.yml: warn at 60, close at 90; adjust message

6 years agoMerge PR #27605 into master
Sage Weil [Mon, 22 Apr 2019 12:04:11 +0000 (07:04 -0500)]
Merge PR #27605 into master

* refs/pull/27605/head:
mon/OSDMonitor: osd add-no{up,down,in,out} - remove state checker

Reviewed-by: Sage Weil <sage@redhat.com>
6 years agoMerge pull request #27695 from dillaman/wip-39072 23653/head
Mykola Golub [Mon, 22 Apr 2019 05:22:20 +0000 (08:22 +0300)]
Merge pull request #27695 from dillaman/wip-39072

test/librbd: drop 'ceph_test_librbd_api' target

Reviewed-by: Mykola Golub <mgolub@suse.com>
6 years agoMerge PR #27288 into master
Sage Weil [Mon, 22 Apr 2019 02:28:02 +0000 (21:28 -0500)]
Merge PR #27288 into master

* refs/pull/27288/head:
osd/PG: discover missing objects when an OSD peers and PG is degraded

Reviewed-by: xie xingguo <xie.xingguo@zte.com.cn>
Reviewed-by: Neha Ojha <nojha@redhat.com>
6 years agoMerge PR #27571 into master
Sage Weil [Mon, 22 Apr 2019 02:26:04 +0000 (21:26 -0500)]
Merge PR #27571 into master

* refs/pull/27571/head:
mgr/telemetry: add report_timestamp to sent reports

Reviewed-by: Sage Weil <sage@redhat.com>
6 years agoMerge PR #27572 into master
Sage Weil [Mon, 22 Apr 2019 02:20:37 +0000 (21:20 -0500)]
Merge PR #27572 into master

* refs/pull/27572/head:
denc: fix build error by calling global snprintf

Reviewed-by: Kefu Chai <kchai@redhat.com>
6 years agoMerge PR #27622 into master
Sage Weil [Mon, 22 Apr 2019 02:20:12 +0000 (21:20 -0500)]
Merge PR #27622 into master

* refs/pull/27622/head:
mon: fix INCOMPAT_OCTOPUS feature number
mon/Elector: format mon_release correctly

Reviewed-by: Brad Hubbard <bhubbard@redhat.com>
Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
Reviewed-by: Neha Ojha <nojha@redhat.com>
6 years agoMerge pull request #27577 from dillaman/wip-rbd-suite-distros
Mykola Golub [Sun, 21 Apr 2019 17:19:15 +0000 (20:19 +0300)]
Merge pull request #27577 from dillaman/wip-rbd-suite-distros

qa/suites/rbd: add random distro selection to librbd tests

Reviewed-by: Mykola Golub <mgolub@suse.com>
6 years agoMerge pull request #27690 from tchaikov/wip-crimson-one-safeguard-is-better-than-two
Kefu Chai [Sun, 21 Apr 2019 03:17:27 +0000 (11:17 +0800)]
Merge pull request #27690 from tchaikov/wip-crimson-one-safeguard-is-better-than-two

crimson: pass `Connection*` to Dispatch::ms_dispatch()

Reviewed-by: Samuel Just <sjust@redhat.com>
6 years agoMerge PR #27595 into master
Sage Weil [Sat, 20 Apr 2019 13:52:01 +0000 (08:52 -0500)]
Merge PR #27595 into master

* refs/pull/27595/head:
osd: add 'ceph osd stop <osd.nnn>' command

Reviewed-by: Sage Weil <sage@redhat.com>
6 years agoMerge PR #27619 into master
Sage Weil [Sat, 20 Apr 2019 13:48:30 +0000 (08:48 -0500)]
Merge PR #27619 into master

* refs/pull/27619/head:
mgr/BaseMgrModule: run MonCommandCompletion on the finisher
mgr/BaseMgrModule: fix leak

Reviewed-by: Mykola Golub <mgolub@suse.com>
Reviewed-by: Tim Serong <tserong@suse.com>
6 years agoMerge PR #27651 into master
Sage Weil [Sat, 20 Apr 2019 13:48:14 +0000 (08:48 -0500)]
Merge PR #27651 into master

* refs/pull/27651/head:
test/osd/safe-to-destroy.sh: fix typo

Reviewed-by: xie xingguo <xie.xingguo@zte.com.cn>
Reviewed-by: Kefu Chai <kchai@redhat.com>
6 years agoMerge PR #27663 into master
Sage Weil [Sat, 20 Apr 2019 13:47:48 +0000 (08:47 -0500)]
Merge PR #27663 into master

* refs/pull/27663/head:
osd/OSDMap: Replace get_out_osds with get_out_existing_osds

Reviewed-by: xie xingguo <xie.xingguo@zte.com.cn>
Reviewed-by: Sage Weil <sage@redhat.com>
6 years agoMerge pull request #27631 from dmick/wip-telemetry-crasharray
Kefu Chai [Sat, 20 Apr 2019 06:18:28 +0000 (14:18 +0800)]
Merge pull request #27631 from dmick/wip-telemetry-crasharray

mgr/telemetry: change crash dict to a list

Reviewed-by: Sage Weil <sage@redhat.com>
6 years agoMerge pull request #27599 from gregsfortytwo/wip-stale-prs
Kefu Chai [Sat, 20 Apr 2019 06:16:33 +0000 (14:16 +0800)]
Merge pull request #27599 from gregsfortytwo/wip-stale-prs

Miscellaneous lost fixes

Reviewed-by: Ricardo Dias <rdias@suse.com>
6 years agoMerge pull request #27615 from ifed01/wip-ifed-verbose-bluefs-replay
Kefu Chai [Sat, 20 Apr 2019 06:12:20 +0000 (14:12 +0800)]
Merge pull request #27615 from ifed01/wip-ifed-verbose-bluefs-replay

os/bluestore: be more verbose doing bluefs log replay.

Reviewed-by: Sage Weil <sage@redhat.com>
6 years agoMerge pull request #27509 from Aran85/fix-pg-dout
Kefu Chai [Sat, 20 Apr 2019 06:10:39 +0000 (14:10 +0800)]
Merge pull request #27509 from Aran85/fix-pg-dout

osd/PG: the warning seems more serious than what it wanna transmit

Reviewed-by: Neha Ojha <nojha@redhat.com>
6 years agoMerge pull request #27592 from tchaikov/wip-ref
Kefu Chai [Sat, 20 Apr 2019 06:09:51 +0000 (14:09 +0800)]
Merge pull request #27592 from tchaikov/wip-ref

message,mgr: drop MessageFactory and friends and use ref_t<> in mgr

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
6 years agoMerge pull request #27633 from xiexingguo/wip-39340
Kefu Chai [Sat, 20 Apr 2019 06:05:24 +0000 (14:05 +0800)]
Merge pull request #27633 from xiexingguo/wip-39340

osd/osd_types: pool_stat_t::dump - fix 'num_store_stats' field

Reviewed-by: Sage Weil <sage@redhat.com>
6 years agoMerge pull request #27658 from gregsfortytwo/wip-39367-crc-peer
Kefu Chai [Sat, 20 Apr 2019 06:04:33 +0000 (14:04 +0800)]
Merge pull request #27658 from gregsfortytwo/wip-39367-crc-peer

msg: output peer address when detecting bad CRCs

Reviewed-by: Kefu Chai <kchai@redhat.com>
6 years agocrimson: pass `Connection*` to Dispatch::ms_dispatch() 27690/head
Kefu Chai [Fri, 19 Apr 2019 08:50:25 +0000 (16:50 +0800)]
crimson: pass `Connection*` to Dispatch::ms_dispatch()

currently, we use a `with_gate()` in `ProtocolV2::read_message()` for
ensuring that `this` (or `Connection` holding this protocol instance)
will outlive the continuation of `dispatcher.ms_dispatch()` which
references `this->dispatcher`. but we also pass a strong reference of
connection to dispatcher. in short, we have *two* safeguards for the
same purpose.

in this change, one of these safeguards is removed -- to pass the raw
pointer of `Connection` to `Dispatch::ms_dispatch()`. the reason why
the `with_gate()` is kept is that, if we have removed `with_gate()` in
Protocol, we need to

1. let `Dispatcher::ms_dispatch()` return `void`, as it should not block
any succeeding calls.
2. add a `with_gate()` in `Dispatcher::ms_dispatch()` to ensure that
`this` is alive during the lifecycle of the continuation(s) in
`Dispatcher::ms_dispatch()`.

Signed-off-by: Yingxin Cheng <yingxincheng@gmail.com>
Signed-off-by: Kefu Chai <kchai@redhat.com>
6 years agoMerge pull request #27657 from anthonyeleven/patch-1
Xie Xingguo [Sat, 20 Apr 2019 00:50:29 +0000 (08:50 +0800)]
Merge pull request #27657 from anthonyeleven/patch-1

doc: operations: improve reweight-by-utilization

Reviewed-by: xie xingguo <xie.xingguo@zte.com.cn>
6 years agodoc: operations: reweight-by-utilization typo 27657/head
Anthony D'Atri [Fri, 19 Apr 2019 22:15:12 +0000 (15:15 -0700)]
doc: operations: reweight-by-utilization typo

Add a missing backquote delimiter.

Signed-off-by: Anthony D'Atri <anthony.datri@gmail.com>
6 years agotest/librbd: drop 'ceph_test_librbd_api' target 27695/head
Jason Dillaman [Fri, 19 Apr 2019 18:15:49 +0000 (14:15 -0400)]
test/librbd: drop 'ceph_test_librbd_api' target

This was used for API backwards compatibility testing, but now that
the C++ API will not remain stable, it serves no purpose.

Fixes: http://tracker.ceph.com/issues/39072
Signed-off-by: Jason Dillaman <dillaman@redhat.com>
6 years agoqa/suites/rbd: added writearound cache test permutations 27694/head
Jason Dillaman [Fri, 19 Apr 2019 18:11:43 +0000 (14:11 -0400)]
qa/suites/rbd: added writearound cache test permutations

Also tweaked the existing writeback cache test permutations to
specify the use of the writeback cache policy.

Fixes: http://tracker.ceph.com/issues/39386
Signed-off-by: Jason Dillaman <dillaman@redhat.com>
6 years agomgr/BaseMgrModule: run MonCommandCompletion on the finisher 27619/head
Sage Weil [Thu, 18 Apr 2019 18:10:15 +0000 (13:10 -0500)]
mgr/BaseMgrModule: run MonCommandCompletion on the finisher

The completion takes the ActivePyModules::lock, but the caller may be
holding an internal lock for Objecter and/or MonClient.  Avoid deadlock
by putting the completion on our finisher.

Signed-off-by: Sage Weil <sage@redhat.com>
6 years agomgr/BaseMgrModule: fix leak
Sage Weil [Thu, 18 Apr 2019 18:28:05 +0000 (13:28 -0500)]
mgr/BaseMgrModule: fix leak

Signed-off-by: Sage Weil <sage@redhat.com>
6 years agoMerge PR #27685 into master
Patrick Donnelly [Thu, 18 Apr 2019 22:36:17 +0000 (15:36 -0700)]
Merge PR #27685 into master

* refs/pull/27685/head:
doc: fix boot transition in mds state diagram

Reviewed-by: Sage Weil <sage@redhat.com>
6 years agoMerge pull request #27686 from dzafman/wip-39383
David Zafman [Thu, 18 Apr 2019 21:52:30 +0000 (14:52 -0700)]
Merge pull request #27686 from dzafman/wip-39383

osd: Increase log level of messages which unnecessarily fill up logs

Reviewed-by: Neha Ojha <nojha@redhat.com>
6 years agoosd: Increase log level of messages which unnecessarily fill up logs 27686/head
David Zafman [Thu, 18 Apr 2019 21:03:17 +0000 (14:03 -0700)]
osd: Increase log level of messages which unnecessarily fill up logs

Caused by: 834d3c19a774f1cc93903447d91d182776e12d18

Fixes: https://tracker.ceph.com/issues/39383
Signed-off-by: David Zafman <dzafman@redhat.com>
6 years agoMerge pull request #27037 from alimaredia/wip-rgw-dpp-rest-s3-swift
Casey Bodley [Thu, 18 Apr 2019 19:57:23 +0000 (15:57 -0400)]
Merge pull request #27037 from alimaredia/wip-rgw-dpp-rest-s3-swift

rgw: log refactoring for rgw_rest_s3/swift ops

Reviewed-by: Casey Bodley <cbodley@redhat.com>
6 years agodoc: fix boot transition in mds state diagram 27685/head
Patrick Donnelly [Thu, 18 Apr 2019 19:48:37 +0000 (12:48 -0700)]
doc: fix boot transition in mds state diagram

MDSMonitor assigns a booting MDS immediately to standby. From there, the
standby may be promoted to standby-replay or some state leading to active.

Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
6 years agoMerge PR #27375 into master
Sage Weil [Thu, 18 Apr 2019 19:09:51 +0000 (14:09 -0500)]
Merge PR #27375 into master

* refs/pull/27375/head:
rgw: pass mostly_omap flag when opening/creating pools
rgw/rgw_rados: pass mostly_omap flag when opening/creating pools
rgw: move rgw_init_ioctx() to rgw_tools.cc

Reviewed-by: Casey Bodley <cbodley@redhat.com>
6 years agoMerge pull request #27576 from cbodley/wip-39287
Casey Bodley [Thu, 18 Apr 2019 18:02:13 +0000 (14:02 -0400)]
Merge pull request #27576 from cbodley/wip-39287

rgw: RGWPeriodPusher uses zone system key for inter-zonegroup messages

Reviewed-by: Matt Benjamin <mbenjamin@redhat.com>
6 years agorbd: fixed memory leak in 'rbd mirror pool' multi-image actions 27682/head
Jason Dillaman [Thu, 18 Apr 2019 17:20:17 +0000 (13:20 -0400)]
rbd: fixed memory leak in 'rbd mirror pool' multi-image actions

Signed-off-by: Jason Dillaman <dillaman@redhat.com>
6 years agolibrbd: async open/close should free ImageCtx before issuing callback
Jason Dillaman [Thu, 18 Apr 2019 17:17:56 +0000 (13:17 -0400)]
librbd: async open/close should free ImageCtx before issuing callback

The destructor for ImageCtx attempts to access librados::IoCtx objects
that might have been destroyed immediately after the async open (failure)
and close actions completed.

Fixes: http://tracker.ceph.com/issues/39031
Signed-off-by: Jason Dillaman <dillaman@redhat.com>
6 years agoMerge pull request #27518 from jtlayton/vstart
jtlayton [Thu, 18 Apr 2019 17:16:02 +0000 (13:16 -0400)]
Merge pull request #27518 from jtlayton/vstart

vstart: add new option to pass list of block devices to bluestore

6 years agoMerge pull request #27567 from cbodley/wip-39283
Casey Bodley [Thu, 18 Apr 2019 13:31:41 +0000 (09:31 -0400)]
Merge pull request #27567 from cbodley/wip-39283

rgw: multisite log trimming only checks peers that sync from us

Reviewed-by: Yehuda Sadeh <yehuda@redhat.com>
6 years agoqa/tasks/rbd_fio: fixed missing delimiter between 'cd' and 'configure' 27577/head
Jason Dillaman [Thu, 18 Apr 2019 12:55:19 +0000 (08:55 -0400)]
qa/tasks/rbd_fio: fixed missing delimiter between 'cd' and 'configure'

Signed-off-by: Jason Dillaman <dillaman@redhat.com>
6 years agoqa/suites/rbd: add random distro selection to librbd tests
Jason Dillaman [Sat, 13 Apr 2019 16:53:35 +0000 (12:53 -0400)]
qa/suites/rbd: add random distro selection to librbd tests

Signed-off-by: Jason Dillaman <dillaman@redhat.com>
6 years agoMerge pull request #27275 from Devp00l/issue-39037
Lenz Grimmer [Thu, 18 Apr 2019 08:58:43 +0000 (10:58 +0200)]
Merge pull request #27275 from Devp00l/issue-39037

mgr/dashboard: Localization for date picker module

Reviewed-by: Alfonso Martínez <almartin@redhat.com>
Reviewed-by: Tiago Melo <tmelo@suse.com>
6 years agoosd: add 'ceph osd stop <osd.nnn>' command 27595/head
xie xingguo [Mon, 15 Apr 2019 12:36:36 +0000 (20:36 +0800)]
osd: add 'ceph osd stop <osd.nnn>' command

stop command can be used to force stopping a specified osd daemon, e.g.,
you don't have to pre-figure out where it located.

Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
6 years agoosd/OSDMap: Replace get_out_osds with get_out_existing_osds 27663/head
Brad Hubbard [Wed, 17 Apr 2019 06:13:27 +0000 (16:13 +1000)]
osd/OSDMap: Replace get_out_osds with get_out_existing_osds

Fixes: http://tracker.ceph.com/issues/39154
Signed-off-by: Brad Hubbard <bhubbard@redhat.com>
6 years agoMerge PR #27235 into master
Patrick Donnelly [Thu, 18 Apr 2019 00:06:22 +0000 (17:06 -0700)]
Merge PR #27235 into master

* refs/pull/27235/head:
cephfs-shell: support non-octal mode
cephfs-shell: fix rm command
cephfs-shell: fix lls command
cephfs-shell: fix lcd command
cephfs-shell: fix umask command
cephfs-shell: fix cat command
cephfs-shell: fix chmod command
cephfs-shell: fix chdir command
cephfs-shell: fix mv command
cephfs-shell: fix rmdir command
cephfs-shell: fix ls command
cephfs-shell: fix get command
cephfs-shell: fix put command
cephfs-shell: fix mkdir command
cephfs-shell: helper functions and infra changes

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
6 years agomsg/Message: add SafeMessage which forbids put/get 27592/head
Kefu Chai [Tue, 16 Apr 2019 02:46:36 +0000 (10:46 +0800)]
msg/Message: add SafeMessage which forbids put/get

This is an opt-in wrapper around Message inheritance to prevent
undesired get/put calls on the object. Instead, you must/should use a
smart ptr to adjust the reference count.

Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
Signed-off-by: Kefu Chai <kchai@redhat.com>
6 years agomgr/MgrClient: use ref_t<M>
Kefu Chai [Mon, 15 Apr 2019 06:55:24 +0000 (14:55 +0800)]
mgr/MgrClient: use ref_t<M>

Signed-off-by: Kefu Chai <kchai@redhat.com>
6 years agomgr/{Mgr,MgrStandby}: use ref_t<M>
Kefu Chai [Mon, 15 Apr 2019 06:39:11 +0000 (14:39 +0800)]
mgr/{Mgr,MgrStandby}: use ref_t<M>

Signed-off-by: Kefu Chai <kchai@redhat.com>
6 years agomgr/DaemonState: pass const by reference
Kefu Chai [Mon, 15 Apr 2019 06:02:33 +0000 (14:02 +0800)]
mgr/DaemonState: pass const by reference

DaemonPerfCounters::update() does not change the `MMgrReport` message,
neither does it expect a nullptr. so pass by const reference.

Signed-off-by: Kefu Chai <kchai@redhat.com>
6 years agomgr/DaemonServer: use ref_t<M>
Kefu Chai [Mon, 15 Apr 2019 06:01:00 +0000 (14:01 +0800)]
mgr/DaemonServer: use ref_t<M>

to avoid explicit use of RefCountedObj interface. and let
intrusive_ptr<> to take care of the life cycle of messages.

Signed-off-by: Kefu Chai <kchai@redhat.com>
6 years agomsg/Message: remove Message::const_ref
Kefu Chai [Mon, 15 Apr 2019 02:26:40 +0000 (10:26 +0800)]
msg/Message: remove Message::const_ref

Signed-off-by: Kefu Chai <kchai@redhat.com>
6 years agocommon: use ref_t<M>
Kefu Chai [Tue, 16 Apr 2019 04:13:48 +0000 (12:13 +0800)]
common: use ref_t<M>

Signed-off-by: Kefu Chai <kchai@redhat.com>
6 years agomsg/Message: drop MessageFactory and friends
Kefu Chai [Sun, 14 Apr 2019 15:33:50 +0000 (23:33 +0800)]
msg/Message: drop MessageFactory and friends

we'd better decouple the helpers from the definition of Message types,
to keep the definition Messages simple and concise.

Signed-off-by: Kefu Chai <kchai@redhat.com>
6 years agomessages/MOSDPeeringOp: do not include PGPeeringEvent.h
Kefu Chai [Sun, 14 Apr 2019 15:32:41 +0000 (23:32 +0800)]
messages/MOSDPeeringOp: do not include PGPeeringEvent.h

forward delcaration would suffice

Signed-off-by: Kefu Chai <kchai@redhat.com>
6 years agomessages: do not use MessageSubType anymore
Kefu Chai [Sun, 14 Apr 2019 15:31:24 +0000 (23:31 +0800)]
messages: do not use MessageSubType anymore

so we can drop it in future commit

Signed-off-by: Kefu Chai <kchai@redhat.com>
6 years agomessages: #include necessary header
Kefu Chai [Sun, 14 Apr 2019 15:28:28 +0000 (23:28 +0800)]
messages: #include necessary header

for instance, MLogRec is defined in PGPeeringEvent.h, we should not rely
other header(s) to do this for us. so just #include PGPeeringEvent.h for
its definition.

Signed-off-by: Kefu Chai <kchai@redhat.com>
6 years agomds/MDSCache: init empty string_view with {}
Kefu Chai [Sun, 14 Apr 2019 07:49:00 +0000 (15:49 +0800)]
mds/MDSCache: init empty string_view with {}

nullptr is not a valid null-terminated character sequence

Signed-off-by: Kefu Chai <kchai@redhat.com>
6 years agosrc/: use make_message<Foo> instead of Foo::create()
Kefu Chai [Sun, 14 Apr 2019 07:01:57 +0000 (15:01 +0800)]
src/: use make_message<Foo> instead of Foo::create()

Signed-off-by: Kefu Chai <kchai@redhat.com>
6 years agomsg/Message: remove msgref_cast()
Kefu Chai [Sun, 14 Apr 2019 06:19:58 +0000 (14:19 +0800)]
msg/Message: remove msgref_cast()

Signed-off-by: Kefu Chai <kchai@redhat.com>
6 years agomsg/Message: add aliases for static_pointer_cast
Kefu Chai [Sun, 14 Apr 2019 06:03:28 +0000 (14:03 +0800)]
msg/Message: add aliases for static_pointer_cast

strictly speaking, they are not "alias" of boost::static_pointer_cast,
but to avoid an extra conversion, instead of using
boost::static_pointer_cast<>, i am using std::static_cast<> directly for
creating boost::intrusive_ptr<> instance.

Signed-off-by: Kefu Chai <kchai@redhat.com>
6 years agoclient,mds: use ref_cast<Foo> instead of Foo::msgref_cast
Kefu Chai [Sun, 14 Apr 2019 04:34:24 +0000 (12:34 +0800)]
client,mds: use ref_cast<Foo> instead of Foo::msgref_cast

Signed-off-by: Kefu Chai <kchai@redhat.com>
6 years agosrc/: use ref_t<Message> instead of Message::ref
Kefu Chai [Sun, 14 Apr 2019 05:18:00 +0000 (13:18 +0800)]
src/: use ref_t<Message> instead of Message::ref

Signed-off-by: Kefu Chai <kchai@redhat.com>
6 years agosrc/: use cref_t<Message> instead of Message::const_ref
Kefu Chai [Sun, 14 Apr 2019 04:33:57 +0000 (12:33 +0800)]
src/: use cref_t<Message> instead of Message::const_ref

Signed-off-by: Kefu Chai <kchai@redhat.com>
6 years agocommon/RefCountedObj: move private members
Patrick Donnelly [Wed, 6 Feb 2019 19:44:04 +0000 (11:44 -0800)]
common/RefCountedObj: move private members

To follow code-style guidelines.

Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
6 years agocommon/RefCountedObj: use type of nref
Patrick Donnelly [Wed, 6 Feb 2019 17:42:27 +0000 (09:42 -0800)]
common/RefCountedObj: use type of nref

"int" may narrow the ref count (although very unlikely!).

Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
6 years agomsg: add aliases of ref_t and cref_t
Kefu Chai [Sun, 14 Apr 2019 05:18:27 +0000 (13:18 +0800)]
msg: add aliases of ref_t and cref_t

Signed-off-by: Kefu Chai <kchai@redhat.com>
6 years agoMerge PR #27649 into master
Sage Weil [Wed, 17 Apr 2019 22:41:38 +0000 (17:41 -0500)]
Merge PR #27649 into master

* refs/pull/27649/head:
PendingReleaseNotes: 14.2.1 note on crush required version

Reviewed-by: Greg Farnum <gfarnum@redhat.com>
6 years agomsg: output peer address when detecting bad CRCs 27658/head
Greg Farnum [Wed, 17 Apr 2019 22:27:22 +0000 (15:27 -0700)]
msg: output peer address when detecting bad CRCs

Fixes: http://tracker.ceph.com/issues/39367
Signed-off-by: Greg Farnum <gfarnum@redhat.com>
6 years agoMerge pull request #27600 from neha-ojha/wip-39006
Neha Ojha [Wed, 17 Apr 2019 22:21:17 +0000 (15:21 -0700)]
Merge pull request #27600 from neha-ojha/wip-39006

osd/OSD.cc: make osd bench description consistent with parameters

Reviewed-by: Sage Weil <sage@redhat.com>
6 years agoMerge PR #27481 into master
Patrick Donnelly [Wed, 17 Apr 2019 22:04:09 +0000 (15:04 -0700)]
Merge PR #27481 into master

* refs/pull/27481/head:
qa: add a method to test if a file can be created
qa: add wrappers for vstart's run() with check_status as False
qa: add a method to create an empty file with any user
qa: add stdin parameter to run_shell
qa: command to be ran as a different user should be a single string
qa: looks for quotes in Python shell and get rid of them
qa: move checks on command arguments to a different method
qa: accept args as strings
qa: add a method to run given command as root user
qa: add a method to run a command as a different user
qa: allow keeping sudo in command arguments
qa: make execution abortion optional for command ran through run_shell()

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
Reviewed-by: Jeff Layton <jlayton@redhat.com>
6 years agodoc: operations: improve reweight-by-utilization
Anthony D'Atri [Wed, 17 Apr 2019 21:48:33 +0000 (14:48 -0700)]
doc: operations: improve reweight-by-utilization

Add the missing `max_change`, `max_osds`, and `--no-increasing` parameters to `reweight-by-utilization` and `test-reweight-by-utilization`.  Minor adjustments to wording.

Signed-off-by: Anthony D'Atri <anthony.datri@gmail.com>
6 years agoMerge pull request #27282 from mogeb/wip-lvmcache
Andrew Schoen [Wed, 17 Apr 2019 20:49:44 +0000 (15:49 -0500)]
Merge pull request #27282 from mogeb/wip-lvmcache

ceph-volume: add utility functions

Reviewed-by: Andrew Schoen <aschoen@redhat.com>
6 years agoqa: add a method to test if a file can be created 27481/head
Rishabh Dave [Sun, 3 Mar 2019 19:29:30 +0000 (00:59 +0530)]
qa: add a method to test if a file can be created

Signed-off-by: Rishabh Dave <ridave@redhat.com>
6 years agoqa: add wrappers for vstart's run() with check_status as False
Rishabh Dave [Wed, 13 Mar 2019 18:40:16 +0000 (00:10 +0530)]
qa: add wrappers for vstart's run() with check_status as False

Too often in tests it is required to run a command that we want to (or
will probably) fail so that we can check if it failed for expected
reason(s) (eg. a testcase where command must fail with "Permission
denied"). Therefore, add few wrapper methods for vstart_runner.py's
run() with check_status's default value as False so that we don't need
to pass check_status's value at every call to run() or it's other
wrappers.

Signed-off-by: Rishabh Dave <ridave@redhat.com>
6 years agoqa: add a method to create an empty file with any user
Rishabh Dave [Mon, 25 Feb 2019 19:54:58 +0000 (01:24 +0530)]
qa: add a method to create an empty file with any user

Signed-off-by: Rishabh Dave <ridave@redhat.com>
6 years agoqa: add stdin parameter to run_shell
Rishabh Dave [Thu, 21 Feb 2019 16:43:28 +0000 (22:13 +0530)]
qa: add stdin parameter to run_shell

Signed-off-by: Rishabh Dave <ridave@redhat.com>
6 years agoqa: command to be ran as a different user should be a single string
Rishabh Dave [Fri, 12 Apr 2019 11:56:45 +0000 (17:26 +0530)]
qa: command to be ran as a different user should be a single string

Signed-off-by: Rishabh Dave <ridave@redhat.com>
6 years agotest/osd/safe-to-destroy.sh: fix typo 27651/head
Sage Weil [Wed, 17 Apr 2019 15:53:54 +0000 (10:53 -0500)]
test/osd/safe-to-destroy.sh: fix typo

Signed-off-by: Sage Weil <sage@redhat.com>
6 years agomsg/async: add async msg perf counter 27599/head
Xinze Chi [Sun, 30 Jul 2017 00:18:53 +0000 (08:18 +0800)]
msg/async: add async msg perf counter

Signed-off-by: Xinze Chi <xinze@xsky.com>
Updated to account for the new ProtocolV1/ProtocolV2

Signed-off-by: Greg Farnum <gfarnum@redhat.com>
6 years agotools/crushtool: add new Option --bucket-tree to show bucket map tree
linbing [Wed, 20 Dec 2017 00:52:12 +0000 (08:52 +0800)]
tools/crushtool: add new Option --bucket-tree to show bucket map tree

Signed-off-by: linbing <linbing@t2cloud.net>
Update the crushtool cli test template for new options.

Signed-off-by: Greg Farnum <gfarnum@redhat.com>
6 years agomsg/async: bug fix 23600 by setting _fd = 1.
shangfufei [Mon, 9 Apr 2018 06:56:24 +0000 (14:56 +0800)]
msg/async: bug fix 23600 by setting _fd = 1.

After listen_socket was closed, it entered the start() processf magically. so
I think we should make '_fd = -1' when listen_socket was closed. And in the
start process, determine the value of listen_socket, if it equal to -1, return
the process.

Fixes: https://tracker.ceph.com/issues/23600
Signed-off-by: shangfufei <shangfufei@inspur.com>
Signed-off-by: Greg Farnum <gfarnum@redhat.com>
6 years agoos/bluestore: be more verbose doing bluefs log replay. 27615/head
Igor Fedotov [Tue, 16 Apr 2019 15:58:53 +0000 (18:58 +0300)]
os/bluestore: be more verbose doing bluefs log replay.

Signed-off-by: Igor Fedotov <ifedotov@suse.com>
6 years agoPendingReleaseNotes: 14.2.1 note on crush required version 27649/head
Sage Weil [Wed, 17 Apr 2019 13:51:29 +0000 (08:51 -0500)]
PendingReleaseNotes: 14.2.1 note on crush required version

Signed-off-by: Sage Weil <sage@redhat.com>
6 years agoMerge pull request #27581 from LenzGr/master-documentation
Ricardo Marques [Wed, 17 Apr 2019 09:03:42 +0000 (10:03 +0100)]
Merge pull request #27581 from LenzGr/master-documentation

doc: Improved the dashboard proxy config section

Reviewed-by: Alfonso Martínez <almartin@redhat.com>
Reviewed-by: Nathan Cutler <ncutler@suse.com>
6 years agoosd/osd_types: pool_stat_t::dump - fix 'num_store_stats' field 27633/head
xie xingguo [Wed, 17 Apr 2019 05:46:07 +0000 (13:46 +0800)]
osd/osd_types: pool_stat_t::dump - fix 'num_store_stats' field

Fixes: http://tracker.ceph.com/issues/39340
Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
6 years agoMerge pull request #27593 from smithfarm/wip-39291
Nathan Cutler [Wed, 17 Apr 2019 05:41:59 +0000 (07:41 +0200)]
Merge pull request #27593 from smithfarm/wip-39291

cmake: revert librados_tp.so version from 3 to 2

Reviewed-by: Kefu Chai <kchai@redhat.com>
Reviewed-by: Tim Serong <tserong@suse.com>
6 years agomgr/telemetry: change crash dict to a list 27631/head
Dan Mick [Wed, 17 Apr 2019 04:02:11 +0000 (21:02 -0700)]
mgr/telemetry: change crash dict to a list

For ease of indexing.  There's no value in timestamp keys.

Signed-off-by: Dan Mick <dan.mick@redhat.com>
6 years agoMerge pull request #27514 from tchaikov/wip-crimson-auth-client
Kefu Chai [Wed, 17 Apr 2019 00:17:48 +0000 (08:17 +0800)]
Merge pull request #27514 from tchaikov/wip-crimson-auth-client

crimson: enable cephx for v2 msgr

Reviewed-by: Samuel Just <sjust@redhat.com>
Reviewed-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
6 years agoMerge PR #27624 into master
Patrick Donnelly [Tue, 16 Apr 2019 23:53:47 +0000 (16:53 -0700)]
Merge PR #27624 into master

* refs/pull/27624/head:
doc: fixed telemetry module reference link

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
6 years agocrimson/auth: let AuthClient::get_auth_request return a named struct 27514/head
Kefu Chai [Tue, 16 Apr 2019 14:30:12 +0000 (22:30 +0800)]
crimson/auth: let AuthClient::get_auth_request return a named struct

better readability this way

Signed-off-by: Kefu Chai <kchai@redhat.com>
6 years agocrimson/mon: use const reference for Connection::auth_registry
Kefu Chai [Tue, 16 Apr 2019 13:43:02 +0000 (21:43 +0800)]
crimson/mon: use const reference for Connection::auth_registry

it is not optional now.

Signed-off-by: Kefu Chai <kchai@redhat.com>
6 years agoauth/AuthRegistry: more constness
Kefu Chai [Tue, 16 Apr 2019 12:13:03 +0000 (20:13 +0800)]
auth/AuthRegistry: more constness

Signed-off-by: Kefu Chai <kchai@redhat.com>
6 years agocrimson/mon: remove "#warning" in v1 implementation
Kefu Chai [Thu, 11 Apr 2019 13:47:29 +0000 (21:47 +0800)]
crimson/mon: remove "#warning" in v1 implementation

since we've implemented the v2 support, these reminders are not needed
anymore.

Signed-off-by: Kefu Chai <kchai@redhat.com>
6 years agocrimson/mon: reset promise using `{}`
Kefu Chai [Thu, 11 Apr 2019 13:44:57 +0000 (21:44 +0800)]
crimson/mon: reset promise using `{}`

better readability, less repeating.

Signed-off-by: Kefu Chai <kchai@redhat.com>
6 years agomon/MonClient: remove unused AuthMethodList
Kefu Chai [Thu, 11 Apr 2019 11:40:38 +0000 (19:40 +0800)]
mon/MonClient: remove unused AuthMethodList

Signed-off-by: Kefu Chai <kchai@redhat.com>
6 years agocrimson/mon: use auth_registry for querying auth_methods
Kefu Chai [Thu, 11 Apr 2019 11:39:32 +0000 (19:39 +0800)]
crimson/mon: use auth_registry for querying auth_methods

Signed-off-by: Kefu Chai <kchai@redhat.com>
6 years agodoc: fixed telemetry module reference link 27624/head
James McClune [Tue, 16 Apr 2019 22:19:38 +0000 (18:19 -0400)]
doc: fixed telemetry module reference link

Used :ref: label for telemetry reference in Nautilus.

Fixes: https://tracker.ceph.com/issues/39337
Signed-off-by: James McClune <jmcclune@mcclunetechnologies.net>
6 years agomon: fix INCOMPAT_OCTOPUS feature number 27622/head
Sage Weil [Tue, 16 Apr 2019 22:16:14 +0000 (17:16 -0500)]
mon: fix INCOMPAT_OCTOPUS feature number

It's overlapping with nautilus, which is a problem!

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