]> git.apps.os.sepia.ceph.com Git - ceph.git/log
ceph.git
9 years agotest_msgr: add server_msgr cleanup
Haomai Wang [Tue, 5 Jul 2016 02:52:27 +0000 (10:52 +0800)]
test_msgr: add server_msgr cleanup

Signed-off-by: Haomai Wang <haomai@xsky.com>
9 years agomsg/async/AsyncConnection: add REPLACING write status to indicate replacing
Haomai Wang [Mon, 4 Jul 2016 04:06:34 +0000 (12:06 +0800)]
msg/async/AsyncConnection: add REPLACING write status to indicate replacing

When replacing, we don't expect any AsyncConnection dispatch new event which
will cause thing chaos

Signed-off-by: Haomai Wang <haomai@xsky.com>
9 years agomsg/async: hold lock avoid state changing
Haomai Wang [Sun, 3 Jul 2016 15:11:35 +0000 (23:11 +0800)]
msg/async: hold lock avoid state changing

Fixes: http://tracker.ceph.com/issues/16554
Signed-off-by: Haomai Wang <haomai@xsky.com>
9 years agotest_async_driver: accept new api changes
Haomai Wang [Sun, 3 Jul 2016 14:02:25 +0000 (22:02 +0800)]
test_async_driver: accept new api changes

Signed-off-by: Haomai Wang <haomai@xsky.com>
9 years agomsg/async: dont delete time event when exchange eventcenter
Haomai Wang [Sat, 2 Jul 2016 06:04:23 +0000 (14:04 +0800)]
msg/async: dont delete time event when exchange eventcenter

When replacing and someone called mark_down, it will delete_time_event which
isn't allowed. Because we're exchaning EventCenter now!

Signed-off-by: Haomai Wang <haomai@xsky.com>
9 years agomsg/async/AsyncConnection: simplify shutdown socket flow
Haomai Wang [Fri, 1 Jul 2016 15:44:46 +0000 (23:44 +0800)]
msg/async/AsyncConnection: simplify shutdown socket flow

Let cleanup resources things all in shutdown_socket

Signed-off-by: Haomai Wang <haomai@xsky.com>
9 years agomsg/async: add assert to ensure create/delete file event within thread
Haomai Wang [Wed, 29 Jun 2016 06:09:00 +0000 (14:09 +0800)]
msg/async: add assert to ensure create/delete file event within thread

Signed-off-by: Haomai Wang <haomai@xsky.com>
9 years agoEvent: remove file_lock
Haomai Wang [Tue, 8 Mar 2016 03:29:51 +0000 (11:29 +0800)]
Event: remove file_lock

Now all EventCenter will exists within one thread, it will let all file events
api changes without locks.

Signed-off-by: Haomai Wang <haomai@xsky.com>
9 years agomsg/async/AsyncMessenger: make sure all connection cleanup handler finished
Haomai Wang [Thu, 30 Jun 2016 08:43:51 +0000 (16:43 +0800)]
msg/async/AsyncMessenger: make sure all connection cleanup handler finished

Fixes: http://tracker.ceph.com/issues/16552
Signed-off-by: Haomai Wang <haomai@xsky.com>
9 years agomsg/async: remove close socket line
Haomai Wang [Wed, 29 Jun 2016 09:22:20 +0000 (17:22 +0800)]
msg/async: remove close socket line

because if we are in STATE_CLOSED, fd must be -1

Signed-off-by: Haomai Wang <haomai@xsky.com>
9 years agomsg/async: make sure worker started before let msgr ready
Haomai Wang [Wed, 29 Jun 2016 09:14:16 +0000 (17:14 +0800)]
msg/async: make sure worker started before let msgr ready

When we create event thread, it need a little time to enter event loop(like
calling set_owner), if caller is going to call create_file_event before event
thread enter event loop, it will trigger assert.

Signed-off-by: Haomai Wang <haomai@xsky.com>
9 years agomsg/async: make EventCenter notify file event creating when set_owner
Haomai Wang [Wed, 29 Jun 2016 08:54:16 +0000 (16:54 +0800)]
msg/async: make EventCenter notify file event creating when set_owner

EventCenter::init is called by other thread instead of event thread, so we
need to move create_file_event to set_owner which is called by event thread.

Signed-off-by: Haomai Wang <haomai@xsky.com>
9 years agomsg/async/AsyncConnection: swap eventcenter when replacing
Haomai Wang [Wed, 29 Jun 2016 08:26:29 +0000 (16:26 +0800)]
msg/async/AsyncConnection: swap eventcenter when replacing

Previously we only exchange fd when replacing, now we will introduce dpdk
plugin in the near future. It needs all fd used locally which not like
kernel socket shared by all cores.

So we need to add EventCenter swapping to let each socket is associated to
EventCenter.

Signed-off-by: Haomai Wang <haomai@xsky.com>
9 years agomsg/async/Event: fix delete_time_event leak
Haomai Wang [Tue, 12 Jul 2016 15:37:43 +0000 (23:37 +0800)]
msg/async/Event: fix delete_time_event leak

