1
0
mirror of https://github.com/ceph/ceph synced 2024-12-25 12:54:16 +00:00
Commit Graph

42350 Commits

Author SHA1 Message Date
Loic Dachary
1e77fcfd98 tests: ceph-disk tests need to install pip > 6.1
Otherwise it will not be able to use the wheelhouse.

http://tracker.ceph.com/issues/11952 Fixes: 

Signed-off-by: Loic Dachary <loic@dachary.org>
2015-06-10 21:00:42 +02:00
Loic Dachary
9c6da3b087 Merge pull request from ceph/wip-cot-list
tools: For ec pools list objects in all shards if the pgid doesn't sp…

Reviewed-by: Loic Dachary <ldachary@redhat.com>
2015-06-10 20:28:05 +02:00
Sage Weil
c06fd71a1a Merge remote-tracking branch 'gh/next' 2015-06-10 13:14:46 -04:00
Loic Dachary
1c1f2c3243 Merge pull request from dachary/wip-11932-non-regression-typo
tests: ceph_erasure_code_non_regression s/stipe/stripe/
2015-06-10 17:43:55 +02:00
Yehuda Sadeh
20f49348a3 Merge pull request from liewegas/wip-rgw-civetweb-port
rgw: strip off port from HTTP_HOST when determining bucket from URL

Reviewed-by: Yehuda Sadeh <yehuda@redhat.com>
2015-06-10 08:12:05 -07:00
Gregory Farnum
5441f89c02 Merge pull request from ceph/wip-10950
: Fix confusing errors using "mds rm"

Reviewed-by: Greg Farnum <gfarnum@redhat.com>
2015-06-10 07:18:48 -07:00
Gregory Farnum
04a11f0f2f Merge pull request from ceph/wip-11746
tools: chunk reads in Dumper

Reviewed-by: Greg Farnum <gfarnum@redhat.com>
2015-06-10 07:17:43 -07:00
Gregory Farnum
b408f009b6 Merge pull request from tchaikov/wip-drop-ignored-mdsbeacon
mon: drop ignored mdsbeacon

Reviewed-by: Greg Farnum <gfarnum@redhat.com>
Reviewed-by: Joao Eduardo Luis <joao@suse.de>
2015-06-10 07:15:38 -07:00
Loic Dachary
2d76e2cbba tests: erasure-code non regression tests must skip isa on i386
When the isa plugin is not present, the isa plugin payloads are not
tested. When that happens, remember that it was intentional by storing
the path that would have been tested so that the inventory is complete.

http://tracker.ceph.com/issues/11949 Fixes: 

Signed-off-by: Loic Dachary <loic@dachary.org>
2015-06-10 15:26:37 +02:00
Jianpeng Ma
e68ea2c942 osdc: Make librbd cache handle NOCACHE fadvise flag.
Signed-off-by: Jianpeng Ma <jianpeng.ma@intel.com>
2015-06-10 16:25:28 +08:00
wuxingyi
ac1e729a75 rgw: fix data corruption when race condition
We should delete the object in the multipart namespace lastly to prevent a previous upload
wrongly deleting objects belong to the following upload.

Fixes: 
Signed-off-by: wuxingyi <wuxingyi@letv.com>
2015-06-10 06:58:40 +00:00
Kefu Chai
5a5dc5f0aa Merge pull request from stiopaa1/IosFwdNew
Removed unnecessary inclusion of iostream in several files

Reviewed-by: Kefu Chai <kchai@redhat.com>
2015-06-10 13:22:00 +08:00
David Zafman
1aa9655f10 tools: For ec pools list objects in all shards if the pgid doesn't specify
Signed-off-by: David Zafman <dzafman@redhat.com>
2015-06-09 18:14:09 -07:00
Sage Weil
98e77d592f mon/PGMap: access num pgs by osd
Signed-off-by: Sage Weil <sage@redhat.com>
2015-06-09 17:56:42 -04:00
Sage Weil
fa0483385b mon/PGMap: fix pg_by_osd
Broken since it was introduced in 7a04762fe0.

Signed-off-by: Sage Weil <sage@redhat.com>
2015-06-09 17:56:42 -04:00
Sage Weil
abe4ec293d rgw: remove trailing :port from host for purposes of subdomain matching
Some clients (ahem, CrossFTP) include the :port in the HTTP_HOST header.
Strip it out.

