Commit Graph

34025 Commits

Author SHA1 Message Date
George Ryall
66a5f3b9ea doc: Fixed Typo in pools documentation - replaced '-' with '_' in example set-quota comands. 2014-06-20 17:54:47 +01:00
Loic Dachary
2c8ff50fb1 Merge pull request #1997 from ceph/wip-tiermsg
mon: name instead of id in "has tiers" message

Reviewed-by: Loic Dachary <loic@dachary.org>
2014-06-20 09:49:32 +02:00
John Spray
97772c2f53 mon: name instead of id in "has tiers" message
Instead of "Pool foo has tiers 1 2" print
"Pool foo has tiers bar baz".

Signed-off-by: John Spray <jspray@redhat.com>
2014-06-19 13:16:23 -04:00
Loic Dachary
34ba8faf75 Merge pull request #1988 from dachary/wip-warnings
test: fix -Wsign-compare warnings

Reviewed-by: Sahid Orentino Ferdjaoui <sahid.ferdjaoui@cloudwatt.com>
2014-06-19 18:06:49 +02:00
Loic Dachary
708776cc35 Merge pull request #1993 from dachary/fix_ut
autotools: avoid check_SCRIPTS duplication

Reviewed-by: Wei Luo <luowei@yahoo-inc.com>
2014-06-19 17:41:25 +02:00
John Wilkins
f9eb77b14d Merge pull request #1995 from sjahl/master
doc: Fixed an incorrect flag in radosgw admin docs.
2014-06-19 05:32:30 -07:00
Stephen Jahl
d2e852e2e7 doc: Fixed an incorrect flag in radosgw admin docs.
Example use a "--gen-key" flag, which doesn't exist. Changed it to
"--gen-access-key".

Signed-off-by: Stephen Jahl <stephenjahl@gmail.com>
2014-06-19 07:44:45 -04:00
Loic Dachary
741ad3ff3b autotools: avoid check_SCRIPTS duplication
The check_SCRIPTS content must be added to EXTRA_DIST, otherwise it will
not be included by make dist and it won't be possible to run make check
successfully.

One solution would be to add $(check_SCRIPTS) to EXTRA_DIST to avoid
duplication and help with long term maintenance. However, $(srcdir) is
not supported in the content of the check_SCRIPTS variable.

A GNU Make variable substitution (patsubst) is used to prepend $(srcdir)
to each script, only when used in the EXTRA_DIST variable.

Signed-off-by: Loic Dachary <loic@dachary.org>
2014-06-19 09:54:32 +02:00
Loic Dachary
c3678ad80a Merge pull request #1992 from luowei62/fix_ut
Fix dist package run unit test failed.

Reviewed-by: Loic Dachary <loic@dachary.org>
2014-06-19 09:38:19 +02:00
Loic Dachary
5fee807810 Merge pull request #1989 from dachary/wip-8622-rados-alignment
tests: prevent gitbuilder trigger in test-erasure-code.sh

Reviewed-by: Lluis Pamies-Juarez <lluis@pamies.cat>
2014-06-19 08:13:26 +02:00
luowei
c0d78c62e8 Fix dist package run unit test failed.
Signed-off-by: Wei Luo <luowei@yahoo-inc.com>
2014-06-19 04:00:33 +00:00
Loic Dachary
b3ace76dea tests: prevent gitbuilder trigger in test-erasure-code.sh
Use the INFO keywork instead of WARNING so that gitbuilder does not
confuse this with a warning worth fixing.

Signed-off-by: Loic Dachary <loic@dachary.org>
2014-06-19 00:09:14 +02:00
Loic Dachary
3a9c0fcd85 test: fix -Wsign-compare warnings
Signed-off-by: Loic Dachary <loic@dachary.org>
2014-06-19 00:02:21 +02:00
Loic Dachary
15f03388b9 Merge pull request #1987 from dachary/wip-8622-rados-alignment
erasure-code: verify that rados put enforces alignment

Reviewed-by: Lluis Pamies-Juarez <lluis@pamies.cat>
2014-06-18 23:21:20 +02:00
Loic Dachary
b46c405601 erasure-code: verify that rados put enforces alignment
http://tracker.ceph.com/issues/8622 refs: #8622

Signed-off-by: Loic Dachary <loic@dachary.org>
2014-06-18 23:13:34 +02:00
Loic Dachary
650055722e Merge pull request #1983 from dachary/wip-osd-config
tests: remove spurious and harmless find command

Reviewed-by: Florent Flament <florent.flament@cloudwatt.com>
2014-06-18 23:12:41 +02:00
Loic Dachary
7182b45599 Merge pull request #1984 from llpamies/wip-8622
enforce rados put aligment

Reviewed-by: Loic Dachary <loic@dachary.org>
2014-06-18 23:01:07 +02:00
Samuel Just
6aa3440d94 Merge pull request #1943 from athanatos/wip-8507
Wip 8507