Signed-off-by: Haomai Wang <haomai@xsky.com>
9 years agoAsyncMessenger: make create/delete_file_event within event thread
Haomai Wang [Tue, 8 Mar 2016 05:59:50 +0000 (13:59 +0800)]
AsyncMessenger: make create/delete_file_event within event thread

We are make each AsyncConnection/AsyncMessenger only modify its file event
in event thread. So make sure create/delete_file_event aren't directly called.

Signed-off-by: Haomai Wang <haomai@xsky.com>
9 years agoEvent: no need to delete_file_Event when deconstruct
Haomai Wang [Tue, 8 Mar 2016 07:51:02 +0000 (15:51 +0800)]
Event: no need to delete_file_Event when deconstruct

Since we are going to close all epoll and cleanup resources, no need to delete
notify fd resource. And another reason is "delete_file_event" doesn't expect
other threads to call

Signed-off-by: Haomai Wang <haomai@xsky.com>
9 years agoasync: change set_socket_option caller
Haomai Wang [Sun, 3 Jan 2016 09:54:36 +0000 (17:54 +0800)]
async: change set_socket_option caller

Signed-off-by: Haomai Wang <haomai@xsky.com>
9 years agonet_handler: adjust set_socket_options to avoid read from conf
Haomai Wang [Thu, 31 Dec 2015 15:46:17 +0000 (23:46 +0800)]
net_handler: adjust set_socket_options to avoid read from conf

We don't want net_handler rely on config value, caller may expect to pass
different value to set_socket_options according to different socket type
like heartbeat, client socket or server socket.

Signed-off-by: Haomai Wang <haomai@xsky.com>
9 years agoMerge pull request #10076 from dillaman/wip-rbd-mirror-cmake
Mykola Golub [Tue, 5 Jul 2016 05:28:43 +0000 (08:28 +0300)]
Merge pull request #10076 from dillaman/wip-rbd-mirror-cmake

test: fix rbd-mirror workunit test cases for cmake

Reviewed-by: Mykola Golub <mgolub@mirantis.com>
9 years agoMerge pull request #9731 from badone/wip-doc-ceph-authtool-print-fix
Kefu Chai [Tue, 5 Jul 2016 05:03:46 +0000 (13:03 +0800)]
Merge pull request #9731 from badone/wip-doc-ceph-authtool-print-fix

doc: ceph-authtool man page option is --print-key not --print

Reviewed-by: Kefu Chai <kchai@redhat.com>
9 years agoMerge pull request #10124 from trociny/wip-cmake-tests
Kefu Chai [Tue, 5 Jul 2016 03:26:46 +0000 (11:26 +0800)]
Merge pull request #10124 from trociny/wip-cmake-tests

test: cmake related fixups for rbd tests

Reviewed-by: Jason Dillaman <dillaman@redhat.com>
Reviewed-by: Kefu Chai <kchai@redhat.com>
9 years agotest: fix run-rbd-tests test for cmake 10124/head
Mykola Golub [Sun, 3 Jul 2016 18:43:17 +0000 (21:43 +0300)]
test: fix run-rbd-tests test for cmake

Signed-off-by: Mykola Golub <mgolub@mirantis.com>
9 years agotest: fix run-rbd-valgrind-unit-tests when built with cmake
Mykola Golub [Mon, 4 Jul 2016 07:02:18 +0000 (10:02 +0300)]
test: fix run-rbd-valgrind-unit-tests when built with cmake

Signed-off-by: Mykola Golub <mgolub@mirantis.com>
9 years agotest: ceph_test_librbd does not need rados_test_stub
Mykola Golub [Mon, 4 Jul 2016 06:48:07 +0000 (09:48 +0300)]
test: ceph_test_librbd does not need rados_test_stub

Signed-off-by: Mykola Golub <mgolub@mirantis.com>
9 years agoMerge pull request #10117 from xiexingguo/xxg-wip-bluestore-2016-07-04
Sage Weil [Mon, 4 Jul 2016 14:55:29 +0000 (10:55 -0400)]
Merge pull request #10117 from xiexingguo/xxg-wip-bluestore-2016-07-04

os/bluestore: fix incorrect pool decoding of bnode

Reviewed-by: Sage Weil <sage@redhat.com>
9 years agoMerge pull request #9755 from dx9/wip-ns-name-uncompress
Kefu Chai [Mon, 4 Jul 2016 13:53:20 +0000 (21:53 +0800)]
Merge pull request #9755 from dx9/wip-ns-name-uncompress

common/dns_resolve: use ns_name_uncompress instead of ns_name_ntop

Reviewed-by: Kefu Chai <kchai@redhat.com>
9 years agoMerge pull request #9766 from dx9/wip-autotools-arch
Kefu Chai [Mon, 4 Jul 2016 13:49:41 +0000 (21:49 +0800)]
Merge pull request #9766 from dx9/wip-autotools-arch