Switch req_info field to a std::string and avoid copying it in preprocess.

Signed-off-by: Sage Weil <sage@redhat.com>
2015-06-09 14:15:10 -04:00
Sage Weil
8d7024a20e Merge pull request from yuyuyu101/wip-async-crc-decouple
AsyncMessenger: Make send/receive logic independent

Reviewed-by: Sage Weil <sage@redhat.com>
2015-06-09 10:54:11 -07:00
Loic Dachary
e4011158e7 tests: ceph_erasure_code_non_regression s/stipe/stripe/
Synchronize withe the ceph-erasure-code-corpus submodule in which all
file names were modified to fix the typo.

http://tracker.ceph.com/issues/11932 Fixes: 

Signed-off-by: Loic Dachary <ldachary@redhat.com>
2015-06-09 16:35:02 +02:00
Loic Dachary
7e0536af25 Merge pull request from dachary/wip-9720-erasure-code-corpus-variants
tests: sync ceph-erasure-code-corpus to verify jerasure variants 

Reviewed-by: Andreas Peters <andreas.joachim.peters@cern.ch>
2015-06-09 16:28:48 +02:00
Loic Dachary
b239945374 Merge pull request from dachary/wip-11931-parallel-tests
tests: automake 1.11 needs parallel-tests

Reviewed-by: Nathan Cutler <ncutler@suse.com>
2015-06-09 15:50:12 +02:00
Loic Dachary
9193afd3d2 tests: automake 1.11 needs parallel-tests
Prior to automake 1.13 the default behavior was serial-tests meaning
tests from make check were run one after the other and their output sent
to stdout/stderr. From automake 1.13 up the default became
parallel-tests which logs the output of each individual test in a
separate .log file and allows them to run in parallel.

http://lists.gnu.org/archive/html/automake/2012-12/msg00038.html

Enable parallel-tests so that tests can always run in parallel, even
with automake 1.11 which is the version found in CentOS 6.5 and Ubuntu
12.04 precise.

A nice side effect of always having the logs in separate files is that
tests do not need to worry about being too verbose because their output
will only be read for diagnostic purposes.

http://tracker.ceph.com/issues/11931 Fixes: 
http://tracker.ceph.com/issues/11906 Fixes: 

Signed-off-by: Loic Dachary <ldachary@redhat.com>
2015-06-09 14:56:49 +02:00
Michal Jarzabek
83f819877f Removed unnecessary inclusion of iostream
In several files the iostream wasn't being used, so it got removed.
In other files the iostream inclusion was replaced by including iosfwd
(for forward declarations), which is much smaller header than iostream,
  so in theory should reduce compilation time.
To make this work some of the functions must have been moved from .h to .cc file.
3 functions also needed to have inline removed - this shouldn't affect
performance in any way: two of them are
probably too long to have been inlined anyway and the third one is for
error reporting, so probably won't be called too often.

test/Makefile-client.am: added linker libs

This was required to avoid linker error when linking
src/test/cls_rbd/test_cls_rbd.cc file. Makefile was specyfing
libcommon.a as a part of a linker command even though this wasn't
required and wasn't being linked against. When inline functions from
buffer.h were moved to buffer.cc(and inline was removed) the
libcommon.a library became necessary. This wouldn't link without also
including additional libraries(CRYPTO_LIBS and EXTRA_LIBS)

Signed-off-by: Michal Jarzabek <stiopa@gmail.com>
2015-06-09 07:26:56 +01:00
Loic Dachary
bfb92bd14b Merge pull request from SUSE/wip-master-specfile-cleanup
ceph.spec.in: remove duplicate BuildRequires

Reviewed-by: Loic Dachary <ldachary@redhat.com>
Reviewed-by: Kefu Chai <kchai@redhat.com>
2015-06-09 07:32:17 +02:00
Samuel Just
228ee47cfd Merge pull request from ceph/wip-10794
ceph-objectstore-tool improvements

Reviewed-by: Samuel Just <sjust@redhat.com>
2015-06-08 19:04:33 -07:00
Josh Durgin
ebc38f4a4b Merge pull request from theanalyst/doc/arch-watch-notify
doc: architecture minor fixes in watch notify

