ceph/doc/changelog/v0.67.6.txt

1259 lines
47 KiB
Plaintext
Raw Normal View History

commit c71f8a60cf49fb3d4efb5ea75b2366cd7f167497 (tag: refs/tags/v0.67.6)
Author: Alfredo Deza <alfredo.deza@inktank.com>
Date: Thu Feb 13 20:20:42 2014 +0000
release build 67.6
Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>
commit 1c42a6e34bb8e318e001384270cda003add946a7
Merge: 3e473d4 c5e12f9
Author: Sage Weil <sage@inktank.com>
Date: Thu Feb 13 06:55:37 2014 -0800
Merge pull request #1232 from ceph/dumpling-7334
backport ceph-disk improvements to dumpling
http://pulpito.ceph.com/ubuntu-2014-02-12_16:52:33-ceph-deploy-dumpling-7334-testing-basic-plana/
commit 3e473d4237697872498094b41fed19b414f8d3c4
Author: David Zafman <david.zafman@inktank.com>
Date: Fri Sep 27 16:23:09 2013 -0700
common,os: Remove filestore_xattr_use_omap option
Now we operate just like when this was set to true
Fixes: #6143
Signed-off-by: David Zafman <david.zafman@inktank.com>
(cherry picked from commit dc0dfb9e01d593afdd430ca776cf4da2c2240a20)
commit c5e12f94efc07f0ff174c0f19494ee2e06d88dfa
Author: Alfredo Deza <alfredo@deza.pe>
Date: Wed Feb 12 16:43:59 2014 -0500
add support for absence of PATH
Note that this commit is actually bisecting the changes from
Loic Dachary that touch ceph-disk only (ad515bf). As that changeset
also touches other files it causes conflicts that are not resolvable
for backporting it to dumpling.
Signed-off-by: Alfredo Deza <alfredo@deza.pe>
commit c531d75f7e86cee619e44c247298b50c37b81539
Author: Sage Weil <sage@inktank.com>
Date: Mon Sep 9 22:27:23 2013 -0700
ceph-disk: make initial journal files 0 bytes
The ceph-osd will resize journal files up and properly fallocate() them
so that the blocks are preallocated and (hopefully) contiguous. We
don't need to do it here too, and getting fallocate() to work from
python is a pain in the butt.
Fixes: #5981
Signed-off-by: Sage Weil <sage@inktank.com>
(cherry picked from commit a786ad773cd33880075f1deb3691528d1afd03ec)
commit 836ead3e42491597c192c63d3171021c5daecbc0
Author: Alfredo Deza <alfredo@deza.pe>
Date: Mon Feb 10 15:07:55 2014 -0500
alert the user about error messages from partx
Signed-off-by: Alfredo Deza <alfredo@deza.pe>
(cherry picked from commit 9bcc42a3e6b08521694b5c0228b2c6ed7b3d312e)
commit cff3547c60dc6d164f9a95eaa2706633e32756d0
Author: Alfredo Deza <alfredo@deza.pe>
Date: Fri Feb 7 11:55:01 2014 -0500
use partx for red hat or centos instead of partprobe
Signed-off-by: Alfredo Deza <alfredo@deza.pe>
(cherry picked from commit 42900ff9da9f5adcac239a84ebf4d2e407c29699)
commit 5ebd7c4520a9805f985cab1a0ba9311e19efa041
Author: Josh Durgin <josh.durgin@inktank.com>
Date: Tue Jan 28 17:26:58 2014 -0800
ceph-disk: run the right executables from udev
When run by the udev rules, PATH is not defined. Thus,
ceph-disk-activate relies on its which() function to locate the
correct executable. The which() function used os.defpath if none was
set, and this worked for anything using it.
ad6b4b4b08b6ef7ae8086f2be3a9ef521adaa88c added a new default value to
PATH, so only /usr/bin was checked by callers that did not use
which(). This resulted in the mount command not being found when
ceph-disk-activate was run by udev, and thus osds failing to start
after being prepared by ceph-deploy.
Make ceph-disk consistently use the existing helpers (command() and
command_check_call()) that use which(), so lack of PATH does not
matter. Simplify _check_output() to use command(),
another wrapper around subprocess.Popen.
Fixes: #7258
Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
(cherry picked from commit d7b0c7faafd37e4ae8a1680edfa60c22b419cbd8)
commit 4a7c4e0a81753dfb811396e7bfa1df72135ee12b
Author: Loic Dachary <loic@dachary.org>
Date: Fri Jan 3 16:06:55 2014 +0100
ceph-disk: implement --sysconfdir as /etc/ceph
Replace hardcoded /etc/ceph with the SYSCONFDIR global variable and
implement the --sysconfdir option to override the default value.
Signed-off-by: Loic Dachary <loic@dachary.org>
(cherry picked from commit a71025d33621257b6fd6632516cfed2849ff1637)
commit c8b9798a69bd1655a7131c7bce8a14c2c74aba0c
Author: Loic Dachary <loic@dachary.org>
Date: Fri Jan 3 16:03:09 2014 +0100
ceph-disk: implement --statedir as /var/lib/ceph
Replace hardcoded /var/lib/ceph with the STATEDIR global variable and
implement the --statedir option to override the default value.
Signed-off-by: Loic Dachary <loic@dachary.org>
(cherry picked from commit ca713f48ae7a1fece2869f1a1c97d23ab33fb441)
commit 79564874be35d4414092fbb80df3214d7f282b1c
Author: Loic Dachary <loic@dachary.org>
Date: Wed Jan 1 22:05:03 2014 +0100
ceph-disk: add copyright notice
Signed-off-by: Loic Dachary <loic@dachary.org>
(cherry picked from commit 51ee3c04777aaf6b9609dde9bc318b5c66c70787)
commit c4b66bbdf09a8bb85ed14fbb643f40f87345463b
Author: Loic Dachary <loic@dachary.org>
Date: Wed Jan 1 22:11:30 2014 +0100
ceph-disk: create the data directory if it does not exist
Instead of failing if the OSD data directory does not exist, create
it. Only do so if the data directory is not enforced to be a device via
the use of the --data-dev flag. The directory is not recursively created.
Signed-off-by: Loic Dachary <loic@dachary.org>
(cherry picked from commit 306b099ab093bfac466d68fe1cb87367bc01e577)
commit c473c65d4b6235fe20a97bd1d4741c12f86858e8
Author: Loic Dachary <loic@dachary.org>
Date: Tue Dec 31 00:03:42 2013 +0100
ceph-disk: run ceph-osd when --mark-init=none
Signed-off-by: Loic Dachary <loic@dachary.org>
(cherry picked from commit 0fcc081858fae4febbb6a613a93cfbbcedd5a320)
commit 6ea59a3a0c38c9629036442aff652489eecce51a
Author: Loic Dachary <loic@dachary.org>
Date: Mon Dec 30 23:57:39 2013 +0100
ceph-disk: implement --mark-init=none
It is meant to be used when preparing and activating a directory that is
not to be used with init. No file is created to identify the init
system, no symbolic link is made to the directory in /var/lib/ceph
and the init scripts are not called.
Signed-off-by: Loic Dachary <loic@dachary.org>
(cherry picked from commit e773b68f4c89ac56b425c710d7dcdc3d74a92926)
commit 3aae5b764c5698c665a708a3f6f2243dfee1a28d
Author: Loic Dachary <loic@dachary.org>
Date: Wed Jan 1 22:07:57 2014 +0100
ceph-disk: fsid is a known configuration option
Use get_conf_with_default instead of get_conf because fsid is a known
ceph configuration option. It allows overriding via CEPH_ARGS which is
convenient for testing. Only options that are not found in config_opts.h
are fetch via get_conf.
Signed-off-by: Loic Dachary <loic@dachary.org>
(cherry picked from commit b65eb377f5e93ea85644e4c0939365fd7ac36072)
commit a33ad6d150206232806ee466f301e41f15c631e9
Author: Loic Dachary <loic@dachary.org>
Date: Wed Jan 1 22:06:29 2014 +0100
ceph-disk: use CalledProcessError.returncode
CalledProcessError has no errno data member
Signed-off-by: Loic Dachary <loic@dachary.org>
(cherry picked from commit 97f516a1ddfb2d014e1f7e762c4155e4b9bcb90b)
commit 6ae96d3ce757b05daeb7c1437778149b07248d47
Author: Loic Dachary <loic@dachary.org>
Date: Wed Jan 1 22:13:15 2014 +0100
ceph-disk: display the command output on OSD creation failure
The string form of a CalledProcessError instance does not include the
output datamember. Add it to the Error exception for debug purposes.
Signed-off-by: Loic Dachary <loic@dachary.org>
(cherry picked from commit d09af0fa50f322c9e59765f3badd497f5ca184d4)
commit deefb3a05788b308631812c652e9eccaeb999d7f
Author: Loic Dachary <loic@dachary.org>
Date: Mon Dec 30 23:07:27 2013 +0100
ceph-disk: which() uses PATH first
Instead of relying on a hardcoded set of if paths. Although this has the
potential of changing the location of the binary being used by ceph-disk
on an existing installation, it is currently only used for sgdisk. It
could be disruptive for someone using a modified version of sgdisk but
the odds of this happening are very low.
Signed-off-by: Loic Dachary <loic@dachary.org>
(cherry picked from commit 2b935bbf60bafb6dd488c0eb30f156fce1b9d197)
commit 06bbea187293ac52aefbcf4e98c162bb3286341d
Author: Loic Dachary <loic@dachary.org>
Date: Mon Dec 30 22:48:46 2013 +0100
ceph-disk: add --prepend-to-path to control execution
/usr/bin is hardcoded in front of some ceph programs which makes it
impossible to control where they are located via the PATH.
The hardcoded path cannot be removed altogether because it will most
likely lead to unexpected and difficult to diagnose problems for
existing installations where the PATH finds the program elsewhere.
The --prepend-to-path flag is added and defaults to /usr/bin : it prepends
to the PATH environment variable. The hardcoded path is removed
and the PATH will be used: since /usr/bin is searched first, the
legacy behavior will not change.
Signed-off-by: Loic Dachary <loic@dachary.org>
(cherry picked from commit ad6b4b4b08b6ef7ae8086f2be3a9ef521adaa88c)
commit ccf182241dc04c79e2377b6ba2f060dbfe6dddae
Author: Loic Dachary <loic@dachary.org>
Date: Mon Dec 30 17:15:29 2013 +0100
ceph-disk: make exception handling terse in main_activate_journal
Signed-off-by: Loic Dachary <loic@dachary.org>
(cherry picked from commit 908348b8047e8577ecf9133f2683f91423694416)
commit d60e952f3c00d0e551b04c0d6486b11d9761be35
Author: Loic Dachary <loic@dachary.org>
Date: Mon Dec 30 17:13:47 2013 +0100
ceph-disk: do not hide main_activate() exceptions
Signed-off-by: Loic Dachary <loic@dachary.org>
(cherry picked from commit 55ca7bb2da73f1be1293710a635cfea42abd7682)
commit 827edb93e3971fc7549fe831894776c6bf163346
Author: Loic Dachary <loic@dachary.org>
Date: Mon Dec 30 17:11:30 2013 +0100
ceph-disk: fix activate() indent
Signed-off-by: Loic Dachary <loic@dachary.org>
(cherry picked from commit 324804a81c37ff89f2488e2ba106033c0e6e119e)
commit 27988cf9ffedf3289099e51f1e7c3f5b8d79d667
Author: Loic Dachary <loic@dachary.org>
Date: Mon Dec 30 17:09:35 2013 +0100
ceph-disk: remove noop try:
Signed-off-by: Loic Dachary <loic@dachary.org>
(cherry picked from commit de0050596b5f56863c3486c1cd5e7ffea62e3d00)
commit 75bdb5430d9403bed8bd2ccd6a9c5a30d63a189e
Author: Loic Dachary <loic@dachary.org>
Date: Mon Dec 30 16:57:36 2013 +0100
ceph-disk: fix Error() messages formatting
Mainly using % instead of ,
Signed-off-by: Loic Dachary <loic@dachary.org>
(cherry picked from commit b82ccfbfa786cd5436b48ec38276c5a48028ce1d)
commit 5df1eaa55bc7ebe68473831d06cf9819fa39baa6
Author: Loic Dachary <loic@dachary.org>
Date: Mon Dec 30 12:26:20 2013 +0100
ceph-disk: prepare --data-dir must not override files
ceph-disk does nothing when given a device that is already prepared. If
given a directory that already contains a successfully prepared OSD, it
will however override it.
Instead of overriding the files in the osd data directory, return
immediately if the magic file exists. Make it so the magic file is
created last to accurately reflect the success of the OSD preparation.
Signed-off-by: Loic Dachary <loic@dachary.org>
(cherry picked from commit 7dfe550ce18623cde4ae43a2416e31ef81381ab9)
commit 2e8496a6e0b74ff351bc2a4f138ddc665d8c30ca
Author: Loic Dachary <loic@dachary.org>
Date: Sun Dec 15 15:34:17 2013 +0100
ceph-disk: zap needs at least one device
If given no argument, ceph-disk zap should display the usage instead of
silently doing nothing. Silence can be confused with "I zapped all the
disks".
http://tracker.ceph.com/issues/6981 fixes #6981
Signed-off-by: Loic Dachary <loic@dachary.org>
(cherry picked from commit 07888ef3fd4440332c8287d0faa9f23a32cf141c)
commit 0fe2ac47a82d3caa9dca2a9009b68a14b577e35c
Author: Alfredo Deza <alfredo@deza.pe>
Date: Fri Dec 13 12:06:25 2013 -0500
use the new get_command helper in check_call
Signed-off-by: Alfredo Deza <alfredo@deza.pe>
(cherry picked from commit 897dfc113fe3b86f3dda53172933bfd4f8089869)
commit 8fc9f1b791248f08a311d619addc4465cac298f9
Author: Alfredo Deza <alfredo@deza.pe>
Date: Thu Dec 12 11:16:38 2013 -0500
use the absolute path for executables if found
Signed-off-by: Alfredo Deza <alfredo@deza.pe>
(cherry picked from commit a9334a1c8c6681305e76b361377864d0dd1e3d34)
commit a2de2f718c9e341c0b6ec2ebffdf03d0f9da0fbd
Author: Alfredo Deza <alfredo@deza.pe>
Date: Thu Dec 12 10:26:05 2013 -0500
remove trailing semicolon
Signed-off-by: Alfredo Deza <alfredo@deza.pe>
(cherry picked from commit 43561f791607f5fd6f03d5421e1f30a29fb4194e)
commit 2cd22b5d39760ac4b5baad0b05c676b833ae9213
Author: Alfredo Deza <alfredo@deza.pe>
Date: Wed Dec 11 15:41:45 2013 -0500
replace sgdisk subprocess calls with a helper
Signed-off-by: Alfredo Deza <alfredo@deza.pe>
(cherry picked from commit e19e38012bc4579054f63865e682c8c3a7829c7b)
commit e8cc27cc8eb981c71b8cafb58964317687ca5787
Author: Jonathan Davies <jonathan.davies@canonical.com>
Date: Tue Dec 3 21:26:43 2013 +0000
Call --mbrtogpt on journal run of sgdisk should the drive require a GPT table.
Signed-off-by: Jonathan Davies <jonathan.davies@canonical.com>
Reviewed-by: Sage Weil <sage@inktank.com>
Reviewed-by: Loic Dachary <loic@dachary.org>
(cherry picked from commit 35011e0b01d65e4c001876882d597451f2028345)
commit 3942a93fae3331b2d08f93181eb2c9cf90b50440
Author: Loic Dachary <loic@dachary.org>
Date: Thu Nov 21 13:21:22 2013 +0100
ceph-disk: blacklist /dev/fd0
blkid -s TYPE /dev/fd0 has been verified to hang forever on a
H8DMR-82 supermicro motherboard running
3.8.0-33-generic #48~precise1-Ubuntu SMP Thu Oct 24 16:28:06 UTC 2013
x86_64
It is unlikely that ceph will ever be used on floppy disks, they
can be blacklisted.
http://tracker.ceph.com/issues/6827 fixes: #6827
Signed-off-by: Loic Dachary <loic@dachary.org>
(cherry picked from commit 65701978715237ff5a4c68573c0696fd9d438e4f)
commit ccc6b0c5e6d47eefef5cf1fa17bf69c47c9459c4
Author: git-harry <git-harry@live.co.uk>
Date: Thu Oct 3 10:32:50 2013 +0100
Make fsid comparison case-insensitive
get_fsid and find_cluster_by_uuid are modified so ceph-disk activate and
ceph-disk activate-all will work if the fsid uses uppercase characters.
Signed-off-by: Harry Harrington <git-harry@live.co.uk>
(cherry picked from commit 22f8325dbfce7ef2e97bf015c0f8bba53e75dfe9)
commit 367cf1bbf86233eb20ff2304e7d6caab77b84fcc
Author: Josh Durgin <josh.durgin@inktank.com>
Date: Tue Feb 11 10:14:36 2014 -0800
librbd: remove limit on number of objects in the cache
The number of objects is not a significant indicated of when data
should be written out for rbd. Use the highest possible value for
number of objects and just rely on the dirty data limits to trigger
flushing. When the number of objects is low, and many start being
flushed before they accumulate many requests, it hurts average request
size and performance for many concurrent sequential writes.
Fixes: #7385
Backport: emperor, dumpling
Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
(cherry picked from commit 0559d31db29ea83bdb6cec72b830d16b44e3cd35)
commit 4856fc2950442fafc208c967af5131f7874f5441
Author: Josh Durgin <josh.durgin@inktank.com>
Date: Tue Feb 11 11:53:00 2014 -0800
ObjectCacher: use uint64_t for target and max values
All the options are uint64_t, but the ObjectCacher was converting them
to int64_t. There's never any reason for these to be negative, so
change the type.
Adjust a few conditionals so that they only convert known-positive
signed values to uint64_t before comparing with the target and max
values. Leave the actual stats accounting as loff_t for now, since
bugs in accounting will have bad effects if negative values wrap
around.
Backport: emperor, dumpling
Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
(cherry picked from commit db034acf546a72739ff6543241543f3bd651f3ae)
commit a572aafe178dfa3ab4d82aa5c7f4a72a63590ccf
Author: Josh Durgin <josh.durgin@inktank.com>
Date: Tue Feb 11 10:35:14 2014 -0800
ObjectCacher: remove max_bytes and max_ob arguments to trim()
These are never passed, so replace them with the defaults.
Backport: emperor, dumpling
Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
(cherry picked from commit bf8cf2d6d21a204a099347f3dcd5b48100b8c445)
commit 4889234f80dbd95e5b2012ff9bfc2d87960034e2
Merge: 6af657b 30a604d
Author: Sage Weil <sage@inktank.com>
Date: Tue Feb 11 08:32:37 2014 -0800
Merge pull request #1210 from dachary/dumpling
common: admin socket fallback to json-pretty format (dumpling)
Reviewed-by: Sage Weil <sage@inktank.com>
commit 30a604d2d6a5de7bcbfa0136e9a766cb34f96c0a
Author: Loic Dachary <loic@dachary.org>
Date: Mon Feb 10 23:42:38 2014 +0100
common: admin socket fallback to json-pretty format
If the format argument to a command sent to the admin socket is not
among the supported formats ( json, json-pretty, xml, xml-pretty ) the
new_formatter function will return null and the AdminSocketHook::call
function must fall back to a sensible default.
The CephContextHook::call and HelpHook::call failed to do that and a
malformed format argument would cause the mon to crash. A check is added
to each of them and fallback to json-pretty if the format is not
recognized.
To further protect AdminSocketHook::call implementations from similar
problems the format argument is checked immediately after accepting the
command in AdminSocket::do_accept and replaced with json-pretty if it is
not known.
A test case is added for both CephContextHook::call and HelpHook::call
to demonstrate the problem exists and is fixed by the patch.
Three other instances of unsafe calls to new_formatter were found and
a fallback to json-pretty was added. All other calls have been audited
and appear to be safe.
http://tracker.ceph.com/issues/7378 fixes #7378
Signed-off-by: Loic Dachary <loic@dachary.org>
(cherry picked from commit 165e76d4d03ffcc490fd3c2ba60fb37372990d0a)
commit 6af657bd4cf87f6f7afe84257941e7e2366183d9
Author: Josh Durgin <josh.durgin@inktank.com>
Date: Wed Feb 5 17:26:02 2014 -0800
qa: add script for testing rados client timeout options
Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
(cherry picked from commit 9e62beb80b6c92a97ec36c0db5ea39e417661b35)
commit 21a0b67440eaa07fa051d26321a88654ee9e8ae7
Author: Josh Durgin <josh.durgin@inktank.com>
Date: Wed Feb 5 17:25:24 2014 -0800
rados: check return values for commands that can now fail
A few places were not checking the return values of commands, since
they could not fail before timeouts were added.
Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
(cherry picked from commit 79c1874346ff55e2dc74ef860db16ce70242fd00)
commit 2c42dd26b49d01922bd3df3ba26af70da0387a99
Author: Josh Durgin <josh.durgin@inktank.com>
Date: Wed Feb 5 17:24:16 2014 -0800
librados: check and return on error so timeouts work
Some functions could not previously return errors, but they had an
int return value, which can now receive ETIMEDOUT.
Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
(cherry picked from commit 8e9459e897b1bc2f97d52ee07701fd22069efcf3)
commit 28fe564cc298b96c063e6cca44304046a86942cd
Author: Josh Durgin <josh.durgin@inktank.com>
Date: Wed Feb 5 17:22:14 2014 -0800
msg/Pipe: add option to restrict delay injection to specific msg type
This makes it possible to test timeouts reliably by delaying certain
messages effectively forever, but still being able to e.g. connect and
authenticate to the monitors.
Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
(cherry picked from commit d389e617c1019e44848330bf9570138ac7b0e5d4)
commit 1d2ea9efb844cb902d282c3a752acc45fefeb04c
Author: Josh Durgin <josh.durgin@inktank.com>
Date: Mon Feb 3 18:30:00 2014 -0800
MonClient: add a timeout on commands for librados
Just use the conf option directly, since librados is the only caller.
Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
(cherry picked from commit 671a76d64bc50e4f15f4c2804d99887e22dcdb69)
commit 30dafacd0b54bb98b01284851e0d5abf76324e95
Author: Josh Durgin <josh.durgin@inktank.com>
Date: Mon Feb 3 17:59:21 2014 -0800
Objecter: implement mon and osd operation timeouts
This captures almost all operations from librados other than mon_commands().
Get the values for the timeouts from the Objecter constructor, so only
librados uses them.
Add C_Cancel_*_Op, finish_*_op(), and *_op_cancel() for each type of
operation, to mirror those for Op. Create a callback and schedule it
in the existing timer thread if the timeouts are specified.
Fixes: #6507
Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
(cherry picked from commit 3e1f7bbb4217d322f4e0ece16e676cd30ee42a20)
Conflicts:
src/osd/OSD.cc
src/osd/ReplicatedPG.cc
src/osdc/Objecter.cc
src/osdc/Objecter.h
commit 2aa0d7589283fe5879a966601fbfe690816cc4a1
Author: Josh Durgin <josh.durgin@inktank.com>
Date: Mon Feb 3 12:53:15 2014 -0800
librados: add timeout to wait_for_osdmap()
This is used by several pool operations independent of the objecter,
including rados_ioctx_create() to look up the pool id in the first
osdmap.
Unfortunately we can't just rely on WaitInterval returning ETIMEDOUT,
since it may also get interrupted by a signal, so we can't avoid
keeping track of time explicitly here.
Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
(cherry picked from commit 1829d2c9fd13f2cbae4e192c9feb553047dad42c)
Conflicts:
src/librados/RadosClient.cc
commit 55d254d5ea113d8be29afe18dc9940f14d93ebb4
Author: Josh Durgin <josh.durgin@inktank.com>
Date: Mon Feb 3 12:09:34 2014 -0800
conf: add options for librados timeouts
These will be implemented in subsequent patches.
Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
(cherry picked from commit 0dcceff1378d85ca6d81d102d201890b8a71af6b)
commit ccd9b5eb9bcfe2360e874b306d4decd8815ecf46
Author: Sage Weil <sage@inktank.com>
Date: Sun Feb 9 20:53:00 2014 -0800
test_striper: fix warning
Signed-off-by: Sage Weil <sage@inktank.com>
commit bb44c0d5e06c2b699c81a76373a0edde1bf90445
Author: Sage Weil <sage@inktank.com>
Date: Sat Feb 8 12:21:26 2014 -0800
crushtool: add cli test for off-by-one tries vs retries bug
See bug #7370. This passes on dumpling and breaks prior to the #7370 fix.
Backport: emperor, dumpling
Signed-off-by: Sage Weil <sage@inktank.com>
(cherry picked from commit ed32c4002fb5cb1dd546331651eaf7de1a017471)
commit c301ec689caac5bbcd21e02226045d3454cc82fd
Author: Sage Weil <sage@inktank.com>
Date: Mon Feb 3 08:54:14 2014 -0800
client: use 64-bit value in sync read eof logic
The file size can jump to a value that is very much larger than our current
position (for example, it could be a disk image file that gets a sparse
write at a large offset). Use a 64-bit value so that 'some' doesn't
overflow.
Signed-off-by: Sage Weil <sage@inktank.com>
Reviewed-by: John Spray <john.spray@inktank.com>
(cherry picked from commit 7ff2b541c24d1c81c3bcfbcb347694c2097993d7)
commit 183deb899bc6b1b7b2a1ec639425e45786e56b01
Author: Sage Weil <sage@inktank.com>
Date: Sat Jan 4 22:39:35 2014 -0800
osd: do not send peering messages during init
Do not send any peering messages while we are still working our way
through init().
Fixes: #7093
Signed-off-by: Sage Weil <sage@inktank.com>
(cherry picked from commit 35da8f9d80e0c6c33fb6c6e00f0bf38f1eb87d0e)
Signed-off-by: Greg Farnum <greg@inktank.com>
commit 3a1fb17401897b0717cf1d264c6e6fdf99fc703f
Author: Sage Weil <sage@inktank.com>
Date: Tue Jan 28 18:46:37 2014 -0800
OSDMap: fix deepish_copy_from
Start with a shallow copy!
Signed-off-by: Sage Weil <sage@inktank.com>
(cherry picked from commit d0f13f54146694a197535795da15b8832ef4b56f)
Conflicts:
src/osd/OSDMap.h
commit f01bd8a94adc4ce8e7131f74282de5b434a7316a
Author: Yehuda Sadeh <yehuda@inktank.com>
Date: Thu Jan 23 13:48:28 2014 -0800
rgw: fix listing of multipart upload parts
Fixes: #7169
There are two issues here. One is that we may return more entries than
we should (as specified by max_parts). Second issue is that the
NextPartNumberMarker is set incorrectly. Both of these issues mainly
affect uploads with > 1000 parts, although can be triggered with less
than that.
Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
commit 6b657992d2397e399eacf554357ed807fda399fa
Author: Yehuda Sadeh <yehuda@inktank.com>
Date: Thu Nov 21 06:52:48 2013 -0800
rgw: initialize RGWUserAdminOpState::system_specified
Fixes: #6829
Backport: dumpling, emperor
We didn't init this member variable, which might cause that when
modifying user info that has this flag set the 'system' flag might
inadvertently reset.
Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
(cherry picked from commit 561e7b0b287e65e90b80699e45a52ae44e94684f)
commit 585e0e7eec1bbee60fe352166b593d53476003f8
Author: Robin H. Johnson <robbat2@gentoo.org>
Date: Sun Dec 15 12:26:19 2013 -0800
rgw: Fix CORS allow-headers validation
This fix is needed because Ceph presently validates CORS headers in a
case-sensitive manner. Keeps a local cache of lowercased allowed headers
to avoid converting the allowed headers to lowercase each time.
CORS 6.2.6: If any of the header field-names is not a ASCII
case-insensitive match for any of the values in list of headers do not
set any additional headers and terminate this set of steps.
Signed-off-by: Robin H. Johnson <robbat2@gentoo.org>
Reviewed-by: Yehuda Sadeh <yehuda@inktank.com>
(cherry picked from commit 31b60bfd9347a386ff12b4e4f1812d664bcfff01)
commit da2267a87e37afa28385e915a566de26fd784b70
Author: Robin H. Johnson <robbat2@gentoo.org>
Date: Sun Dec 15 11:40:31 2013 -0800
rgw: Clarify naming of case-change functions
It is not clear that the lowercase_http_attr & uppercase_http_attr
functions replace dashes with underscores. Rename them to match the
pattern established by the camelcase_dash_http_attr function in
preperation for more case-change functions as needed by later fixes.
Signed-off-by: Robin H. Johnson <robbat2@gentoo.org>
Reviewed-by: Yehuda Sadeh <yehuda@inktank.com>
(cherry picked from commit 6a7edab2673423c53c6a422a10cb65fe07f9b235)
commit fdea76ef6f7d7c4ee89c30d430f5495072023957
Author: Robin H. Johnson <robbat2@gentoo.org>
Date: Sun Dec 15 11:27:49 2013 -0800
rgw: Look at correct header about headers for CORS
The CORS standard dictates that preflight requests are made with the
Access-Control-Request-Headers header containing the headers of the
author request. The Access-Control-Allow-Headers header is sent in the
response.
The present code looks for Access-Control-Allow-Headers in request, so
fix it to look at Access-Control-Request-Headers instead.
Signed-off-by: Robin H. Johnson <robbat2@gentoo.org>
Reviewed-by: Yehuda Sadeh <yehuda@inktank.com>
(cherry picked from commit 2abacd9678ae04cefac457882ba718a454948915)
commit a5739e3903f8c30c8168199413818922b4a29bad
Author: Yehuda Sadeh <yehuda@inktank.com>
Date: Fri Dec 6 11:07:09 2013 -0800
rgw: fix reading bucket policy in RGWBucket::get_policy()
Fixes: 6940
Backport: dumpling, emperor
We changed the way we keep the bucket policy, and we shouldn't try to
access the bucket object directly. This had changed when we added the
bucket instance object around dumpling.
Reported-by: Gao, Wei M <wei.m.gao@intel.com>
Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
(cherry picked from commit 7a9a088d82d04f6105d72f6347673724ac16c9f8)
commit ba792cdc8d0c006e260e6a02a33ca1491b35901c
Author: Yehuda Sadeh <yehuda@inktank.com>
Date: Thu Jan 16 11:45:27 2014 -0800
rgw: handle racing object puts when object doesn't exist
If the object didn't exist before and now we have multiple puts coming
in concurrently, we need to make sure that we behave correctly. Only one
needs to win, the other one can fail silently. We do that by setting
exclusive flag on the object creation and handling the error correctly.
Note that we still want to return -EEXIST in some cases (when the
exclusive flag is passed to put_obj_meta(), e.g., on bucket creation).
Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
(cherry picked from commit 86c15480fc3e33b9a3b84d0af68d8398fc732bae)
commit 1e2984ac4a4ed0508e1b5efc8b69ae4da0cac824
Author: Yehuda Sadeh <yehuda@inktank.com>
Date: Thu Jan 16 11:33:49 2014 -0800
rgw: don't return -ENOENT in put_obj_meta()
Fixes: #7168
An object put may race with the same object's delete. In this case just
ignore the error, same behavior as if object was created and then
removed.
Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
(cherry picked from commit 5c24a7ea905587fd4077e3b0cfc0f5ad2b178c29)
commit 5f1eea2ccf1ee4296530fc754bfd5dfc1571cd62
Author: Robin H. Johnson <robbat2@gentoo.org>
Date: Sat Jan 18 18:01:20 2014 -0800
rgw: Use correct secret key for POST authn
The POST authentication by signature validation looked up a user based
on the access key, then used the first secret key for the user. If the
access key used was not the first access key, then the expected
signature would be wrong, and the POST would be rejected.
Signed-off-by: Robin H. Johnson <robbat2@gentoo.org>
(cherry picked from commit 586ad1f69e6260ef298805647911b3b49f8be7cf)
commit 14e933eb2e8adf3e7df2aef823d863c593067928
Author: Joao Eduardo Luis <joao.luis@inktank.com>
Date: Tue Jan 28 15:54:33 2014 +0000
osd: OSDMonitor: ignore pgtemps from removed pool
There's a window in-between receiving an MOSDPGTemp message from an OSD
and actually handling it that may lead to the pool the pg temps refer to
no longer existing. This may happen if the MOSDPGTemp message is queued
pending dispatching due to an on-going proposal (maybe even the pool
removal).
This patch fixes such behavior in two steps:
1. Check if the pool exists in the osdmap upon preprocessing
- if pool does not exist in the osdmap, then the pool must have been
removed prior to handling the message, but after the osd sent it.
- safe to ignore the pg update
2. If all pg updates in the message have been ignored, ignore the whole
message. Otherwise, let prepare handle the rest.
3. Recheck if pool exists in the osdmap upon prepare
- We may have ignored this pg back in preprocess, but other pgs in the
message may have led the message to be passed on to prepare; ignore
pg update once more.
4. Check if pool is pending removal and ignore pg update if so.
We delegate checking the pending value to prepare_pgtemp() because in this
case we should only ignore the update IFF the pending value is in fact
committed. Otherwise we should retry the message. prepare_pgtemp() is
the appropriate place to do so.
Fixes: 7116
Signed-off-by: Joao Eduardo Luis <joao.luis@inktank.com>
(cherry picked from commit f513f66f48383a07c70ca18a4dba6c2449ea9860)
commit 3672fc49bea5efcef2069673c88fd8e526eeb7e5
Author: Sage Weil <sage@inktank.com>
Date: Tue Jan 28 11:00:34 2014 -0800
OSDMonitor: use deepish_copy_from for remove_down_pg_temp
This is a backport of 368852f6c0a884b8fdc80a5cd6f9ab72e814d412.
Make a deep copy of the OSDMap to avoid clobbering the in-memory copy with
the call to apply_incremental.
Fixes: #7060
Signed-off-by: Sage Weil <sage@inktank.com>
commit e6fef68ee51a2e3be6537086246ce860819788eb
Author: Sage Weil <sage@inktank.com>
Date: Fri Jan 24 11:03:26 2014 -0800
OSDMap: deepish_copy_from()
Make a deep(ish) copy of another OSDMap. Unfortunatley we can't make the
compiler-generated copy operator/constructors private until c++11. :(
Signed-off-by: Sage Weil <sage@inktank.com>
(cherry picked from commit bd54b9841b9255406e56cdc7269bddb419453304)
commit 2528186c0dd327607b1e76a51581df72aeea52f2
Author: Sage Weil <sage@inktank.com>
Date: Tue Jan 28 10:26:12 2014 -0800
buffer: make 0-length splice() a no-op
This was causing a problem in the Striper, but fixing it here will avoid
corner cases all over the tree. Note that we have to bail out before
the end-of-buffer check to avoid hitting that check when the bufferlist is
also empty.
Signed-off-by: Sage Weil <sage@inktank.com>
Reviewed-by: Greg Farnum <greg@inktank.com>
(cherry picked from commit ff5abfbdae07ae8a56fa83ebaa92000896f793c2)
commit fe9539e118a830aa71bc5846ad2693543213fc59
Author: Sage Weil <sage@inktank.com>
Date: Tue Jan 28 10:09:17 2014 -0800
osdc/Striper: test zero-length add_partial_result
If we add a partial result that is 0-length, we used to hit an assert in
buffer::list::splice(). Add a unit test to verify the fix.
Signed-off-by: Sage Weil <sage@inktank.com>
Reviewed-by: Greg Farnum <greg@inktank.com>
(cherry picked from commit 28c7388d320a47657c2e12c46907f1bf40672b08)
commit 9801718e6e7a335d1aeffb7f3d958b012d8dd79c
Author: Derek Yarnell <derek@umiacs.umd.edu>
Date: Mon Jan 27 12:27:51 2014 -0700
packaging: apply udev hack rule to RHEL
In the RPM spec file there is a test to deploy the uuid hack udev rules
for older udev operating systems. This includes CentOS and RHEL, but the
check currently only is for CentOS, causing RHEL clients to get a bogus
osd rules file.
Adjust the conditional to apply to RHEL as well as CentOS. (The %{rhel}
macro is defined in both platforms' redhat-rpm-config package.)
Fixes http://tracker.ceph.com/issues/7245
Signed-off-by: Ken Dreyer <ken.dreyer@inktank.com>
(cherry picked from commit 64a0b4fa563795bc22753940aa3a4a2946113109)
commit 5817078ba9b2aa38f39e1f62d8d08e943646c0bb
Merge: 9fa8d2d abf6441
Author: Sage Weil <sage@inktank.com>
Date: Thu Jan 23 07:12:37 2014 -0800
Merge pull request #1129 from ceph/wip-dumpling-backport-6620
mds: MDSMap: adjust buffer size for uint64 values with more than 5 chars
commit abf64417bbc63b006c53a07262a5774d0d40550e
Author: Joao Eduardo Luis <jecluis@gmail.com>
Date: Thu Jan 23 11:36:16 2014 +0000
mds: MDSMap: adjust buffer size for uint64 values with more than 5 chars
Fixes: #6620
Signed-off-by: Joao Eduardo Luis <joao.luis@inktank.com>
(cherry picked from commit 0e8182edd850f061421777988974efbaa3575b9f)
commit 9fa8d2d1bd4a8cf56fdebae484bee35c3930fb62
Author: Sage Weil <sage@inktank.com>
Date: Tue Jan 21 11:29:56 2014 -0800
mon/MDSMonitor: do not generate mdsmaps from already-laggy mds
There is one path where a mds that is not sending its beacon (e.g.,
because it is not running at all) will lead to proposal of new mdsmaps.
Fix it.
Backport: emperor, dumpling
Signed-off-by: Sage Weil <sage@inktank.com>
(cherry picked from commit 584c2dd6bea3fe1a3c7f306874c054ce0cf0d2b5)
commit a2e34b8d322e571600d6c0a3ff363f9910196f28
Author: John Spray <john.spray@inktank.com>
Date: Mon Jan 20 11:08:27 2014 +0000
Fix #7187: Include all summary items in JSON health output
Signed-off-by: John Spray <john.spray@inktank.com>
(cherry picked from commit fdf3b5520d150f14d90bdfc569b70c07b0579b38)
commit ee9fc590a556354b54a174a4790c4142949d7409
Author: Yehuda Sadeh <yehuda@inktank.com>
Date: Mon Jan 6 18:32:42 2014 -0800
rgw: convert bucket info if needed
Fixes: #7110
In dumpling, the bucket info was separated into bucket entry point and
bucket instance objects. When setting bucket attrs we only ended up
updating the bucket instance object. However, pre-dumpling buckets still
keep everything at the entry-point object, so acl changes didn't affect
anything (because we never updated the entry point). This change just
converts the bucket info into the new format.
Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
(cherry picked from commit a5f8cc7ec9ec8bef4fbc656066b4d3a08e5b215b)
commit 69a99e6edd67aadaea77fb5ebcf46190dbfcdbc6
Author: Sage Weil <sage@inktank.com>
Date: Sat Jan 4 22:40:43 2014 -0800
osd: ignore OSDMap messages while we are initializing
The mon may occasionally send OSDMap messages to random OSDs, but is not
very descriminating in that we may not have authenticated yet. Ignore any
messages if that is the case; we will reqeust whatever we need during the
BOOTING state.
Fixes: #7093
Signed-off-by: Sage Weil <sage@inktank.com>
(cherry picked from commit f68de9f352d53e431b1108774e4a23adb003fe3f)
commit 1545ddbd3b6dfac897d483bd58a4335d68b9b62a
Author: Sage Weil <sage@inktank.com>
Date: Sat Jan 4 22:43:26 2014 -0800
mon: only send messages to current OSDs
When choosing a random OSD to send a message to, verify not only that
the OSD id is up but that the session is for the same instance of that OSD
by checking that the address matches.
Fixes: #7093
Backport: emperor, dumpling
Signed-off-by: Sage Weil <sage@inktank.com>
(cherry picked from commit 98ed9ac5fed6eddf68f163086df72faabd9edcde)
commit 4d88dd10bfab4e5fb45632245be5f79eeba73a30
Author: Sage Weil <sage@inktank.com>
Date: Mon Aug 26 13:58:47 2013 -0700
osd: discriminate based on connection messenger, not peer type
Replace ->get_source().is_osd() checks and instead see if it is the
cluster_messenger so that we do not confuse ourselves when we get
legit requests from other OSDs on our public interface.
NOTE: backporting this because a mixed cluster may send OSD requests
via the client interface, even though dumpling doesn't do this.
Signed-off-by: Sage Weil <sage@inktank.com>
(cherry picked from commit a6b04c5d8ba043727a2e39a62e9d4126485bcfeb)
Conflicts:
src/osd/OSD.cc
commit 2354b957b7924af3e81f559d5c975262848c3bf6
Author: Loic Dachary <loic@dachary.org>
Date: Sun Dec 15 22:59:51 2013 +0100
qa: test for error when ceph osd rm is EBUSY
http://tracker.ceph.com/issues/6824 fixes #6824
Signed-off-by: Loic Dachary <loic@dachary.org>
(cherry picked from commit 31507c90f0161c4569a2cc634c0b5f671179440a)
commit 10ad2e01f832ca9293a1bda1c82f7d318e8c3589
Author: Loic Dachary <loic@dachary.org>
Date: Sun Dec 15 16:27:02 2013 +0100
mon: set ceph osd (down|out|in|rm) error code on failure
Instead of always returning true, the error code is set if at least one
operation fails.
EINVAL if the OSD id is invalid (osd.foobar for instance).
EBUSY if trying to remove and OSD that is up.
When used with the ceph command line, it looks like this:
ceph -c ceph.conf osd rm osd.0
Error EBUSY: osd.0 is still up; must be down before removal.
kill PID_OF_osd.0
ceph -c ceph.conf osd down osd.0
marked down osd.0.
ceph -c ceph.conf osd rm osd.0 osd.1
Error EBUSY: removed osd.0, osd.1 is still up; must be down before removal.
http://tracker.ceph.com/issues/6824 fixes #6824
Signed-off-by: Loic Dachary <loic@dachary.org>
(cherry picked from commit 15b8616b13a327701c5d48c6cb7aeab8fcc4cafc)
commit db82bfec50349783c3bb708c65cf89bceff7b2ca
Author: Joao Eduardo Luis <joao.luis@inktank.com>
Date: Tue Oct 29 20:30:37 2013 +0000
mon: OSDMonitor: fix some annoying whitespace
Signed-off-by: Joao Eduardo Luis <joao.luis@inktank.com>
(cherry picked from commit 42c4137cbfacad5654f02c6608cc0e81b45c06be)
commit 1abb16988727a21dff61edca48bfcb05c60d26f2
Author: Josh Durgin <josh.durgin@inktank.com>
Date: Thu Dec 26 17:38:52 2013 -0800
librbd: call user completion after incrementing perfcounters
The perfcounters (and the ictx) are only valid while the image is
still open. If the librbd user gets the callback for its last I/O,
then closes the image, the ictx and its perfcounters will be
invalid. If the AioCompletion object is has not run the rest of its
complete() method yet, it will access these now-invalid addresses,
possibly leading to a crash.
The AioCompletion object is independent of the ictx and does not
access it again after incrementing perfcounters, so avoid this race by
calling the user's callback after this step. The AioCompletion object
will be cleaned up by the rest of complete_request(), independent of
the ImageCtx.
Fixes: #5426
Backport: dumpling, emperor
Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
(cherry picked from commit 4cea7895da7331b84d8c6079851fdc0ff2f4afb1)
commit 4dae50d8a0ffe3f3491eae62950fe46229aaa0e6
Author: Josh Durgin <josh.durgin@inktank.com>
Date: Fri Dec 6 16:03:20 2013 -0800
objecter: don't take extra throttle budget for resent ops
These ops have already taken their budget in the original op_submit().
It will be returned via put_op_budget() when they complete.
If there were many localized reads of missing objects from replicas,
or cache pool redirects, this would cause the objecter to use up all
of its op throttle budget and hang.
Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
(cherry picked from commit 8d0180b1b7b48662daef199931efc7f2a6a1c431)
Conflicts:
src/osdc/Objecter.cc
commit 4944618fe74e1875e37a5de9a5bfe3922f906942
Author: Josh Durgin <josh.durgin@inktank.com>
Date: Thu Dec 5 17:44:37 2013 -0800
rbd: check write return code during bench-write
This is allows rbd-bench to detect http://tracker.ceph.com/issues/6938
when combined with rapidly changing the mon osd full ratio.
Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
(cherry picked from commit 3caf3effcb113f843b54e06099099909eb335453)
commit 3a7a0353d7c8521a2c3c25778801d45f0ca10957
Author: Josh Durgin <josh.durgin@inktank.com>
Date: Thu Dec 5 17:36:33 2013 -0800
objecter: resend all writes after osdmap loses the full flag
Now that the osd does not respond if it gets a map with the full flag
set first, clients need to resend all writes.
Clients talking to old osds are still subject to the race condition,
so both sides must be upgraded to avoid it.
Refs: #6938
Backport: dumpling, emperor
Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
(cherry picked from commit e32874fc5aa6f59494766b7bbeb2b6ec3d8f190e)
Conflicts:
src/osdc/Objecter.h
commit f4d0fc8afac1bcbea76f46744397a8462025f373
Author: Josh Durgin <josh.durgin@inktank.com>
Date: Thu Dec 5 17:34:38 2013 -0800
osd: drop writes when full instead of returning an error
There's a race between the client and osd with a newly marked full
osdmap. If the client gets the new map first, it blocks writes and
everything works as expected, with no errors from the osd.
If the osd gets the map first, however, it will respond to any writes
with -ENOSPC. Clients will pass this up the stack, and not retry these
writes later. -ENOSPC isn't handled well by all clients. RBD, for
example, may pass it on to qemu or kernel rbd which will both
interpret it as EIO. Filesystems on top of rbd will not behave well
when they receive EIOs like this, especially if the cluster oscillates
between full and not full, so some writes succeed.
To fix this, never return ENOSPC from the osd because of a map marked
full, and rely on the client to retry all writes when the map is no
longer marked full.
Old clients talking to osds with this fix will hang instead of
propagating an error, but only if they run into this race
condition. ceph-fuse and rbd with caching enabled are not affected,
since the ObjectCacher will retry writes that return errors.
Refs: #6938
Backport: dumpling, emperor
Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
(cherry picked from commit 4111729dda7437c23f59e7100b3c4a9ec4101dd0)
commit 2b109803b9434d2d495e65d6a045838ee3ebf169
Author: Yehuda Sadeh <yehuda@inktank.com>
Date: Wed Nov 6 16:55:52 2013 -0800
objecter: clean pause / unpause logic
op->paused holds now whether operation should be paused or not, and it's
being updated when scanning requests. No need to do a second scan.
Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
(cherry picked from commit 5fe3dc647bf936df8e1eb2892b53f44f68f19821)
commit 7b229fca346b7ab60131b229591d35f63f64a6ad
Author: Yehuda Sadeh <yehuda@inktank.com>
Date: Wed Nov 6 16:15:47 2013 -0800
objecter: set op->paused in recalc_op_target(), resend in not paused
When going through scan_requests() in handle_osd_map() we need to make
sure that if an op should not be paused anymore then set it on the op
itself, and return NEED_RESEND. Otherwise we're going to miss reset of
the full flag.
Also in handle_osd_map(), make sure that op shouldn't be paused before
sending it. There's a lot of cleanup around that area that we should
probably be doing, make the code much more tight.
Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
(cherry picked from commit 98ab7d64a191371fa39d840c5f8e91cbaaa1d7b7)
commit a3fdfd77c953a8f9ace4991630599e60f56abc4a
Author: Josh Durgin <josh.durgin@inktank.com>
Date: Wed Nov 6 10:46:37 2013 +0800
objecter: don't resend paused ops
Paused ops are meant to block on the client side until a new map that
unpauses them is recieved. If we send paused writes when the FULL flag
is set, we'll get -ENOSPC from the osds, which is not what Objecter
users expect. This may cause rbd without caching to produce an I/O
error instead of waiting for the cluster to have capacity.
Fixes: #6725
Backport: dumpling
Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
(cherry picked from commit c5c399d327cfc0d232d9ec7d49ababa914d0b21a)
commit 78037886ae6ae278d8fb2db52d7767b8612313fe
Merge: a60ac91 4e47a00
Author: Sage Weil <sage@inktank.com>
Date: Sun Dec 29 21:10:30 2013 -0800
Merge pull request #808 from ceph/wip-6152-dumpling
rgw: Fix S3 auth when using response-* query string params
commit 4e47a003bb3adcee690067d4037c248a93023c20
Author: Sylvain Munaut <s.munaut@whatever-company.com>
Date: Thu Aug 29 16:17:30 2013 +0200
rgw: Fix S3 auth when using response-* query string params
Reviewed-by: Yehuda Sadeh <yehuda@inktank.com>
Signed-off-by: Sylvain Munaut <s.munaut@whatever-company.com>
(cherry picked from commit 7a7361d7e798d94796d4985e2e4b35af22463ae2)