configure.ac: Use uname instead of arch.

Reviewed-by: Willem Jan Withagen <wjw@digiware.nl>
9 years agoMerge pull request #9772 from tchaikov/wip-close-on-exec
Kefu Chai [Mon, 4 Jul 2016 13:47:42 +0000 (21:47 +0800)]
Merge pull request #9772 from tchaikov/wip-close-on-exec

msgr: set close on exec flag

Reviewed-by: Haomai Wang <haomai@xsky.com>
9 years agoMerge pull request #9820 from dx9/wip-fix-res-query-detect
Kefu Chai [Mon, 4 Jul 2016 13:46:55 +0000 (21:46 +0800)]
Merge pull request #9820 from dx9/wip-fix-res-query-detect

configure.ac: fix res_query detection

Reviewed-by: Kefu Chai <kchai@redhat.com>
9 years agoMerge pull request #9845 from dx9/wip-config-basename
Kefu Chai [Mon, 4 Jul 2016 13:46:14 +0000 (21:46 +0800)]
Merge pull request #9845 from dx9/wip-config-basename

common: remove basename() dependency

Reviewed-by: Kefu Chai <kchai@redhat.com>
9 years agoMerge pull request #9969 from liewegas/wip-assert
Kefu Chai [Mon, 4 Jul 2016 13:44:17 +0000 (21:44 +0800)]
Merge pull request #9969 from liewegas/wip-assert

include/assert: clean up ceph assertion macros

Reviewed-by: Greg Farnum <gfarnum@redhat.com>
9 years agoMerge pull request #9983 from SUSE/wip-cmake-fix-fcgi-include-dir
Kefu Chai [Mon, 4 Jul 2016 13:43:24 +0000 (21:43 +0800)]
Merge pull request #9983 from SUSE/wip-cmake-fix-fcgi-include-dir

cmake: Fix FCGI include directory

Reviewed-by: Kefu Chai <kchai@redhat.com>
9 years agoMerge pull request #10093 from tchaikov/wip-16525
Kefu Chai [Mon, 4 Jul 2016 13:42:34 +0000 (21:42 +0800)]
Merge pull request #10093 from tchaikov/wip-16525

crush: reset bucket->h.items[i] when removing tree item

Reviewed-by: Sage Weil <sage@redhat.com>
9 years agoMerge pull request #9330 from dachary/wip-15176-partprobe
Kefu Chai [Mon, 4 Jul 2016 12:29:16 +0000 (20:29 +0800)]
Merge pull request #9330 from dachary/wip-15176-partprobe

ceph-disk: partprobe should block udev induced BLKRRPART

Reviewed-by: Kefu Chai <kchai@redhat.com>
9 years agoMerge pull request #9394 from xiexingguo/xxg-wip-batchpgstates
Kefu Chai [Mon, 4 Jul 2016 12:19:20 +0000 (20:19 +0800)]
Merge pull request #9394 from xiexingguo/xxg-wip-batchpgstates

mon/PGMonitor: batch filter pg states; add sanity check

Reviewed-by: Kefu Chai <kchai@redhat.com>
9 years agoos/bluestore: fix incorrect pool decoding of bnode 10117/head
xie xingguo [Mon, 4 Jul 2016 09:08:59 +0000 (17:08 +0800)]
os/bluestore: fix incorrect pool decoding of bnode

"pool" is a pointer, and we shall alter its value instead.

Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
9 years agoceph-disk: partprobe should block udev induced BLKRRPART 9330/head
Loic Dachary [Thu, 26 May 2016 07:38:47 +0000 (09:38 +0200)]
ceph-disk: partprobe should block udev induced BLKRRPART

Wrap partprobe with flock to stop udev from issuing BLKRRPART because
this is racy and frequently fails with a message like:

    Error: Error informing the kernel about modifications to partition
    /dev/vdc1 -- Device or resource busy.  This means Linux won't know about
    any changes you made to /dev/vdc1 until you reboot -- so you shouldn't
    mount it or use it in any way before rebooting.

Opening a device (/dev/vdc for instance) in write mode indirectly
triggers a BLKRRPART ioctl from udev (starting version 214 and up)
when the device is closed (see below for the udev release note).

However, if udev fails to acquire an exclusive lock (with
flock(fd, LOCK_EX|LOCK_NB); ) the BLKRRPART ioctl is not issued.

https://github.com/systemd/systemd/blob/045e00cf16c47bc516c0823d059b7548f3ce9c7c/src/udev/udevd.c#L1042

Acquiring an exclusive lock before running the process that opens the
device in write mode is therefore an effective way to control this
behavior.