Reviewed-by: Sage Weil <sage@inktank.com>
2014-06-18 11:32:23 -07:00
Samuel Just
e6466aa94d Merge pull request #1986 from ceph/wip-scrub-sleep
osd: introduce simple sleep during scrub

Reviewed-by: Samuel Just <sam.just@inktank.com>
2014-06-18 11:16:18 -07:00
Sage Weil
48ed75b84f Merge pull request #1985 from concubidated/master
ReplicatedPG: Fix 'ajusted' typo

Reviewed-by: Sage Weil <sage@inktank.com>
2014-06-18 11:09:14 -07:00
Concubidated
c35ceef57c ReplicatedPG: 'ajusted' typo
Signed-off-by: <Tyler Brekke> tyler.brekke@inktank.com
2014-06-18 10:56:25 -07:00
Sylvain Munaut
de2c085c0f rgw-admin: Fix the 'show log' command
Fixes: #8553

Reviewed-by: Yehuda Sadeh <yehuda@inktank.com>
Signed-off-by: Sylvain Munaut <s.munaut@whatever-company.com>
2014-06-18 10:26:44 -07:00
Lluis Pamies-Juarez
304b08a23a enforce rados put aligment
Signed-off-by: Lluis Pamies-Juarez <lluis.pamies-juarez@hgst.com>
2014-06-18 10:00:09 -07:00
Loic Dachary
8d9201fd51 tests: remove spurious and harmless find command
Signed-off-by: Loic Dachary <loic@dachary.org>
2014-06-18 15:58:28 +02:00
Sage Weil
c4e8451cc5 osd: introduce simple sleep during scrub
This option is similar to osd_snap_trim_sleep: simply inject an optional
sleep in the thread that is doing scrub work.  This is a very kludgey and
coarse knob for limiting the impact of scrub on the cluster, but can help
until we have a more robust and elegant solution.

Only sleep if we are in the NEW_CHUNK state to avoid delaying processing of
an in-progress chunk.  In this state nothing is blocked on anything.
Conveniently, chunky_scrub() requeues itself for each new chunk.

Backport: firefly, dumpling
Signed-off-by: Sage Weil <sage@inktank.com>
2014-06-17 10:47:24 -07:00
Sage Weil
af503402d5 Merge pull request #1968 from ceph/wip-ceph-isatty
ceph: output prompt only if stdin is tty

Reviewed-by: Sage Weil <sage@inktank.com>
2014-06-17 10:39:25 -07:00
Gregory Farnum
ab12a03675 Merge pull request #1947 from ceph/wip-import-purge
Wip import purge

Reviewed-by: Greg Farnum <greg@inktank.com>
2014-06-16 15:08:17 -07:00
Loic Dachary
9cc1a7580f Merge pull request #1970 from dachary/wip-erasure-code-profile-default
Revert "erasure-code: create default profile if necessary"
2014-06-16 22:04:03 +02:00
Loic Dachary
0bb0095b5f Revert "erasure-code: create default profile if necessary"
This reverts commit 360de6adf2.

Conflicts:
	src/mon/OSDMonitor.cc
2014-06-16 21:41:00 +02:00
Sage Weil
f53bed1b79 mon/OSDMonitor: fix build error
This broke with 360de6adf2.

Signed-off-by: Sage Weil <sage@inktank.com>
2014-06-16 12:28:33 -07:00
Loic Dachary
1c72465187 osd: verify osd config sanity checks
Signed-off-by: Loic Dachary <loic@dachary.org>
2014-06-16 12:28:33 -07:00
Loic Dachary
84948517e9 Merge pull request #1951 from sahid/ut-lru
Adds tests to include/lru.h + fixes small issues

Reviewed-by: Loic Dachary <loic@dachary.org>
Reviewed-by: Sage Weil <sage@inktank.com>
2014-06-16 19:16:13 +02:00
Sage Weil
a0689eccb8 Merge pull request #1954 from dachary/wip-8307-erasure-code-profile-implicit-creation
erasure code profile implicit creation

Reviewed-by: Joao Eduardo Luis <joao.luis@inktank.com>
2014-06-16 09:35:05 -07:00
Sage Weil
e0315268b0 Merge pull request #1964 from ceph/wip-osd-configs
osd: add sanity check/warning on a few key configs

Reviewed-by: Loic Dachary <loic@dachary.org>
2014-06-16 09:30:34 -07:00
Sage Weil
11ce9119ce Merge pull request #1966 from dachary/wip-erasure-code-profile-default
DNM: erasure-code: create default profile if necessary

