]> git.apps.os.sepia.ceph.com Git - ceph.git/log
ceph.git
7 years agodoc/release-notes: update device class cli 16851/head
xie xingguo [Mon, 7 Aug 2017 02:04:35 +0000 (10:04 +0800)]
doc/release-notes: update device class cli

These commands are nearly stable and are unlikely to change for now...

Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
7 years agoMerge pull request #14053 from jdurgin/wip-19191
Sage Weil [Sun, 6 Aug 2017 19:08:34 +0000 (14:08 -0500)]
Merge pull request #14053 from jdurgin/wip-19191

osd/PG: reset the missing set when restarting backfill

Reviewed-by: David Zafman <dzafman@redhat.com>
Reviewed-by: Sage Weil <sage@redhat.com>
7 years agoMerge pull request #16837 from xiexingguo/wip-still-more-class-fixes
Sage Weil [Sun, 6 Aug 2017 19:07:33 +0000 (14:07 -0500)]
Merge pull request #16837 from xiexingguo/wip-still-more-class-fixes

crush: more class fixes

Reviewed-by: Sage Weil <sage@redhat.com>
7 years agoMerge pull request #16838 from xiexingguo/wip-fix-purge
Sage Weil [Sun, 6 Aug 2017 19:07:15 +0000 (14:07 -0500)]
Merge pull request #16838 from xiexingguo/wip-fix-purge

mon/OSDMonitor: sanity check osd before performing 'osd purge'

Reviewed-by: Sage Weil <sage@redhat.com>
7 years agoMerge pull request #16832 from liewegas/wip-20915
Sage Weil [Sun, 6 Aug 2017 18:41:21 +0000 (13:41 -0500)]
Merge pull request #16832 from liewegas/wip-20915

rpm,deb: fix ceph-volume

Reviewed-by: Nathan Cutler <ncutler@suse.com>
7 years agoMerge pull request #16849 from liewegas/wip-jewel-x
Sage Weil [Sun, 6 Aug 2017 14:15:35 +0000 (09:15 -0500)]
Merge pull request #16849 from liewegas/wip-jewel-x

qa/suites/upgarde/jewel-x/parallel: more whitelisting

7 years agoqa/suites/upgarde/jewel-x/parallel: more whitelisting 16849/head
Sage Weil [Sun, 6 Aug 2017 14:04:14 +0000 (10:04 -0400)]
qa/suites/upgarde/jewel-x/parallel: more whitelisting

Signed-off-by: Sage Weil <sage@redhat.com>
7 years agoMerge pull request #16830 from liewegas/wip-jewel-x
Sage Weil [Sun, 6 Aug 2017 13:57:55 +0000 (08:57 -0500)]
Merge pull request #16830 from liewegas/wip-jewel-x

upgrade/jewel-x: a few fixes

Reviewed-by: Josh Durgin <jdurgin@redhat.com>
7 years agoqa/suites/upgrade/jewel-x/parallel: more whitelisting 16830/head
Sage Weil [Sat, 5 Aug 2017 18:44:05 +0000 (14:44 -0400)]
qa/suites/upgrade/jewel-x/parallel: more whitelisting