git clone git://anonscm.debian.org/pkg-systemd/systemd.git
systemd/NEWS:
CHANGES WITH 214:

  * As an experimental feature, udev now tries to lock the
       disk device node (flock(LOCK_SH|LOCK_NB)) while it
       executes events for the disk or any of its partitions.
       Applications like partitioning programs can lock the
       disk device node (flock(LOCK_EX)) and claim temporary
       device ownership that way; udev will entirely skip all event
       handling for this disk and its partitions. If the disk
       was opened for writing, the close will trigger a partition
       table rescan in udev's "watch" facility, and if needed
       synthesize "change" events for the disk and all its partitions.
       This is now unconditionally enabled, and if it turns out to
       cause major problems, we might turn it on only for specific
       devices, or might need to disable it entirely. Device Mapper
       devices are excluded from this logic.

Fixes: http://tracker.ceph.com/issues/15176
Signed-off-by: Marius Vollmer <marius.vollmer@redhat com>
Signed-off-by: Loic Dachary <loic@dachary.org>
9 years agoMerge pull request #10077 from yehudasa/wip-16494 10246/head
Orit Wasserman [Mon, 4 Jul 2016 07:37:34 +0000 (09:37 +0200)]
Merge pull request #10077 from yehudasa/wip-16494

rgw: fix for issue 16494

9 years agoMerge pull request #10064 from BlaXpirit/brag-py3
Kefu Chai [Mon, 4 Jul 2016 07:00:50 +0000 (15:00 +0800)]
Merge pull request #10064 from BlaXpirit/brag-py3

Port ceph-brag to Python 3 (+ small fixes)

Reviewed-by: Kefu Chai <kchai@redhat.com>
9 years agoMerge pull request #10115 from runsisi/ss-wip-fix-rpm-spec
Kefu Chai [Mon, 4 Jul 2016 06:08:35 +0000 (14:08 +0800)]
Merge pull request #10115 from runsisi/ss-wip-fix-rpm-spec

ceph.spec.in: fix rpm package building error

Reviewed-by: Kefu Chai <kchai@redhat.com>
9 years agoceph.spec.in: fix rpm package building error as follows: 10115/head
runsisi [Mon, 4 Jul 2016 05:52:03 +0000 (13:52 +0800)]
ceph.spec.in: fix rpm package building error as follows:

      error: Installed (but unpackaged) file(s) found:
      /usr/lib/udev/rules.d/60-ceph-by-parttypeuuid.rules

Signed-off-by: runsisi <runsisi@zte.com.cn>
9 years agoMerge pull request #9885 from dachary/wip-16351-by-parttypeuuid
Loic Dachary [Sun, 3 Jul 2016 11:31:46 +0000 (13:31 +0200)]
Merge pull request #9885 from dachary/wip-16351-by-parttypeuuid

udev: always populate /dev/disk/by-parttypeuuid

Reviewed-by: Alexandre Derumier <aderumier@odiso.com>
9 years agoMerge pull request #10095 from liewegas/wip-bluestore-trim
Sage Weil [Sat, 2 Jul 2016 22:53:40 +0000 (18:53 -0400)]
Merge pull request #10095 from liewegas/wip-bluestore-trim

os/bluestore: trim cache on reads

Reviewed-by: xie xingguo <xie.xingguo@zte.com.cn>
9 years agoMerge pull request #10080 from xiexingguo/xxg-wip-add-compress-ratio
Sage Weil [Sat, 2 Jul 2016 11:32:38 +0000 (07:32 -0400)]
Merge pull request #10080 from xiexingguo/xxg-wip-add-compress-ratio

os/bluestore: improve required compression threshold

Reviewed-by: Sage Weil <sage@
9 years agomsg/simple: set close on exec on server sockets 9772/head
Kefu Chai [Fri, 17 Jun 2016 05:58:55 +0000 (13:58 +0800)]
msg/simple: set close on exec on server sockets

mds execv() when handling the "respawn" command, to avoid fd leakage,
and enormous CLOSE_WAIT connections after respawning, we need to set
FD_CLOEXEC flag for the socket fds.

Fixes: http://tracker.ceph.com/issues/16390
Signed-off-by: Kefu Chai <kchai@redhat.com>
9 years agomsg/async: set close on exec on server sockets
Kefu Chai [Thu, 16 Jun 2016 17:17:05 +0000 (01:17 +0800)]
msg/async: set close on exec on server sockets

mds execv() when handling the "respawn" command, to avoid fd leakage,
and enormous CLOSE_WAIT connections after respawning, we need to set
FD_CLOEXEC flag for the socket fds.

Fixes: http://tracker.ceph.com/issues/16390
Signed-off-by: Kefu Chai <kchai@redhat.com>
9 years agoos/bluestore: add compression required ratio to enable/disable compression 10080/head
xie xingguo [Fri, 1 Jul 2016 03:19:13 +0000 (11:19 +0800)]
os/bluestore: add compression required ratio to enable/disable compression

Require the net gain of compression at least to be at a specified ratio,
otherwise we don't compress.

Ask for compressing at least 12.5% off, by default.

This is for the sake of performance because if the compression turns out
to be meaningless(saving little space), we can simply shut it down, as we
know the compression/decompression can be rather CPU-consuming.

Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
9 years agoMerge pull request #10096 from ceph/wip-dang-cmake
Sage Weil [Fri, 1 Jul 2016 18:46:26 +0000 (14:46 -0400)]
Merge pull request #10096 from ceph/wip-dang-cmake