Reviewed-by: Josh Durgin <jdurgin@redhat.com>
2015-06-08 12:36:35 -07:00
Abhishek Lekshmanan
437c9e1e25 doc: architecture minor fixes in watch notify
Signed-off-by: Abhishek Lekshmanan <abhishek.lekshmanan@ril.com>
2015-06-09 00:21:58 +05:30
Nathan Cutler
21f9e1f66c ceph.spec.in: remove duplicate BuildRequires
Signed-off-by: Nathan Cutler <ncutler@suse.com>
2015-06-08 20:11:15 +02:00
Abhishek Dixit
b711e3124f Transaction Id added in response
Signed-off-by: Abhishek Dixit dixitabhi@gmail.com
2015-06-08 22:59:54 +05:30
Ken Dreyer
142a6ce711 Merge pull request from SUSE/wip-master-specfile-cleanup2
ceph.spec.in: move specific BuildRequires to where they belong

Reviewed-by: Loic Dachary <ldachary@redhat.com>
Reviewed-by: Ken Dreyer <kdreyer@redhat.com>
2015-06-08 10:27:44 -06:00
Kefu Chai
2c1f4d4afb Merge pull request from dachary/wip-11272-ceph-disk-false-negative
tests: ceph-disk.sh test zap gitbuilder false negative

Reviewed-by: Kefu Chai <kchai@redhat.com>
2015-06-08 22:12:46 +08:00
Kefu Chai
66f3b192ae Merge pull request from dachary/wip-11905-test-isa
tests: skip isa tests if the plugin is not available

Reviewed-by: Kefu Chai <kchai@redhat.com>
2015-06-08 10:38:52 +08:00
Loic Dachary
3e12863784 tests: ceph-disk.sh test zap gitbuilder false negative
When looking for an error message and testing ceph disk zap, use grep -q
to not display the error message that will be mistaken for a real error
by the gitbuilder parser.

http://tracker.ceph.com/issues/11272 Refs: 

Signed-off-by: Loic Dachary <ldachary@redhat.com>
2015-06-07 13:52:01 +02:00
Loic Dachary
4d58c07490 tests: skip isa tests if the plugin is not available
http://tracker.ceph.com/issues/11905 Fixes: 

Signed-off-by: Loic Dachary <ldachary@redhat.com>
2015-06-07 10:53:49 +02:00
Loic Dachary
90baef866f tests: use erasure_code_plugin_exists from ceph-helpers.sh
Instead of the local plugin_exists helper that does the same.

Signed-off-by: Loic Dachary <ldachary@redhat.com>
2015-06-07 10:52:31 +02:00
Loic Dachary
deb651b61c tests: implement erasure_code_plugin_exists in ceph-helpers.sh
Return 0 if the erasure code *plugin* is available, 1 otherwise.

Signed-off-by: Loic Dachary <ldachary@redhat.com>
2015-06-07 10:51:07 +02:00
Loic Dachary
2beba7c9d1 Merge pull request from dachary/wip-10358-erasure-code-k-1-m-1
ceph osd erasure-code-profile set myprofile k=1 should fail

Reviewed-by: Andreas Peters <andreas.joachim.peters@cern.ch>
2015-06-07 09:58:26 +02:00
Loic Dachary
fda586f5f8 tests: sync ceph-erasure-code-corpus to verify jerasure variants
http://tracker.ceph.com/issues/9720 Refs: 

Signed-off-by: Loic Dachary <ldachary@redhat.com>
2015-06-07 00:15:05 +02:00
Loic Dachary
864e2007d9 tests: add --{show_,}path to ceph_erasure_code_non_regression
The directory in which the payload is stored is created from the plugin
parameters. The --show-path shows the directory and exits. This
directory can then be used with --path to override the path created from
the plugin parameters. This is useful to verifying that the jerasure
variant optimized with AVX, SSE4 etc. instructions can be used on the
same payload and show no difference at all. In this case the directory
used by the default jerasure variant is used for each variant and the
parameter that sets the variant to use ( --parameters jerasure-variant )
must not be taken into account to figure out the location of the
payload.

http://tracker.ceph.com/issues/9720 Refs: 

Signed-off-by: Loic Dachary <ldachary@redhat.com>
2015-06-07 00:15:05 +02:00
Loic Dachary
1d60786c39 Merge pull request from dachary/wip-11346-isa
ceph-erasure-code-corpus: isa: add k=10 m=4