Reviewed-by: Sage Weil <sage@inktank.com>
2014-06-16 09:29:28 -07:00
Sage Weil
4bd1b5e2b0 PendingReleaseNotes: note about keyvaluestore-dev on-disk format change
Signed-off-by: Sage Weil <sage@inktank.com>
2014-06-16 09:17:36 -07:00
Sage Weil
d3d60af4cf Merge pull request #1941 from yuyuyu101/wip-8564
Fix write operation on a deleted object in the same transaction(KeyValueStore)

Reviewed-by: Sage Weil <sage@inktank.com>
2014-06-16 09:13:49 -07:00
Sahid Orentino Ferdjaoui
d93e74e7e7 common: Enforces the methods lru_pin() and lru_unpin()
If lru_*pin() is called twice, the counter will be incr/decr
incorrectly since it will count more/less pinned objects than
there is and so corrupts the balancing (lru_adjust()).

Signed-off-by: Sahid Orentino Ferdjaoui <sahid.ferdjaoui@cloudwatt.com>
2014-06-16 15:54:57 +02:00
Sahid Orentino Ferdjaoui
d48ed68a70 common: Fixes issue with lru_clear() + add new test
The method lru_clear() must set attribute lru_num to zero
after lru_top, lru_bot and lru_mid are reseted. indeed, lru_num
is the total number of elements found in all of them.

Also the test insures the good behavior of the method
lru_adjust() - lru_touch() calls lru_adjust every time
to balance lru_top and lru_bot by the value of lru_midpoint.

Signed-off-by: Sahid Orentino Ferdjaoui <sahid.ferdjaoui@cloudwatt.com>
2014-06-16 15:54:53 +02:00
Sahid Orentino Ferdjaoui
62aa5c5c96 common: Adds simple tests to verify good behavior
Adds new tests to the lib include/lru.h to verify methods
insert_*.

Signed-off-by: Sahid Orentino Ferdjaoui <sahid.ferdjaoui@cloudwatt.com>
2014-06-16 15:54:48 +02:00
Ilya Dryomov
64f623260a ceph: output prompt only if stdin is tty
In loop mode, output prompt only if stdin is tty, i.e. if we are
interactive.

Signed-off-by: Ilya Dryomov <ilya.dryomov@inktank.com>
2014-06-16 13:12:40 +04:00
Loic Dachary
360de6adf2 erasure-code: create default profile if necessary
http://tracker.ceph.com/issues/8601

Signed-off-by: Loic Dachary <loic@dachary.org>
2014-06-15 16:54:02 +02:00
Loic Dachary
1fb4574116 Merge pull request #1960 from dachary/wip-test-kill-race
tests: prevent kill race condition

Reviewed-by: Joao Eduardo Luis <joao.luis@inktank.com>
2014-06-15 11:56:54 +02:00
Sage Weil
f3ec7d0b23 osd: add sanity check/warning on a few key configs
Warn when certain config values are set to bad values.

Backport: firefly, dumpling
Signed-off-by: Sage Weil <sage@inktank.com>
2014-06-14 10:30:50 -07:00
Josh Durgin
82c547952d Merge pull request #1949 from ceph/wip-7774-3
Wip 7774 3

Reviewed-by: Josh Durgin <josh.durgin@inktank.com>
2014-06-13 18:24:19 -07:00
Josh Durgin
f7c093e974 Merge pull request #1915 from ceph/wip-8452-2
rgw: set meta object in extra flag when initializing it

Reviewed-by: Josh Durgin <josh.durgin@inktank.com>
2014-06-13 16:30:37 -07:00
Loic Dachary
4786a485df osd: remove non const get_erasure_code_profile
It is not used and will unexpectedly create a new entry if selected when
the const version is intended.

Signed-off-by: Loic Dachary <loic@dachary.org>
2014-06-13 14:59:51 +02:00
Loic Dachary
a1c13c57ba tests: prevent kill race condition
When trying to kill a daemon, keep its pid in a variable instead of
retrieving it from the pidfile multiple times. It prevents the following
race condition:

  * try to kill ceph-mon
  * ceph-mon is in the process of dying and removed its pidfile
  * try to kill ceph-mon fails because the pidfile is not found
  * another ceph-mon is spawned and fails to bind the port
    because the previous ceph-mon is still holding it

Signed-off-by: Loic Dachary <loic@dachary.org>
2014-06-13 14:46:42 +02:00
Loic Dachary
5c1f9aaec1 osd: improve osd pool create error message readability
Add std::endl to separate independent error messages related to erasure
code.

Signed-off-by: Loic Dachary <loic@dachary.org>
2014-06-13 12:56:14 +02:00
Loic Dachary
6bf8183fd1 erasure-code: consistent argument parsing for profiles
Remove the = from the goodchars of the erasure_code_profile argument of
osd pool create so that it is consistent with the goodchars of osd
erasure-code-profile set / rm.

Signed-off-by: Loic Dachary <loic@dachary.org>
2014-06-13 12:56:14 +02:00