debian: set libexec dir to correct value as autotools did

Reviewed-by: Sage Weil <sage@redhat.com>
9 years agoMerge pull request #9941 from weiqiaomiao/wqm-wip-fetchobj
Matt Benjamin [Fri, 1 Jul 2016 18:33:40 +0000 (14:33 -0400)]
Merge pull request #9941 from weiqiaomiao/wqm-wip-fetchobj

rgw: fix memory leak in RGWRados::fetch_remote_obj()

lgtm

9 years agodebian - set libexec dir to correct value as autotools did 10096/head
Daniel Gryniewicz [Fri, 1 Jul 2016 18:12:47 +0000 (14:12 -0400)]
debian - set libexec dir to correct value as autotools did

Signed-off-by: Daniel Gryniewicz <dang@redhat.com>
9 years agoos/bluestore: trim cache on reads too 10095/head
Sage Weil [Fri, 1 Jul 2016 18:04:52 +0000 (14:04 -0400)]
os/bluestore: trim cache on reads too

We were only trimming in the write path, which would make memory explode
on a read-only workload.

Signed-off-by: Sage Weil <sage@redhat.com>
9 years agoMerge pull request #10082 from chhabaramesh/bitalloc_warn
Sage Weil [Fri, 1 Jul 2016 17:29:46 +0000 (13:29 -0400)]
Merge pull request #10082 from chhabaramesh/bitalloc_warn

os/bluestore: Remove bit alloc Woverloaded-virtual warnings

9 years agoMerge pull request #10068 from jcsp/wip-cephfs-doc-update
Sage Weil [Fri, 1 Jul 2016 17:28:17 +0000 (13:28 -0400)]
Merge pull request #10068 from jcsp/wip-cephfs-doc-update

doc: update CephFS "early adopters" info

Reviewed-by: Sage Weil <sage@redhat.com>
Reviewed-by: Greg Farnum <gfarnum@redhat.com>
9 years agoMerge pull request #9584 from chiehtu/fix-16188
pritha-srivastava [Fri, 1 Jul 2016 16:07:12 +0000 (21:37 +0530)]
Merge pull request #9584 from chiehtu/fix-16188

rgw: fix Swift API returns double space usage

Reviewed by: Pritha Srivastava <prsrivas@redhat.com>

9 years agoMerge pull request #10000 from weiqiaomiao/wqm-wip-add-userlist-cli
Matt Benjamin [Fri, 1 Jul 2016 15:40:08 +0000 (11:40 -0400)]
Merge pull request #10000 from weiqiaomiao/wqm-wip-add-userlist-cli

rgw: add "radosgw-admin user list" cli

verified

9 years agoMerge pull request #9838 from xiexingguo/xxg-wip-fix-synthetic-client
John Spray [Fri, 1 Jul 2016 14:31:22 +0000 (15:31 +0100)]
Merge pull request #9838 from xiexingguo/xxg-wip-fix-synthetic-client

client: misc fixes

Reviewed-by: John Spray <john.spray@redhat.com>
9 years agoMerge pull request #10092 from ceph/wip-dang-cmake
Sage Weil [Fri, 1 Jul 2016 14:05:20 +0000 (10:05 -0400)]
Merge pull request #10092 from ceph/wip-dang-cmake

cmake: fixes

Reviewed-by: Kefu Chai <kchai@redhat.com>
9 years agoMerge pull request #10028 from ceph/wip-cmake-readme-fixes
Sage Weil [Fri, 1 Jul 2016 13:59:53 +0000 (09:59 -0400)]
Merge pull request #10028 from ceph/wip-cmake-readme-fixes

cmake: Removed README.cmake.md, edited README.md

9 years agocrush: reset bucket->h.items[i] when removing tree item 10093/head
Kefu Chai [Fri, 1 Jul 2016 12:44:35 +0000 (20:44 +0800)]
crush: reset bucket->h.items[i] when removing tree item

* crush: so we don't see the reference after the removing, this keeps
  check_item_loc() happy, and move_bucket() use check_item_loc() to see if
  the removed bucket disappears after the removal.
* test: also add unittest_crush_wrapper::CrushWrapper.insert_item

Fixes: http://tracker.ceph.com/issues/16525
Signed-off-by: Kefu Chai <kchai@redhat.com>
9 years agoinit-ceph - use sbindir, rather than hard coding sbin 10092/head
Daniel Gryniewicz [Fri, 1 Jul 2016 13:07:43 +0000 (09:07 -0400)]
init-ceph - use sbindir, rather than hard coding sbin

Signed-off-by: Daniel Gryniewicz <dang@redhat.com>
9 years agoCMake - add libexecdir to the replaced variables
Daniel Gryniewicz [Fri, 1 Jul 2016 13:04:17 +0000 (09:04 -0400)]
CMake - add libexecdir to the replaced variables