Signed-off-by: Sage Weil <sage@redhat.com>
7 years agoqa/suites/upgrade/*-x/parallel: whitelist more stuff
Sage Weil [Sat, 5 Aug 2017 02:28:54 +0000 (22:28 -0400)]
qa/suites/upgrade/*-x/parallel: whitelist more stuff

Signed-off-by: Sage Weil <sage@redhat.com>
7 years agoqa/suites/upgrade/*/parallel: disable POOL_APP_NOT_ENABLED
Sage Weil [Sat, 5 Aug 2017 02:25:52 +0000 (22:25 -0400)]
qa/suites/upgrade/*/parallel: disable POOL_APP_NOT_ENABLED

There is some other random workload running (that creates pools)
while we upgrade and wait for healthy.  Just disable the warning
for these tests.

Signed-off-by: Sage Weil <sage@redhat.com>
7 years agomon: add mon_warn_on_pool_no_app option
Sage Weil [Sat, 5 Aug 2017 02:24:07 +0000 (22:24 -0400)]
mon: add mon_warn_on_pool_no_app option

Signed-off-by: Sage Weil <sage@redhat.com>
7 years agoqa/suites/upgrade/jewel-x/parallel: whitelist OSD_DOWN
Sage Weil [Fri, 4 Aug 2017 18:39:53 +0000 (14:39 -0400)]
qa/suites/upgrade/jewel-x/parallel: whitelist OSD_DOWN

We restart OSDs during the upgrade.

Signed-off-by: Sage Weil <sage@redhat.com>
7 years agomon/MgrMonitor: use static mgr commands if none defined yet
Sage Weil [Fri, 4 Aug 2017 18:37:41 +0000 (14:37 -0400)]
mon/MgrMonitor: use static mgr commands if none defined yet

During upgrade, we may get a command (like 'pg dump') before we
pass our first post-upgrade paxos round embedding the static mgr
commands.  Until then, use the static mgr commands (that's what
we're proposing anyway).

Signed-off-by: Sage Weil <sage@redhat.com>
7 years agoMerge pull request #16780 from amitkumar50/cov-1414521
Jos Collin [Sun, 6 Aug 2017 09:05:34 +0000 (09:05 +0000)]
Merge pull request #16780 from amitkumar50/cov-1414521

libradosstriper: Initialize member variable m_writeRc in WriteCompletionData

Reviewed-by: Jos Collin <jcollin@redhat.com>
7 years agoMerge pull request #16781 from amitkumar50/cov-1414516
Jos Collin [Sun, 6 Aug 2017 06:47:59 +0000 (06:47 +0000)]
Merge pull request #16781 from amitkumar50/cov-1414516

msg: Initialize member variables in Infiniband

Reviewed-by: Jos Collin <jcollin@redhat.com>
7 years agomsg: Initilization of uninitialized class Infiniband members 16781/head
amitkuma [Thu, 3 Aug 2017 16:09:36 +0000 (21:39 +0530)]
msg: Initilization of uninitialized class Infiniband members

Fixes coverity issues:

1414516 Uninitialized pointer field
2. uninit_member: Non-static class member max_send_wr is not initialized in this constructor nor in any functions that it calls.
4. uninit_member: Non-static class member max_recv_wr is not initialized in this constructor nor in any functions that it calls.
6. uninit_member: Non-static class member max_sge is not initialized in this constructor nor in any functions that it calls.
8. uninit_member: Non-static class member ib_physical_port is not initialized in this constructor nor in any functions that it calls.
10. uninit_member: Non-static class member memory_manager is not initialized in this constructor nor in any functions that it calls.
12. uninit_member: Non-static class member srq is not initialized in this constructor nor in any functions that it calls.
14. uninit_member: Non-static class member device is not initialized in this constructor nor in any functions that it calls.
CID 1414516 (#1 of 1): Uninitialized pointer field (UNINIT_CTOR)
16. uninit_member: Non-static class member pd is not initialized in this constructor nor in any functions that it calls.

Signed-off-by: Amit Kumar amitkuma@redhat.com
7 years agoMerge pull request #16662 from ivancich/wip-dmclock-options-doc
Sage Weil [Sat, 5 Aug 2017 18:36:37 +0000 (13:36 -0500)]
Merge pull request #16662 from ivancich/wip-dmclock-options-doc

doc: op queue and mclock related options

Reviewed-by: Kefu Chai <kchai@redhat.com>
Reviewed-by: Sage Weil <sage@redhat.com>
7 years agoMerge pull request #16824 from liewegas/wip-more-scrub-time
Sage Weil [Sat, 5 Aug 2017 18:35:55 +0000 (13:35 -0500)]
Merge pull request #16824 from liewegas/wip-more-scrub-time

qa/tasks/ceph: wait longer for scrub

7 years agoMerge pull request #16841 from liewegas/wip-debug-ceph
Sage Weil [Sat, 5 Aug 2017 18:35:13 +0000 (13:35 -0500)]
Merge pull request #16841 from liewegas/wip-debug-ceph

qa/tasks/ceph: debug osd setup

7 years agoMerge pull request #16633 from dillaman/wip-bootstrap-rbd-mirror
Sage Weil [Sat, 5 Aug 2017 18:27:32 +0000 (13:27 -0500)]
Merge pull request #16633 from dillaman/wip-bootstrap-rbd-mirror

mon: added bootstrap-rbd auth profile

Reviewed-by: Sage Weil <sage@redhat.com>
7 years agoMerge pull request #16704 from dillaman/wip-20644
Sage Weil [Sat, 5 Aug 2017 18:24:09 +0000 (13:24 -0500)]
Merge pull request #16704 from dillaman/wip-20644

rbd-mirror: assertion failure during pool replayer shut down

Reviewed-by: Venky Shankar <vshankar@redhat.com>
7 years agoqa/tasks/ceph: debug osd setup 16841/head
Sage Weil [Sat, 5 Aug 2017 17:53:26 +0000 (13:53 -0400)]
qa/tasks/ceph: debug osd setup

I've seen a couple rbd runs that seem to skip the next block :/

Signed-off-by: Sage Weil <sage@redhat.com>
7 years agoceph.spec.in: ceph_volume but not ceph_volume_client in ceph-base 16832/head
Sage Weil [Fri, 4 Aug 2017 19:47:10 +0000 (15:47 -0400)]
ceph.spec.in: ceph_volume but not ceph_volume_client in ceph-base

Fixes: http://tracker.ceph.com/issues/20915
Signed-off-by: Sage Weil <sage@redhat.com>
7 years agoMerge pull request #16784 from amitkumar50/cov-1244200
Jos Collin [Sat, 5 Aug 2017 15:41:15 +0000 (15:41 +0000)]
Merge pull request #16784 from amitkumar50/cov-1244200

test: Initialize pointer msg in MonClientHelper

Reviewed-by: Jos Collin <jcollin@redhat.com>
7 years agoMerge pull request #16839 from amitkumar50/cov-messages-3
Jos Collin [Sat, 5 Aug 2017 15:35:22 +0000 (15:35 +0000)]
Merge pull request #16839 from amitkumar50/cov-messages-3

messages: Initialize member variables

Reviewed-by: Jos Collin <jcollin@redhat.com>
7 years agomessages: Initializing variables various classes 16839/head
amitkuma [Sat, 5 Aug 2017 19:01:28 +0000 (00:31 +0530)]
messages: Initializing variables various classes

Fixes the coverity Issues:

** 1019615 Uninitialized scalar field
2. uninit_member: Non-static class member service_type is not initialized
 in this constructor nor in any functions that it calls.
CID 1019615 (#1 of 1): Uninitialized scalar field (UNINIT_CTOR)
4. uninit_member: Non-static class member service_op is not initialized
 in this constructor nor in any functions that it calls.

** 1019618 Uninitialized scalar field
2. uninit_member: Non-static class member op is not initialized in this
 constructor nor in any functions that it calls.
4. uninit_member: Non-static class member cookie is not initialized in
 this constructor nor in any functions that it calls.
CID 1019618 (#1 of 1): Uninitialized scalar field (UNINIT_CTOR)
6. uninit_member: Non-static class member last_committed is not initialized
 in this constructor nor in any functions that it calls.

** 1019620 Uninitialized scalar field
2. uninit_member: Non-static class member epoch is not initialized in this
 constructor nor in any functions that it calls.
CID 1019620 (#1 of 1): Uninitialized scalar field (UNINIT_CTOR)
4. uninit_member: Non-static class member request_ack is not initialized
in this constructor nor in any functions that it calls.

** 1019621 Uninitialized scalar field
2. uninit_member: Non-static class member epoch is not initialized in this
constructor nor in any functions that it calls.
CID 1019621 (#1 of 1): Uninitialized scalar field (UNINIT_CTOR)
4. uninit_member: Non-static class member round is not initialized in this
constructor nor in any functions that it calls

Signed-off-by: Amit Kumar amitkuma@redhat.com
7 years agoMerge pull request #16819 from amitkumar50/cov-messages-1
Jos Collin [Sat, 5 Aug 2017 15:15:45 +0000 (15:15 +0000)]
Merge pull request #16819 from amitkumar50/cov-messages-1

messages: Initialize member variables

Reviewed-by: Jos Collin <jcollin@redhat.com>
Reviewed-by: Greg Farnum <gfarnum@redhat.com>
7 years agomessages: Initializing member variables of module messages 16819/head
amitkuma [Fri, 4 Aug 2017 17:11:18 +0000 (22:41 +0530)]
messages: Initializing member variables of module messages

Fixes coverity Issue:

CID 717307 (#1 of 1): Uninitialized scalar field (UNINIT_CTOR)
2. uninit_member: Non-static class member epoch is not initialized in this constructor nor in any functions that it calls.
2. uninit_member: Non-static class member epoch is not initialized in this constructor nor in any functions that it calls.
CID 717308 (#1 of 1): Uninitialized scalar field (UNINIT_CTOR)
4. uninit_member: Non-static class member query_epoch is not initialized in this constructor nor in any functions that it calls.
CID 717310 (#1 of 1): Uninitialized scalar field (UNINIT_CTOR)
2. uninit_member: Non-static class member epoch is not initialized in this constructor nor in any functions that it calls.
CID 717311 (#1 of 1): Uninitialized scalar field (UNINIT_CTOR)
2. uninit_member: Non-static class member epoch is not initialized in this constructor nor in any functions that it calls.
CID 717312 (#1 of 1): Uninitialized scalar field (UNINIT_CTOR)
2. uninit_member: Non-static class member epoch is not initialized in this constructor nor in any functions that it calls.
uninit_member: Non-static class member op is not initialized in this constructor nor in any functions that it calls.
uninit_member: Non-static class member map_epoch is not initialized in this constructor nor in any functions that it calls.
CID 717313 (#1 of 1): Uninitialized scalar field (UNINIT_CTOR)
uninit_member: Non-static class member query_epoch is not initialized in this constructor nor in any functions that it calls.
CID 717314 (#1 of 1): Uninitialized scalar field (UNINIT_CTOR)
2. uninit_member: Non-static class member map_epoch is not initialized in this constructor nor in any functions that it calls.
CID 717315 (#1 of 1): Uninitialized scalar field (UNINIT_CTOR)
2. uninit_member: Non-static class member epoch is not initialized in this constructor nor in any functions that it calls.
uninit_member: Non-static class member map_epoch is not initialized in this constructor nor in any functions that it calls.
CID 717316 (#1 of 1): Uninitialized scalar field (UNINIT_CTOR)
uninit_member: Non-static class member op is not initialized in this constructor nor in any functions that it calls.
uninit_member: Non-static class member map_epoch is not initialized in this constructor nor in any functions that it calls.
CID 717317 (#1 of 1): Uninitialized scalar field (UNINIT_CTOR)
uninit_member: Non-static class member min_epoch is not initialized in this constructor nor in any functions that it calls.
uninit_member: Non-static class member repair is not initialized in this constructor nor in any functions that it calls.
CID 717318 (#1 of 1): Uninitialized scalar field (UNINIT_CTOR)
4. uninit_member: Non-static class member deep is not initialized in this constructor nor in any functions that it calls.
uninit_member: Non-static class member map_epoch is not initialized in this constructor nor in any functions that it calls.
uninit_member: Non-static class member acks_wanted is not initialized in this constructor nor in any functions that it calls.
uninit_member: Non-static class member old_exists is not initialized in this constructor nor in any functions that it calls.
uninit_member: Non-static class member old_size is not initialized in this constructor nor in any functions that it calls.
uninit_member: Non-static class member first is not initialized in this constructor nor in any functions that it calls.
CID 717319 (#1 of 1): Uninitialized scalar field (UNINIT_CTOR)
uninit_member: Non-static class member complete is not initialized in this constructor nor in any functions that it calls.
uninit_member: Non-static class member map_epoch is not initialized in this constructor nor in any functions that it calls.
uninit_member: Non-static class member ack_type is not initialized in this constructor nor in any functions that it calls.
CID 717320 (#1 of 1): Uninitialized scalar field (UNINIT_CTOR)
uninit_member: Non-static class member result is not initialized in this constructor nor in any functions that it calls.
CID 717321 (#1 of 1): Uninitialized scalar field (UNINIT_CTOR)
uninit_member: Non-static class member epoch is not initialized in this constructor nor in any functions that it calls
uninit_member: Non-static class member pool is not initialized in this constructor nor in any functions that it calls.
uninit_member: Non-static class member op is not initialized in this constructor nor in any functions that it calls.
uninit_member: Non-static class member auid is not initialized in this constructor nor in any functions that it calls.
CID 717322 (#1 of 1): Uninitialized scalar field (UNINIT_CTOR)
uninit_member: Non-static class member crush_rule is not initialized in this constructor nor in any functions that it calls.
uninit_member: Non-static class member replyCode is not initialized in this constructor nor in any functions that it calls.
CID 717323 (#1 of 1): Uninitialized scalar field (UNINIT_CTOR)
uninit_member: Non-static class member epoch is not initialized in this constructor nor in any functions that it calls.

Signed-off-by: Amit Kumar amitkuma@redhat.com
7 years agoMerge pull request #16785 from amitkumar50/cov-1274326
Jos Collin [Sat, 5 Aug 2017 13:56:07 +0000 (13:56 +0000)]
Merge pull request #16785 from amitkumar50/cov-1274326

test: Initialize pointer variables in TestMemIoCtxImpl

Reviewed-by: Jos Collin <jcollin@redhat.com>
7 years agorados: Initializing uninitialized pointer variables 16785/head
amitkuma [Thu, 3 Aug 2017 17:16:25 +0000 (22:46 +0530)]
rados: Initializing uninitialized pointer variables

Fixes the coverity issue:

1274326 Uninitialized pointer field
2. uninit_member: Non-static class member m_client is not initialized in
this constructor nor in any functions that it calls.
CID 1274326 (#1 of 1): Uninitialized pointer field (UNINIT_CTOR)
4. uninit_member: Non-static class member m_pool is not initialized in
this constructor nor in any functions that it calls.

Signed-off-by: Amit Kumar amitkuma@redhat.com
7 years agomon/OSDMonitor: sanity check osd before performing 'osd purge' 16838/head
xie xingguo [Sat, 5 Aug 2017 11:21:33 +0000 (19:21 +0800)]
mon/OSDMonitor: sanity check osd before performing 'osd purge'

This will pervent OSDMonitor from crashing on purging a very large
non-existent osd id as below:

osd e11 prepare_command_osd_purge purging osd.8
    -1> 2017-08-05 18:59:44.994319 7f6076968700 10 mon.a@0(leader).osd e11 prepare_command_osd_destroy osd.8 does not exist.
     0> 2017-08-05 18:59:45.002309 7f6076968700 -1 /home/xxg/build/ceph-dev/src/osd/OSDMap.h: In function 'int OSDMap::get_state(int) const'
 thread 7f6076968700 time 2017-08-05 18:59:44.994336
/home/xxg/build/ceph-dev/src/osd/OSDMap.h: 690: FAILED assert(o < max_osd)

Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
7 years agocrush: automatically kill dead classes 16837/head
xie xingguo [Sat, 5 Aug 2017 10:25:21 +0000 (18:25 +0800)]
crush: automatically kill dead classes

If a class is no more referenced by any devices or crush rules,
it shall be considered as dead.

This patch makes Ceph automatically recycles those dead classes,
so user does not to explicitly call 'class rm', which is unsafe
and annoying.

Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
7 years agocrush: remove 'class rm' command
xie xingguo [Sat, 5 Aug 2017 08:21:27 +0000 (16:21 +0800)]
crush: remove 'class rm' command

The current version is broken. E.g., it should only remove a class
which is never referenced by any device.

Since we now create new classes automatically, we shall automatically
recycle dead classes too. So this command is definitely unuseful.
(Actually it is weird that we keep 'class rm' without keeping the
 corresponding 'class create' command).

Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
7 years agocrush: do not automatically recycle class for 'rm-device-class'
xie xingguo [Sat, 5 Aug 2017 07:34:58 +0000 (15:34 +0800)]
crush: do not automatically recycle class for 'rm-device-class'

This will prevent the current crush rule from referencing a non-existent
shadow tree and hence avoid a coredump such as below:

 0> 2017-08-05 09:54:19.943349 7f73887d6700 -1 /clove/vm/xxg/rpm/ceph/rpmbuild/BUILD/ceph-12.1.2.1/src/crush/CrushWrapper.cc: In function 'int CrushWrapper::get_rule_weight_osd_map(unsigned
 int, std::map<int, float>*)' thread 7f73887d6700 time 2017-08-05 09:54:19.941291
/clove/vm/xxg/rpm/ceph/rpmbuild/BUILD/ceph-12.1.2.1/src/crush/CrushWrapper.cc: 1631: FAILED assert(b)

 ceph version 12.1.2.1-11-gd0f812a (d0f812a3a757b319c26794f558b57770663ab324) luminous (rc)
 1: (ceph::__ceph_assert_fail(char const*, char const*, int, char const*)+0x110) [0x7f7398b66ea0]
 2: (CrushWrapper::get_rule_weight_osd_map(unsigned int, std::map<int, float, std::less<int>, std::allocator<std::pair<int const, float> > >*)+0x54e) [0x7f7398daac4e]
 3: (PGMap::get_rule_avail(OSDMap const&, int) const+0x68) [0x7f73989a6428]
 4: (PGMap::get_rules_avail(OSDMap const&, std::map<int, long, std::less<int>, std::allocator<std::pair<int const, long> > >*) const+0x35c) [0x7f73989b748c]
 5: (PGMap::encode_digest(OSDMap const&, ceph::buffer::list&, unsigned long) const+0x16) [0x7f73989b7506]
 6: (DaemonServer::send_report()+0x2a4) [0x7f73989f5474]
 7: (DaemonServer::maybe_ready(int)+0x2f9) [0x7f73989f6129]
 8: (DaemonServer::ms_dispatch(Message*)+0xce) [0x7f73989ff68e]
 9: (DispatchQueue::entry()+0x792) [0x7f7398dd2a22]
 10: (DispatchQueue::DispatchThread::entry()+0xd) [0x7f7398c1429d]
 11: (()+0x7df3) [0x7f739640cdf3]
 12: (clone()+0x6d) [0x7f73954f23ed]

Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
7 years agoMerge pull request #16791 from amitkumar50/cov-717285
Jos Collin [Sat, 5 Aug 2017 10:32:13 +0000 (10:32 +0000)]
Merge pull request #16791 from amitkumar50/cov-717285

messages: Initializing member variable in MMDSCacheRejoin

Reviewed-by: Jos Collin <jcollin@redhat.com>
7 years agoMerge pull request #16783 from amitkumar50/cov-1414515
Jos Collin [Sat, 5 Aug 2017 09:51:55 +0000 (09:51 +0000)]
Merge pull request #16783 from amitkumar50/cov-1414515

msg: Remove unused variable perf_counter in RDMAStack

Reviewed-by: Jos Collin <jcollin@redhat.com>
7 years agoMerge pull request #16810 from amitkumar50/cov-717293
Jos Collin [Sat, 5 Aug 2017 08:40:31 +0000 (08:40 +0000)]
Merge pull request #16810 from amitkumar50/cov-717293

messages: Initialize members in MMDSTableRequest

Reviewed-by: Jos Collin <jcollin@redhat.com>
7 years agoMerge PR #16833 into master
Patrick Donnelly [Sat, 5 Aug 2017 03:26:09 +0000 (20:26 -0700)]
Merge PR #16833 into master

* refs/remotes/upstream/pull/16833/head:
qa: whitelist expected MDS_CLIENT_OLDEST_TID warn
qa: ignore insufficient standby during failover
qa: fix read-only whitelist
mds: MDS_DAMAGED to MDS_DAMAGE
doc: remove duplicate CephFS health check doc

7 years agoMerge PR #16825 into master
Patrick Donnelly [Sat, 5 Aug 2017 03:26:07 +0000 (20:26 -0700)]
Merge PR #16825 into master

* refs/remotes/upstream/pull/16825/head:
Changing 'int const' to 'const int'

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
7 years agoMerge PR #16751 into master
Patrick Donnelly [Sat, 5 Aug 2017 03:26:05 +0000 (20:26 -0700)]
Merge PR #16751 into master

* refs/remotes/upstream/pull/16751/head:
mds: Added NULL check before dereference

Reviewed-by: Jos Collin <jcollin@redhat.com>
Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
7 years agoMerge PR #16750 into master
Patrick Donnelly [Sat, 5 Aug 2017 03:26:03 +0000 (20:26 -0700)]
Merge PR #16750 into master

* refs/remotes/upstream/pull/16750/head:
client: Added NULL check before dereference

Reviewed-by: John Spray <john.spray@redhat.com>
Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
7 years agoMerge PR #16725 into master
Patrick Donnelly [Sat, 5 Aug 2017 03:26:01 +0000 (20:26 -0700)]
Merge PR #16725 into master

* refs/remotes/upstream/pull/16725/head:
mds: Removing unused variables from StrayManager

Reviewed-by: Jos Collin <jcollin@redhat.com>
Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
7 years agoMerge PR #16671 into master
Patrick Donnelly [Sat, 5 Aug 2017 03:25:59 +0000 (20:25 -0700)]
Merge PR #16671 into master

* refs/remotes/upstream/pull/16671/head:
Client: Add function "get_default_data_pool_name" for java client

Reviewed-by: Zheng Yan <zyan@redhat.com>
7 years agoMerge PR #16570 into master
Patrick Donnelly [Sat, 5 Aug 2017 03:25:56 +0000 (20:25 -0700)]
Merge PR #16570 into master

* refs/remotes/upstream/pull/16570/head:
mds: clean up obsolete declarations/headers

Reviewed-by: Zheng Yan <zyan@redhat.com>
7 years agoqa: whitelist expected MDS_CLIENT_OLDEST_TID warn 16833/head
Patrick Donnelly [Sat, 5 Aug 2017 03:21:13 +0000 (20:21 -0700)]
qa: whitelist expected MDS_CLIENT_OLDEST_TID warn

Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
7 years agoqa: ignore insufficient standby during failover
Patrick Donnelly [Fri, 4 Aug 2017 20:08:07 +0000 (13:08 -0700)]
qa: ignore insufficient standby during failover

Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
7 years agoqa: fix read-only whitelist
Patrick Donnelly [Sat, 5 Aug 2017 03:14:48 +0000 (20:14 -0700)]
qa: fix read-only whitelist

Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
7 years agoChanging 'int const' to 'const int' 16825/head
amitkuma [Fri, 4 Aug 2017 21:58:21 +0000 (03:28 +0530)]
Changing 'int const' to 'const int'

As per coding Guidelines 'const int' is recommended over 'int const'
https://github.com/isocpp/CppCoreGuidelines/blob/master/CppCoreGuidelines.md

NL.26: Use conventional const notation
Example:
const int x = 7;    // OK
int const y = 9;    // bad

Note
We are well aware that you could claim the "bad" examples more logical than the ones marked "OK", but they also confuse more people, especially novices relying on teaching material using the far more common, conventional OK style.
As ever, remember that the aim of these naming and layout rules is consistency and that aesthetics vary immensely.

Enforcement
Flag const used as a suffix for a type.

Signed-off-by: Amit Kumar amitkuma@redhat.com
7 years agoMerge PR #16761 into master
Patrick Donnelly [Fri, 4 Aug 2017 20:33:51 +0000 (13:33 -0700)]
Merge PR #16761 into master

* refs/remotes/upstream/pull/16761/head:
doc/cephfs: Document ceph auth fs
mon/AuthMonitor: Add special syntax for cephfs

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
7 years agoMerge pull request #16811 from amitkumar50/cov-717296
Gregory Farnum [Fri, 4 Aug 2017 20:31:06 +0000 (13:31 -0700)]
Merge pull request #16811 from amitkumar50/cov-717296

messages: Initializing uninitialized members MMonGetVersion

Reviewed-by: Greg Farnum <gfarnum@redhat.com>
7 years agomds: MDS_DAMAGED to MDS_DAMAGE
Patrick Donnelly [Fri, 4 Aug 2017 20:00:47 +0000 (13:00 -0700)]
mds: MDS_DAMAGED to MDS_DAMAGE

We had both and MDS_DAMAGE looks to be the right/intended one.

Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
7 years agodebian: include ceph-volume python
Sage Weil [Fri, 4 Aug 2017 19:47:42 +0000 (15:47 -0400)]
debian: include ceph-volume python

Be careful not to grab ceph_volume_client.py

Signed-off-by: Sage Weil <sage@redhat.com>
7 years agodoc: remove duplicate CephFS health check doc
Patrick Donnelly [Fri, 4 Aug 2017 19:27:54 +0000 (12:27 -0700)]
doc: remove duplicate CephFS health check doc

These are documented in doc/cephfs/health-messages.rst.

Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
7 years agoMerge branch 'wip-qa-rbd-health' of git://github.com/dillaman/ceph
Sage Weil [Fri, 4 Aug 2017 19:07:22 +0000 (15:07 -0400)]
Merge branch 'wip-qa-rbd-health' of git://github.com/dillaman/ceph

# Conflicts:
# qa/tasks/ceph.py

7 years agoMerge pull request #16815 from amitkumar50/cov-717299
Gregory Farnum [Fri, 4 Aug 2017 19:04:38 +0000 (12:04 -0700)]
Merge pull request #16815 from amitkumar50/cov-717299

messages: Initializing uninitialized members MMonProbe

Reviewed-by: Greg Farnum <gfarnum@redhat.com>
7 years agoMerge pull request #16817 from amitkumar50/cov-messages-branch
Gregory Farnum [Fri, 4 Aug 2017 19:04:16 +0000 (12:04 -0700)]
Merge pull request #16817 from amitkumar50/cov-messages-branch

messages: Initializing uninitialized members module messages

Reviewed-by: Greg Farnum <gfarnum@redhat.com>
7 years agoMerge pull request #16816 from amitkumar50/cov-717300
Gregory Farnum [Fri, 4 Aug 2017 19:01:49 +0000 (12:01 -0700)]
Merge pull request #16816 from amitkumar50/cov-717300

messages: Initializing uninitialized members MOSDAlive

Reviewed-by: Greg Farnum <gfarnum@redhat.com>
7 years agoMerge pull request #16814 from amitkumar50/cov-717298
Gregory Farnum [Fri, 4 Aug 2017 19:00:57 +0000 (12:00 -0700)]
Merge pull request #16814 from amitkumar50/cov-717298

messages: Initializing uninitialized members MMonPaxos

Reviewed-by: Greg Farnum <gfarnum@redhat.com>
7 years agoMerge pull request #16813 from amitkumar50/cov-717297
Gregory Farnum [Fri, 4 Aug 2017 19:00:13 +0000 (12:00 -0700)]
Merge pull request #16813 from amitkumar50/cov-717297

messages: Initializing uninitialized members MMonGetVersionReply

Reviewed-by: Greg Farnum <gfarnum@redhat.com>
7 years agoMerge pull request #16803 from dmick/master
Gregory Farnum [Fri, 4 Aug 2017 18:58:41 +0000 (11:58 -0700)]
Merge pull request #16803 from dmick/master

common/buffer: off-by-one error in max iov length blocking

Reviewed-by: Greg Farnum <gfarnum@redhat.com>
7 years agoMerge pull request #16675 from Jeegn-Chen/skip-err-entries
Josh Durgin [Fri, 4 Aug 2017 18:47:38 +0000 (11:47 -0700)]
Merge pull request #16675 from Jeegn-Chen/skip-err-entries

osd/PGLog: skip ERROR entires in _merge_object_divergent_entries

Reviewed-by: Josh Durgin <jdurgin@redhat.com>
7 years agoMerge pull request #16744 from joscollin/wip-osd-warning
Josh Durgin [Fri, 4 Aug 2017 18:45:42 +0000 (11:45 -0700)]
Merge pull request #16744 from joscollin/wip-osd-warning

osd: silence warning from -Wint-in-bool-context

Reviewed-by: Josh Durgin <jdurgin@redhat.com>
Reviewed-by: Eric Ivancich <ivancich@redhat.com>
Reviewed-by: Kefu Chai <kchai@redhat.com>
7 years agoMerge pull request #16820 from liewegas/wip-more-whitelist
Sage Weil [Fri, 4 Aug 2017 18:44:08 +0000 (13:44 -0500)]
Merge pull request #16820 from liewegas/wip-more-whitelist

qa/suites/rados: a bit more whitelisting

Reviewed-by: Josh Durgin <jdurgin@redhat.com>
7 years agodoc: op queue and mclock related options 16662/head
J. Eric Ivancich [Wed, 2 Aug 2017 21:28:11 +0000 (17:28 -0400)]
doc: op queue and mclock related options

Additionally makes sure when unexpected values are provided for config
options osd_op_queue or osd_op_queue_cut_off that the default values
are assigned.

Signed-off-by: J. Eric Ivancich <ivancich@redhat.com>
7 years agoMerge pull request #16614 from neha-ojha/recovery_sleep_hybrid
Sage Weil [Fri, 4 Aug 2017 18:23:38 +0000 (13:23 -0500)]
Merge pull request #16614 from neha-ojha/recovery_sleep_hybrid

osd: Check whether journal is rotational or not

Reviewed-by: Sage Weil <sage@redhat.com>
7 years agoMerge pull request #16809 from jdurgin/wip-20904
Sage Weil [Fri, 4 Aug 2017 18:20:32 +0000 (13:20 -0500)]
Merge pull request #16809 from jdurgin/wip-20904

osd/PG: fix lost unfound + delete when there are no missing objects

Reviewed-by: Sage Weil <sage@redhat.com>
7 years agoMerge pull request #16812 from xiexingguo/wip-class-startup
Sage Weil [Fri, 4 Aug 2017 18:13:23 +0000 (13:13 -0500)]
Merge pull request #16812 from xiexingguo/wip-class-startup

osd/OSD: tolerate any 'set-device-class' error on OSD startup

Reviewed-by: Sage Weil <sage@redhat.com>
7 years agoosd: use recovery sleep hdd for ssd store and hdd journal case 16614/head
Neha Ojha [Fri, 4 Aug 2017 18:07:19 +0000 (11:07 -0700)]
osd: use recovery sleep hdd for ssd store and hdd journal case

Signed-off-by: Neha Ojha <nojha@redhat.com>
7 years agoMerge pull request #16827 from liewegas/wip-more-more-more
Sage Weil [Fri, 4 Aug 2017 17:39:47 +0000 (12:39 -0500)]
Merge pull request #16827 from liewegas/wip-more-more-more

qa/suites/rados/basic/tasks/rados_python: POOL_APP_NOT_ENABLED

7 years agoqa/suites/rados/basic/tasks/rados_python: POOL_APP_NOT_ENABLED 16827/head
Sage Weil [Fri, 4 Aug 2017 17:39:13 +0000 (13:39 -0400)]
qa/suites/rados/basic/tasks/rados_python: POOL_APP_NOT_ENABLED

Signed-off-by: Sage Weil <sage@redhat.com>
7 years agomessages: Initializing uninitialized members module messages 16817/head
amitkuma [Fri, 4 Aug 2017 16:46:04 +0000 (22:16 +0530)]
messages: Initializing uninitialized members module messages

Fixes coverity Issue:

** 2. uninit_member: Non-static class member flags is not initialized in this constructor nor in any functions that it calls.
4. uninit_member: Non-static class member epoch is not initialized in this constructor nor in any functions that it calls.
CID 717301 (#1 of 1): Uninitialized scalar field (UNINIT_CTOR)
6. uninit_member: Non-static class member failed_for is not initialized in this constructor nor in any functions that it calls.

2. uninit_member: Non-static class member oldest_map is not initialized in this constructor nor in any functions that it calls.
CID 717302 (#1 of 1): Uninitialized scalar field (UNINIT_CTOR)
4. uninit_member: Non-static class member newest_map is not initialized in this constructor nor in any functions that it calls.

2. uninit_member: Non-static class member client_inc is not initialized in this constructor nor in any functions that it calls.
4. uninit_member: Non-static class member osdmap_epoch is not initialized in this constructor nor in any functions that it calls.
6. uninit_member: Non-static class member flags is not initialized in this constructor nor in any functions that it calls.
8. uninit_member: Non-static class member retry_attempt is not initialized in this constructor nor in any functions that it calls.
CID 717303 (#1 of 1): Uninitialized scalar field (UNINIT_CTOR)
10. uninit_member: Non-static class member features is not initialized in this constructor nor in any functions that it calls.

2. uninit_member: Non-static class member flags is not initialized in this constructor nor in any functions that it calls.
4. uninit_member: Non-static class member user_version is not initialized in this constructor nor in any functions that it calls.
6. uninit_member: Non-static class member osdmap_epoch is not initialized in this constructor nor in any functions that it calls.
CID 717304 (#1 of 1): Uninitialized scalar field (UNINIT_CTOR)
8. uninit_member: Non-static class member retry_attempt is not initialized in this constructor nor in any functions that it calls.

2. uninit_member: Non-static class member op is not initialized in this constructor nor in any functions that it calls.
4. uninit_member: Non-static class member map_epoch is not initialized in this constructor nor in any functions that it calls.
CID 717305 (#1 of 1): Uninitialized scalar field (UNINIT_CTOR)
6. uninit_member: Non-static class member query_epoch is not initialized in this constructor nor in any functions that it calls.

CID 717306 (#1 of 1): Uninitialized scalar field (UNINIT_CTOR)
2. uninit_member: Non-static class member epoch is not initialized in this constructor nor in any functions that it calls.

Signed-off-by: Amit Kumar amitkuma@redhat.com
7 years agoMerge pull request #16724 from amitkumar50/cov-1403249
Jason Dillaman [Fri, 4 Aug 2017 16:34:29 +0000 (12:34 -0400)]
Merge pull request #16724 from amitkumar50/cov-1403249

librbd: Uninitialized variable used handle_refresh()

Reviewed-by: Jason Dillaman <dillaman@redhat.com>
7 years agoMerge pull request #16742 from scienceluo/wip-luo-coverity-fix-branch
Jason Dillaman [Fri, 4 Aug 2017 16:32:34 +0000 (12:32 -0400)]
Merge pull request #16742 from scienceluo/wip-luo-coverity-fix-branch

test/librbd/fsx: Add break in case OP_WRITESAME and OP_COMPARE_AND_WRITE

Reviewed-by: Jason Dillaman <dillaman@redhat.com>
7 years agoMerge pull request #16729 from amitkumar50/cov-1402141
Jason Dillaman [Fri, 4 Aug 2017 16:30:44 +0000 (12:30 -0400)]
Merge pull request #16729 from amitkumar50/cov-1402141

rbd: Drop unused member variable reopen in C_OpenComplete

Reviewed-by: Jason Dillaman <dillaman@redhat.com>
7 years agoqa/tasks/ceph: wait longer for scrub 16824/head
Sage Weil [Fri, 4 Aug 2017 16:06:27 +0000 (12:06 -0400)]
qa/tasks/ceph: wait longer for scrub

Signed-off-by: Sage Weil <sage@redhat.com>
7 years agoMerge pull request #16769 from liewegas/wip-20295-b
Sage Weil [Fri, 4 Aug 2017 16:04:38 +0000 (11:04 -0500)]
Merge pull request #16769 from liewegas/wip-20295-b

os/bluestore: allow multiple DeferredBatches in flight at once

Reviewed-by: xie xingguo <xie.xingguo@zte.com.cn>
7 years agoMerge pull request #16787 from liewegas/wip-bluestore-docs
Abhishek L [Fri, 4 Aug 2017 15:37:47 +0000 (17:37 +0200)]
Merge pull request #16787 from liewegas/wip-bluestore-docs

doc/release-notes: fix bluestore links

Reviewed-by: Abhishek Lekshmanan <abhishek@suse.com>
7 years agodoc/cephfs: Document ceph auth fs 16761/head
Douglas Fuller [Wed, 2 Aug 2017 15:39:51 +0000 (11:39 -0400)]
doc/cephfs: Document ceph auth fs

Change the CephFS auth caps documentation to reflect the new ceph auth
fs command.

Signed-off-by: Douglas Fuller <dfuller@redhat.com>
7 years agoMerge pull request #16632 from ceph/wip-volume
Sage Weil [Fri, 4 Aug 2017 14:51:27 +0000 (09:51 -0500)]
Merge pull request #16632 from ceph/wip-volume

ceph-volume: initial take on ceph-volume CLI tool

Reviewed-by: Sage Weil <sage@redhat.com>
Reviewed-by: Andrew Schoen <aschoen@redhat.com>
7 years agomessages: Initializing uninitialized members MOSDAlive 16816/head
amitkuma [Fri, 4 Aug 2017 14:50:08 +0000 (20:20 +0530)]
messages: Initializing uninitialized members MOSDAlive

Fixes coverity Issue:

** 717300 Uninitialized scalar field
CID 717300 (#1 of 1): Uninitialized scalar field (UNINIT_CTOR)
2. uninit_member: Non-static class member want is not initialized in this constructor nor in any functions that it calls.

Signed-off-by: Amit Kumar amitkuma@redhat.com
7 years agomessages: Initializing uninitialized members MMonProbe 16815/head
amitkuma [Fri, 4 Aug 2017 14:44:55 +0000 (20:14 +0530)]
messages: Initializing uninitialized members MMonProbe

Fixes coverity Issue:

** 717299 Uninitialized scalar field
2. uninit_member: Non-static class member op is not initialized in this constructor nor in any functions that it calls.
   4. uninit_member: Non-static class member paxos_first_version is not initialized in this constructor nor in any functions that it calls.
    6. uninit_member: Non-static class member paxos_last_version is not initialized in this constructor nor in any functions that it calls.
    8. uninit_member: Non-static class member has_ever_joined is not initialized in this constructor nor in any functions that it calls.

CID 717299 (#1 of 1): Uninitialized scalar field (UNINIT_CTOR)
10. uninit_member: Non-static class member required_features is not initialized in this constructor nor in any functions that it calls

Signed-off-by: Amit Kumar amitkuma@redhat.com
7 years agomessages: Initializing uninitialized members MMonPaxos 16814/head
amitkuma [Fri, 4 Aug 2017 14:37:30 +0000 (20:07 +0530)]
messages: Initializing uninitialized members MMonPaxos

Fixes coverity Issue:

2. uninit_member: Non-static class member epoch is not initialized in this constructor nor in any functions that it calls.
   4. uninit_member: Non-static class member op is not initialized in this constructor nor in any functions that it calls.
    6. uninit_member: Non-static class member first_committed is not initialized in this constructor nor in any functions that it calls.
    8. uninit_member: Non-static class member last_committed is not initialized in this constructor nor in any functions that it calls.
    10. uninit_member: Non-static class member pn_from is not initialized in this constructor nor in any functions that it calls.
    12. uninit_member: Non-static class member pn is not initialized in this constructor nor in any functions that it calls.
    14. uninit_member: Non-static class member uncommitted_pn is not initialized in this constructor nor in any functions that it calls.

CID 717298 (#1 of 1): Uninitialized scalar field (UNINIT_CTOR)
16. uninit_member: Non-static class member latest_version is not initialized in this constructor nor in any functions that it calls.

Signed-off-by: Amit Kumar amitkuma@redhat.com
7 years agoceph-volume lvm trigger should check for root 16632/head
Alfredo Deza [Fri, 4 Aug 2017 14:12:11 +0000 (10:12 -0400)]
ceph-volume lvm trigger should check for root

Signed-off-by: Alfredo Deza <adeza@redhat.com>
7 years agoceph-volume lvm prepare should check for root
Alfredo Deza [Fri, 4 Aug 2017 14:11:54 +0000 (10:11 -0400)]
ceph-volume lvm prepare should check for root

Signed-off-by: Alfredo Deza <adeza@redhat.com>
7 years agoceph-volume lvm create should check for root
Alfredo Deza [Fri, 4 Aug 2017 14:11:37 +0000 (10:11 -0400)]
ceph-volume lvm create should check for root

Signed-off-by: Alfredo Deza <adeza@redhat.com>
7 years agoceph-volume lvm activate should check for root
Alfredo Deza [Fri, 4 Aug 2017 14:11:17 +0000 (10:11 -0400)]
ceph-volume lvm activate should check for root

Signed-off-by: Alfredo Deza <adeza@redhat.com>
7 years agoceph-volume: decorators should call the wrapped func
Alfredo Deza [Fri, 4 Aug 2017 14:04:04 +0000 (10:04 -0400)]
ceph-volume: decorators should call the wrapped func

Signed-off-by: Alfredo Deza <adeza@redhat.com>
7 years agoceph-volume: systemd script should handle type-data format as argument
Alfredo Deza [Wed, 2 Aug 2017 19:26:25 +0000 (15:26 -0400)]
ceph-volume: systemd script should handle type-data format as argument

Signed-off-by: Alfredo Deza <adeza@redhat.com>
7 years agoceph-volume: lvm add the actual trigger subcommand handler
Alfredo Deza [Wed, 2 Aug 2017 19:25:35 +0000 (15:25 -0400)]
ceph-volume: lvm add the actual trigger subcommand handler

Signed-off-by: Alfredo Deza <adeza@redhat.com>
7 years agoceph-volume: lvm: add a trigger sub-command to parent parser
Alfredo Deza [Wed, 2 Aug 2017 18:42:27 +0000 (14:42 -0400)]
ceph-volume: lvm: add a trigger sub-command to parent parser

Signed-off-by: Alfredo Deza <adeza@redhat.com>
7 years agoceph-volume: tests update argument parsing for systemd
Alfredo Deza [Wed, 2 Aug 2017 17:06:32 +0000 (13:06 -0400)]
ceph-volume: tests update argument parsing for systemd

Signed-off-by: Alfredo Deza <adeza@redhat.com>
7 years agoceph-volume: tests add an lvm trigger for the systemd argument parsing
Alfredo Deza [Wed, 2 Aug 2017 17:06:04 +0000 (13:06 -0400)]
ceph-volume: tests add an lvm trigger for the systemd argument parsing

Signed-off-by: Alfredo Deza <adeza@redhat.com>
7 years agoceph-volume remove activate utilities, not needed with 'osd new'
Alfredo Deza [Thu, 27 Jul 2017 17:55:04 +0000 (13:55 -0400)]
ceph-volume remove activate utilities, not needed with 'osd new'

Signed-off-by: Alfredo Deza <adeza@redhat.com>
7 years agoceph-volume lvm.activate remove unused import for activate utils
Alfredo Deza [Thu, 27 Jul 2017 17:53:03 +0000 (13:53 -0400)]
ceph-volume lvm.activate remove unused import for activate utils

Signed-off-by: Alfredo Deza <adeza@redhat.com>
7 years agoceph-volume: systemd should retry several times to activate a device
Alfredo Deza [Thu, 27 Jul 2017 11:46:18 +0000 (07:46 -0400)]
ceph-volume: systemd should retry several times to activate a device

Allows environment variables to tweak the retries and intervals,
defaulting to 30 tries at 5 second intervals.

Signed-off-by: Alfredo Deza <adeza@redhat.com>
7 years agoceph-volume: devices chown the journal when activating
Alfredo Deza [Wed, 26 Jul 2017 20:33:08 +0000 (16:33 -0400)]
ceph-volume: devices chown the journal when activating

Signed-off-by: Alfredo Deza <adeza@redhat.com>
7 years agoceph-volume: systemd script should pass pre-configured log args
Alfredo Deza [Wed, 26 Jul 2017 18:51:09 +0000 (14:51 -0400)]
ceph-volume: systemd script should pass pre-configured log args

Signed-off-by: Alfredo Deza <adeza@redhat.com>
7 years agoceph-volume: prevent missing conf values for logging config
Alfredo Deza [Wed, 26 Jul 2017 18:50:27 +0000 (14:50 -0400)]
ceph-volume: prevent missing conf values for logging config

Signed-off-by: Alfredo Deza <adeza@redhat.com>
7 years agoceph-volume: systemd: add helpers for the ceph-volume systemd unit
Alfredo Deza [Wed, 26 Jul 2017 15:34:39 +0000 (11:34 -0400)]
ceph-volume: systemd: add helpers for the ceph-volume systemd unit

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