mirror of
https://github.com/ceph/ceph
synced 2024-12-16 08:26:25 +00:00
b0e42d8bd4
Signed-off-by: Sage Weil <sage@inktank.com>
190 lines
7.4 KiB
Plaintext
190 lines
7.4 KiB
Plaintext
commit a913ded2ff138aefb8cb84d347d72164099cfd60
|
|
Author: Gary Lowell <gary.lowell@inktank.com>
|
|
Date: Fri Dec 20 19:28:37 2013 +0000
|
|
|
|
v0.72.2
|
|
|
|
commit b570c8014325fef590d16e6157cb33026155932c
|
|
Author: Yehuda Sadeh <yehuda@inktank.com>
|
|
Date: Wed Dec 18 13:11:01 2013 -0800
|
|
|
|
rgw: fix use-after-free when releasing completion handle
|
|
|
|
Backport: emperor, dumpling
|
|
Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
|
|
Reviewed-by: Sage Weil <sage@inktank.com>
|
|
(cherry picked from commit c8890ab2d46fe8e12200a0d2f9eab31c461fb871)
|
|
|
|
commit 451381b4dba9d50fbee2accf5d75562615001ef3
|
|
Author: Yehuda Sadeh <yehuda@inktank.com>
|
|
Date: Wed Dec 18 13:10:21 2013 -0800
|
|
|
|
rgw: don't return data within the librados cb
|
|
|
|
Fixes: #7030
|
|
The callback is running within a single Finisher thread, thus we
|
|
shouldn't block there. Append read data to a list and flush it within
|
|
the iterate context.
|
|
|
|
Reviewed-by: Sage Weil <sage@inktank.com>
|
|
Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
|
|
(cherry picked from commit d6a4f6adfaa75c3140d07d6df7be03586cc16183)
|
|
|
|
commit c044b218cb32d7ef2f2aa5db81ebf38ad0dc1049
|
|
Author: Sage Weil <sage@inktank.com>
|
|
Date: Sun Dec 1 22:21:31 2013 -0800
|
|
|
|
Partial revert "mon: osd pool set syntax relaxed, modify unit tests"
|
|
|
|
This reverts commit 08327fed8213a5d24cd642e12b38a171b98924cb, except
|
|
for the hashpspool bit. We switched back to an integer argument in
|
|
commit 337195f04653eed8e8f153a5b074f3bd48408998.
|
|
|
|
Signed-off-by: Sage Weil <sage@inktank.com>
|
|
(cherry picked from commit e80ab94bf44e102fcd87d16dc11e38ca4c0eeadb)
|
|
Reviewed-by: Greg Farnum <greg@inktank.com>
|
|
|
|
commit 1983ed8013637e9b80ca4a0e8d2800d6a36b120e
|
|
Author: Joao Eduardo Luis <joao.luis@inktank.com>
|
|
Date: Tue Nov 19 23:21:11 2013 +0000
|
|
|
|
mon: OSDMonitor: drop cmdval_get() for unused variable
|
|
|
|
We don't ever use any value as a float, so just drop obtaining it. This
|
|
makes it easier to partially revert 2fe0d0d9 in an upcoming patch.
|
|
|
|
Backport: emperor
|
|
|
|
Signed-off-by: Joao Eduardo Luis <joao.luis@inktank.com>
|
|
(cherry picked from commit 7191bb2b2485c7819ca7b9d9434d803d0c94db7a)
|
|
Reviewed-by: Sage Weil <sage@inktank.com>
|
|
Reviewed-by: Greg Farnum <greg@inktank.com>
|
|
|
|
commit 1c4759cebb11ae1aa84aa3cbdb2b31e6d111f28e
|
|
Author: Joao Eduardo Luis <joao.luis@inktank.com>
|
|
Date: Fri Nov 22 02:10:35 2013 +0000
|
|
|
|
mon: OSDMonitor: receive CephInt on 'osd pool set' instead on CephString
|
|
|
|
This partially reverts 2fe0d0d9 in order to allow Emperor monitors to
|
|
forward mon command messages to Dumpling monitors without breaking a
|
|
cluster.
|
|
|
|
The need for this patch became obvious after issue #6796 was triggered.
|
|
Basically, in a mixed cluster of Emperor/Dumpling monitors, if a client
|
|
happens to obtain the command descriptions from an Emperor monitor and
|
|
then issue an 'osd pool set' this can turn out in one of two ways:
|
|
|
|
1. client msg gets forwarded to an Emperor leader and everything's a-okay;
|
|
2. client msg gets forwarded to a Dumpling leader and the string fails to
|
|
be interpreted without the monitor noticing, thus leaving the monitor with
|
|
an uninitialized variable leading to trouble.
|
|
|
|
If 2 is triggered, a multitude of bad things can happen, such as thousands
|
|
of pg splits, due to a simple 'osd set pool foo pg_num 128' turning out
|
|
to be interpreted as 109120394 or some other random number.
|
|
|
|
This patch is such that we make sure the client sends an integer instead
|
|
of a string. We also make sure to interpret anything the client sends as
|
|
possibly being a string, or an integer.
|
|
|
|
Fixes: 6796
|
|
Backport: emperor
|
|
|
|
Signed-off-by: Joao Eduardo Luis <joao.luis@inktank.com>
|
|
(cherry picked from commit 337195f04653eed8e8f153a5b074f3bd48408998)
|
|
Reviewed-by: Sage Weil <sage@inktank.com>
|
|
Reviewed-by: Greg Farnum <greg@inktank.com>
|
|
|
|
commit c2e467b42dfaecf082a3838b10e873f9f8bd5633
|
|
Author: Josh Durgin <josh.durgin@inktank.com>
|
|
Date: Mon Nov 25 13:43:43 2013 -0800
|
|
|
|
init, upstart: prevent daemons being started by both
|
|
|
|
There can be only one init system starting a daemon. If there is a
|
|
host entry in ceph.conf for a daemon, sysvinit would try to start it
|
|
even if the daemon's directory did not include a sysvinit file. This
|
|
preserves backwards compatibility with older installs using sysvinit,
|
|
but if an upstart file is present in the daemon's directory, upstart
|
|
will try to start them, regardless of host entries in ceph.conf.
|
|
|
|
If there's an upstart file in a daemon's directory and a host entry
|
|
for that daemon in ceph.conf, both sysvinit and upstart would attempt
|
|
to manage it.
|
|
|
|
Fix this by only starting daemons if the marker file for the other
|
|
init system is not present. This maintains backwards compatibility
|
|
with older installs using neither sysvinit or upstart marker files,
|
|
and does not break any valid configurations. The only configuration
|
|
that would break is one with both sysvinit and upstart files present
|
|
for the same daemon.
|
|
|
|
Backport: emperor, dumpling
|
|
Reported-by: Tim Spriggs <tims@uahirise.org>
|
|
Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
|
|
(cherry picked from commit 5e34beb61b3f5a1ed4afd8ee2fe976de40f95ace)
|
|
|
|
commit 8812c4e958b154ca401c7257fd48ccaffe013639
|
|
Author: Yehuda Sadeh <yehuda@inktank.com>
|
|
Date: Wed Nov 27 13:34:00 2013 -0800
|
|
|
|
rgw: don't error out on empty owner when setting acls
|
|
|
|
Fixes: #6892
|
|
Backport: dumpling, emperor
|
|
s3cmd specifies empty owner field when trying to set acls on object
|
|
/ bucket. We errored out as it didn't match the current owner name, but
|
|
with this change we ignore it.
|
|
|
|
Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
|
|
(cherry picked from commit 14cf4caff58cc2c535101d48c53afd54d8632104)
|
|
|
|
commit 69e055f367ae37dd771c050f158d684e2a3cb246
|
|
Author: Yehuda Sadeh <yehuda@inktank.com>
|
|
Date: Fri Nov 22 07:04:01 2013 -0800
|
|
|
|
rgw: lower some debug message
|
|
|
|
Fixes: #6084
|
|
Backport: dumpling, emperor
|
|
|
|
Reported-by: Ron Allred <rallred@itrefined.com>
|
|
Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
|
|
(cherry picked from commit b35fc1bc2ec8c8376ec173eb1c3e538e02c1694e)
|
|
|
|
commit 53951fd5459d6396b6ab59b52dc33df0ebcf4414
|
|
Author: Samuel Just <sam.just@inktank.com>
|
|
Date: Tue Nov 12 15:15:26 2013 -0800
|
|
|
|
ReplicatedPG: test for missing head before find_object_context
|
|
|
|
find_object_context doesn't return EAGAIN for a missing head.
|
|
I chose not to change that behavior since it might hide bugs
|
|
in the future. All other callers check for missing on head
|
|
before calling into find_object_context because we potentially
|
|
need head or snapdir to map a snapid onto a clone.
|
|
|
|
Backport: emperor
|
|
Fixes: 6758
|
|
Signed-off-by: Samuel Just <sam.just@inktank.com>
|
|
Reviewed-by: Josh Durgin <josh.durgin@inktank.com>
|
|
Reviewed-by: David Zafman <david.zafman@inktank.com>
|
|
(cherry picked from commit dd9d8b020286d5e3a69455023c3724a7b436d687)
|
|
|
|
commit 83ee6843de6432278dc2891526691006c51eb1fa
|
|
Author: Josh Durgin <josh.durgin@inktank.com>
|
|
Date: Mon Nov 18 14:39:12 2013 -0800
|
|
|
|
osd: fix bench block size
|
|
|
|
The command was declared to take 'size' in dumpling, but was trying to
|
|
read 'bsize' instead, so it always used the default of 4MiB. Change
|
|
the bench command to read 'size', so it matches what existing clients
|
|
are sending.
|
|
|
|
Fixes: #6795
|
|
Backport: emperor, dumpling
|
|
Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
|
|
(cherry picked from commit 40a76ef0d09f8ecbea13712410d9d34f25b91935)
|