Signed-off-by: Daniel Gryniewicz <dang@redhat.com>
9 years agoMerge pull request #10081 from tchaikov/wip-16556
John Spray [Fri, 1 Jul 2016 13:02:19 +0000 (14:02 +0100)]
Merge pull request #10081 from tchaikov/wip-16556

cmake: remove unnecessary linked libs from libcephfs

Reviewed-by: John Spray <john.spray@redhat.com>
9 years agoMerge pull request #10083 from trociny/wip-16539
Jason Dillaman [Fri, 1 Jul 2016 12:29:38 +0000 (08:29 -0400)]
Merge pull request #10083 from trociny/wip-16539

rbd-mirror: remove ceph_test_rbd_mirror_image_replay test case

Reviewed-by: Jason Dillaman <dillaman@redhat.com>
9 years agoMerge pull request #10065 from dillaman/wip-16558
Mykola Golub [Fri, 1 Jul 2016 11:19:28 +0000 (14:19 +0300)]
Merge pull request #10065 from dillaman/wip-16558

rbd-mirror: gracefully fail if object map is unavailable

Reviewed-by: Mykola Golub <mgolub@mirantis.com>
9 years agocmake: remove unnecessary linked libs from libcephfs 10081/head
Kefu Chai [Fri, 1 Jul 2016 03:09:09 +0000 (11:09 +0800)]
cmake: remove unnecessary linked libs from libcephfs

* some of the libs shares the same .cc which has static C++ variables. if
  we link against the different libs sharing the same static C++
  variables, and the dtor of the C++ variables has side-effects, among
  other things, deallocates a memory chunk. then, we are in the trouble of
  double free. so "osd" lib is removed.
* some of the libs are referenced by the linked lib, so no need to link
  against them again. for example, BLKID_LIBRARIES are linked by
  libcommon, so we can remove it from the linked libs list.
* and lib "os" and "cls_references_objs" are not used by libcephfs at all,
  so remove them.

Fixes: http://tracker.ceph.com/issues/16556
Signed-off-by: Kefu Chai <kchai@redhat.com>
9 years agoRemove bit alloc Woverloaded-virtual warnings 10082/head
Ramesh Chander [Fri, 1 Jul 2016 05:06:12 +0000 (22:06 -0700)]
Remove bit alloc Woverloaded-virtual warnings

Signed-off-by: Ramesh Chander <Ramesh.Chander@sandisk.com>
9 years agorgw: fix double counting in RGWRados::update_containers_stats() 9584/head
Albert Tu [Tue, 28 Jun 2016 02:38:07 +0000 (10:38 +0800)]
rgw: fix double counting in RGWRados::update_containers_stats()

Fixes: http://tracker.ceph.com/issues/16188
Signed-off-by: Albert Tu <albert.t@inwinstack.com>
9 years agorgw: forward input data when forwarding set_bucket_version to master 10077/head
Yehuda Sadeh [Fri, 1 Jul 2016 00:36:16 +0000 (17:36 -0700)]
rgw: forward input data when forwarding set_bucket_version to master

Fixes: http://tracker.ceph.com/issues/16494
Needed to keep input data around to be forwarded correctly. Also, master
does not send any data back, so don't try to parse anything.

Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
9 years agorgw: set args when rebuilding req_info
Yehuda Sadeh [Fri, 1 Jul 2016 00:34:38 +0000 (17:34 -0700)]
rgw: set args when rebuilding req_info

Was missing args, so it didn't sign requests with subresources
correctly when forwarding.

Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
9 years agodoc/cephfs: rename early-adopters to best-practices 10068/head
John Spray [Thu, 30 Jun 2016 15:20:20 +0000 (16:20 +0100)]
doc/cephfs: rename early-adopters to best-practices

...and give it a link in the TOC (in addition
to where it is referenced from the top of index.rst)

Signed-off-by: John Spray <john.spray@redhat.com>
9 years agodoc/cephfs: remove some scary warnings
John Spray [Thu, 30 Jun 2016 15:18:46 +0000 (16:18 +0100)]
doc/cephfs: remove some scary warnings

...and restructure the "early adopters" page
into a "best practices" guide.  Early adopters
are now just adopters :-)

Signed-off-by: John Spray <john.spray@redhat.com>
9 years agocmake: Removed README.cmake.md, edited README.md 10028/head
Ali Maredia [Wed, 29 Jun 2016 20:05:26 +0000 (16:05 -0400)]
cmake: Removed README.cmake.md, edited README.md

Deleted README.cmake.md and put any helpful
information it had into README.md.

Also a added note about which tests get executed
by ctest.

Signed-off-by: Ali Maredia <amaredia@redhat.com>
9 years agoqa/workunits/rbd: fix issues under cmake build environment 10076/head
Jason Dillaman [Thu, 30 Jun 2016 20:59:19 +0000 (16:59 -0400)]
qa/workunits/rbd: fix issues under cmake build environment