Reviewed-by: Yuan Zhou <yuan.zhou@intel.com>
2015-06-07 00:12:11 +02:00
Loic Dachary
3ec41030cc erasure-code: add sanity check to guard against k=1
Add a call to ErasureCode::sanity_check_k for the isa and jerasure
plugins, with associated tests.

http://tracker.ceph.com/issues/10358 Fixes: 

Signed-off-by: Loic Dachary <ldachary@redhat.com>
2015-06-06 23:51:57 +02:00
Sage Weil
efededa010 Merge pull request from SUSE/wip-submitting-patches-clarification
SubmittingPatches: clarify how Reviewed-by lines are added

Reviewed-by: Sage Weil <sage@redhat.com>
2015-06-06 06:24:17 -07:00
Nathan Cutler
85b327f85a SubmittingPatches: clarify how Reviewed-by lines are added
Signed-off-by: Nathan Cutler <ncutler@suse.com>
2015-06-06 15:16:16 +02:00
Loic Dachary
eefdeb6b79 Merge pull request from dachary/wip-11901-spec
ceph.spec.in: add missing BuildRequires from SUSE block

Reviewed-by: Nathan Cutler <ncutler@suse.cz>
2015-06-06 14:37:23 +02:00
Nathan Cutler
e1cb7e5104 ceph.spec.in: move specific BuildRequires to where they belong
Move distro-specific BuildRequires out of "common" section and
into the appropriate %if statement in the "specific" section.
Also remove a duplicated "Requires: gdisk".

Signed-off-by: Nathan Cutler <ncutler@suse.com>
Reviewed-by: Loic Dachary <ldachary@redhat.com>
2015-06-06 14:14:50 +02:00
Loic Dachary
739ef0f9a3 install-deps.sh: detect yum-builddep errors
yum-builddep does not exit on error when an error happens: grep the
output for the error: string instead.

Signed-off-by: Loic Dachary <ldachary@redhat.com>
2015-06-06 12:31:56 +02:00
Loic Dachary
34e4dd9c8a ceph.spec.in: add missing BuildRequires from SUSE block
The %endif removed by f94f23297c is restored.
The %else removed by 75e87a20da is restored.
The stray %endif added by d8abde3338 is removed.

May 29th, in d8abde3338 Owen added a stray
%endif after

   BuildRequires: gperftools-devel

around line 133. June 3rd, in f94f23297c
Ken correctly attributed the error

     "error: /srv/autobuild-ceph/gitbuilder.git/build/ceph.spec:140: Got a
      %endif with no %if"

to a stray %endif but did not remove the one causing problem and in
doing so created another problem. June 4th, in
75e87a20da Owen incorrectly fixed
this new problem by removing the %else that is near

   BuildRequires: gperftools-devel

around line 116, instead of reverting
f94f23297c. As a consequence the
ceph.spec.in became syntactically correct but implemented an if/else
logic different from what was intended originally and a number of
BuildRequires became exclusive to SUSE and were not installed for CentOS
7 etc.

http://tracker.ceph.com/issues/11901 Fixes: 

Signed-off-by: Loic Dachary <ldachary@redhat.com>
2015-06-06 12:05:19 +02:00
Loic Dachary
37f7360760 erasure-code: implement ErasureCode::sanity_check_k
Implement the ErasureCode::sanity_check_k helper for plugins
that need to verify k >= 2.

http://tracker.ceph.com/issues/10358 Fixes: 

Signed-off-by: Loic Dachary <ldachary@redhat.com>
2015-06-06 09:43:49 +02:00
Noah Watkins
a8500ce981 Merge pull request from liewegas/wip-11461
fix build on arm w/ old kernels

Signed-off-by: Noah Watkins <nwatkins@redhat.com>
2015-06-05 12:12:13 -07:00
Greg Farnum
905a867043 Merge remote-tracking branch 'origin/wip-client-fsync'
Conflicts:
	src/client/Client.cc

Reviewed-by: Greg Farnum <gfarnum@redhat.com>
2015-06-05 11:25:21 -07:00
Gregory Farnum
3f4dfb80e3 Merge pull request from ceph/wip-11807
qa: fix multiple_rsync.sh to avoid using /usr/ directly

Reviewed-by: Greg Farnum <gfarnum@redhat.com>
Reviewed-by: Sage Weil <sage@redhat.com>
2015-06-05 11:22:53 -07:00