Signed-off-by: Jason Dillaman <dillaman@redhat.com>
9 years agovstart: avoid using relative paths when finding cmake directories
Jason Dillaman [Thu, 30 Jun 2016 20:57:16 +0000 (16:57 -0400)]
vstart: avoid using relative paths when finding cmake directories

If the "run" directory is a symbolic link (e.g. to fast storage),
this will fail.

Signed-off-by: Jason Dillaman <dillaman@redhat.com>
9 years agoMerge pull request #9891 from diluga/rgw_socket_backlog_with_sign
Matt Benjamin [Thu, 30 Jun 2016 20:50:31 +0000 (16:50 -0400)]
Merge pull request #9891 from diluga/rgw_socket_backlog_with_sign

RGW : setting socket backlog for via ceph.conf

9 years agoMerge pull request #9852 from dx9/wip-rgw-ldap-link
Matt Benjamin [Thu, 30 Jun 2016 20:28:05 +0000 (16:28 -0400)]
Merge pull request #9852 from dx9/wip-rgw-ldap-link

rgw-ldap: add ldap lib to rgw lib deps based on build config

9 years agoMerge pull request #9931 from Yan-waller/yj-wip-rgwmainccfix
Matt Benjamin [Thu, 30 Jun 2016 20:12:02 +0000 (16:12 -0400)]
Merge pull request #9931 from Yan-waller/yj-wip-rgwmainccfix

rgw: remove uneccessary temporary variable and fix some indents

cosmetic changes only--build verified, f23

9 years agoMerge pull request #9979 from Yan-waller/yj-wip-rgwrests3cc
Matt Benjamin [Thu, 30 Jun 2016 19:53:02 +0000 (15:53 -0400)]
Merge pull request #9979 from Yan-waller/yj-wip-rgwrests3cc

rgw: fix potential memory leaks and remove unnecessary assignment

lgtm--should backport

9 years agoMerge pull request #10031 from yehudasa/wip-16530
Casey Bodley [Thu, 30 Jun 2016 19:02:54 +0000 (15:02 -0400)]
Merge pull request #10031 from yehudasa/wip-16530

rgw: finish error_repo cr in stop_spawned_services()

Reviewed-by: Casey Bodley <cbodley@redhat.com>
9 years agorbd-mirror: remove ceph_test_rbd_mirror_image_replay test case 10083/head
Mykola Golub [Thu, 30 Jun 2016 13:18:56 +0000 (16:18 +0300)]
rbd-mirror: remove ceph_test_rbd_mirror_image_replay test case

Fixes: http://tracker.ceph.com/issues/16539
Signed-off-by: Mykola Golub <mgolub@mirantis.com>
9 years agoMerge pull request #9985 from Yan-waller/yj-wiph-rgwopccfixstring
Matt Benjamin [Thu, 30 Jun 2016 18:05:23 +0000 (14:05 -0400)]
Merge pull request #9985 from Yan-waller/yj-wiph-rgwopccfixstring

rgw: use string::npos while 'string::find()' find nothing

lgtm--build verified, f23

9 years agoMerge pull request #9898 from liewegas/wip-bluefs-async-flush
Sage Weil [Thu, 30 Jun 2016 17:50:46 +0000 (13:50 -0400)]
Merge pull request #9898 from liewegas/wip-bluefs-async-flush

os/bluestore/BlueFS: do not hold internal lock while waiting for IO

9 years agoMerge pull request #10002 from Yan-waller/yj-wip-rgwradosccremoveunused
Matt Benjamin [Thu, 30 Jun 2016 17:49:00 +0000 (13:49 -0400)]
Merge pull request #10002 from Yan-waller/yj-wip-rgwradosccremoveunused

rgw: remove uneccessary temporary variable 'r'

lgtm--build verified, f23

9 years agoMerge pull request #10067 from liewegas/wip-do-cmake
Sage Weil [Thu, 30 Jun 2016 17:45:30 +0000 (13:45 -0400)]
Merge pull request #10067 from liewegas/wip-do-cmake

do_cmake.sh: set up initial plugin dir

9 years agoos/bluestore/BlueFS: debug statements to check log_fnode size in super block 9898/head
Varada Kari [Wed, 29 Jun 2016 13:16:04 +0000 (18:46 +0530)]
os/bluestore/BlueFS: debug statements to check log_fnode size in super block

Signed-off-by: Varada Kari <varada.kari@sandisk.com>
9 years agoos/bluestore/Bluefs: Unit test cases for bluefs async flush
Varada Kari [Wed, 29 Jun 2016 13:13:55 +0000 (18:43 +0530)]
os/bluestore/Bluefs: Unit test cases for bluefs async flush

Signed-off-by: Varada Kari <varada.kari@sandisk.com>
9 years agoos/bluestore/BlueFS: Make fysnc return the status
Varada Kari [Wed, 29 Jun 2016 13:12:24 +0000 (18:42 +0530)]
os/bluestore/BlueFS: Make fysnc return the status

Signed-off-by: Varada Kari <varada.kari@sandisk.com>
9 years agoos/bluestore/BlueFS: make _sync_and_flush_log smarter
Sage Weil [Fri, 24 Jun 2016 13:43:52 +0000 (09:43 -0400)]
os/bluestore/BlueFS: make _sync_and_flush_log smarter

If we know what event we need to wait for, only wait long enough for it
to flush.  This helps the situation where another thread flushed what we
needed, and more dirty stuff was added to log_t, but we don't need to
wait for that too for our caller to be happy.

Signed-off-by: Sage Weil <sage@redhat.com>
9 years agoos/bluestore: drop lock while we flush the log
Sage Weil [Fri, 24 Jun 2016 13:23:21 +0000 (09:23 -0400)]
os/bluestore: drop lock while we flush the log

Handle cases where we have multiple racing threads trying to flush the
log by only allowing one concurrent log flush to be in progress at a time,
and behave if, after flushing, there are no more dirty records to flush.

Signed-off-by: Sage Weil <sage@redhat.com>
9 years agoos/bluestore/BlueFS: use existing IOContext for _write_super
Sage Weil [Thu, 23 Jun 2016 15:23:46 +0000 (11:23 -0400)]
os/bluestore/BlueFS: use existing IOContext for _write_super

We need to be careful with IOContext destruction racing with the bdev
io completion thread.

Signed-off-by: Sage Weil <sage@redhat.com>
9 years agoos/bluestore/BlueFS: pass unique_lock down through _fsync and log flush
Sage Weil [Fri, 24 Jun 2016 13:21:24 +0000 (09:21 -0400)]
os/bluestore/BlueFS: pass unique_lock down through _fsync and log flush

Signed-off-by: Sage Weil <sage@redhat.com>
9 years agoos/bluestore/BlueFS: carry lock for mkfs
Sage Weil [Thu, 23 Jun 2016 14:10:17 +0000 (10:10 -0400)]
os/bluestore/BlueFS: carry lock for mkfs

Signed-off-by: Sage Weil <sage@redhat.com>
9 years agoos/bluestore/KernelDevice: rebuild buffer if we exceed IOV_MAX
Sage Weil [Thu, 23 Jun 2016 14:00:17 +0000 (10:00 -0400)]
os/bluestore/KernelDevice: rebuild buffer if we exceed IOV_MAX

Signed-off-by: Sage Weil <sage@redhat.com>
9 years agocommon/buffer: assert that we do not exceed IOV_MAX in prepare_iov
Sage Weil [Thu, 23 Jun 2016 14:00:01 +0000 (10:00 -0400)]
common/buffer: assert that we do not exceed IOV_MAX in prepare_iov

Signed-off-by: Sage Weil <sage@redhat.com>
9 years agounittest_bluefs: don't silence log
Sage Weil [Thu, 23 Jun 2016 13:55:20 +0000 (09:55 -0400)]
unittest_bluefs: don't silence log

Signed-off-by: Sage Weil <sage@redhat.com>
9 years agoos/bluestore/BlueFS: _flush_bdev -> flush_bdev
Sage Weil [Thu, 23 Jun 2016 13:48:21 +0000 (09:48 -0400)]
os/bluestore/BlueFS: _flush_bdev -> flush_bdev

It is safe to call without a lock.

Signed-off-by: Sage Weil <sage@redhat.com>
9 years agoos/bluestore/BlueFS: drop lock while waiting for user io to complete
Sage Weil [Thu, 23 Jun 2016 13:39:31 +0000 (09:39 -0400)]
os/bluestore/BlueFS: drop lock while waiting for user io to complete

_flush_wait is safe to call without a lock, as long as our reference is
stable.  Rename it wait_for_aio() to be more clear about what it does and
the fact that it doesn't require a lock.

Signed-off-by: Sage Weil <sage@redhat.com>
9 years agoos/bluestore/BlueFS: track dirty by log_seq, log_seq_stable
Sage Weil [Thu, 23 Jun 2016 13:33:09 +0000 (09:33 -0400)]
os/bluestore/BlueFS: track dirty by log_seq, log_seq_stable

Note when we dirty a file, and clean it only if that seq has been
committed.  Currently this is always the case because we don't drop the
lock, but that will change shortly.

Signed-off-by: Sage Weil <sage@redhat.com>
9 years agoos/bluestore/BlueFS: do not dirty log file when size changes
Sage Weil [Thu, 23 Jun 2016 13:26:16 +0000 (09:26 -0400)]
os/bluestore/BlueFS: do not dirty log file when size changes

We only need to dirty the log file if the allocate changes.  Replay is
smart enough to learn the file size as it goes.

Signed-off-by: Sage Weil <sage@redhat.com>
9 years agoos/bluestore/BlueFS: track log_seq_stable
Sage Weil [Wed, 22 Jun 2016 21:42:32 +0000 (17:42 -0400)]
os/bluestore/BlueFS: track log_seq_stable

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