Commit Graph

23673 Commits

Author SHA1 Message Date
Yehuda Sadeh
70532d1985 rgw: get bucket_owner from policy
We already read the bucket policy, we can get the bucket
owner from there.

Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
2013-02-07 14:54:54 -08:00
Yehuda Sadeh
f6af1e76a8 rgw: fix bucket_owner assignment
s->bucket_acl may be null, so reverting to old behavior.

Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
2013-02-07 14:54:54 -08:00
caleb miles
e345dfe04a Feature 3667: Support extra canned acls.
Support the bucket-owner-read and bucket-owner-full
canned acls.

Signed-off-by caleb miles <caleb.miles@inktank.com>
Reviewed-by: Yehuda Sadeh <yehuda@inktank.com>
2013-02-07 14:54:39 -08:00
tamil
5896b971a3 modified the script to run on both argonaut and bobtail.
Signed-off-by: tamil <tamil.muthamizhan@inktank.com>
2013-02-07 11:47:06 -08:00
Greg Farnum
9432353519 mds: rename mds_traceless_replies to mds_inject_traceless_reply_probability
Sage pointed out we should try for consistent naming on these debug
options, and this option is like our other inject_[x] stuff.
While we're here, make the config option comment a bit clearer.

08b82b3ef6
2013-02-07 10:25:33 -08:00
Sage Weil
75c40fac60 qa: fix iogen script
Wait 10 minutes and then stop.

Signed-off-by: Sage Weil <sage@inktank.com>
2013-02-06 22:01:24 -08:00
Greg Farnum
46d7dbd347 client: trigger the completion in _flush when short-cutting
We missed a shortcut return from _flush() when doing
e9a6694d01, so _fsync() calls
were failing. To fix, if _flush discovers there's nothing to flush,
trigger the completion by calling onfinish->finish().

Fixes #4038

Signed-off-by: Greg Farnum <greg@inktank.com>
Reviewed-by: Sage Weil <sage@inktank.com>
2013-02-06 16:47:51 -08:00
Greg Farnum
08b82b3ef6 mds: add "mds traceless replies" debug option
This option specifies (in the range 0-1) the percentage of modifying
operations that should be responded to without including a trace
for the dentries.
In order to implement it, we add a "modify" parameter to
set_trace_dist(), and the callers fill that with
mdr->req->may_write().

Signed-off-by: Greg Farnum <greg@inktank.com>
Reviewed-by: Sam Lang <sam.lang@inktank.com>
2013-02-06 13:34:02 -08:00
Gary Lowell
9871cf2751 logrotate.conf: Silence rgw logrotate some more
Apply the same change as commit d02340d90c to
the radosgw logrotate.conf.

Signed-off-by: Gary Lowell  <gary.lowell@inktank.com>
2013-02-06 11:45:30 -08:00
Alexandre Oliva
d02340d90c silence logrotate some more
I was getting email with logrotate error output from “which invoke-rc.d”
on systems without an invoke-rc.d.  This patch silences it.

Silence stderr from which when running logrotate

From: Alexandre Oliva <oliva@gnu.org>

Signed-off-by: Alexandre Oliva <oliva@gnu.org>
2013-02-06 11:44:56 -08:00
Sage Weil
f81e09525f Merge remote-tracking branch 'gh/wip-danny-cleanups'
Reviewed-by: Sage Weil <sage@inktank.com>
2013-02-06 10:43:29 -08:00
Sage Weil
0aea4dba04 Merge remote-tracking branch 'gh/wip-3768' 2013-02-06 10:38:08 -08:00
Greg Farnum
c0e1070f2f test: fix Throttle unit test.
A bunch of these are slightly racy so they're enclosed in loops. This
particular one, though, changes the Throttle state in ways that
invalidate the asserts. To fix, reset the state before commencing a
rerun.

Signed-off-by: Greg Farnum <greg@inktank.com>
2013-02-06 10:22:06 -08:00
Sage Weil
1948a02bc8 osd: do not spam system log on successful read_log
Signed-off-by: Sage Weil <sage@inktank.com>
2013-02-06 09:02:54 -08:00
Danny Al-Gaaf
3acc4d2cfe rbd-fuse: fix for loop in open_rbd_image()
Remove uninitialized usage of 'int i' as i++ from 'for' loop.
The variale 'i' is never used in this loop and initialized
before the next use with 0.

Related warning from clang++:

rbd_fuse/rbd-fuse.c:141:36: warning: variable 'i' is uninitialized
when used here [-Wuninitialized]
        for (im = rbd_images; im != NULL; i++, im = im->next) {

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-02-06 08:42:43 -08:00
Danny Al-Gaaf
db0dbe5db8 msg/Message.h: fix C-style pointer casting
Replace C-style pointer casting with correct static_cast<>().

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-02-06 08:42:04 -08:00
Danny Al-Gaaf
b1fc10ef93 messages/MOSDRepScrub.h: initialize member variable in constructor
Initialize chunky and deep bool member variables in the constructor
with false.

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-02-06 08:42:04 -08:00
Danny Al-Gaaf
22e48b5751 include/xlist.h: fix C-style pointer casting
Replace C-style pointer casting with correct static_cast<>().

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-02-06 08:42:03 -08:00
Danny Al-Gaaf
d54bd1700f include/types.h: change operator<< function parameter
Fix "Function parameter 'v' should be passed by reference." from cppchecker.
Use 'const pair<A,B>& v' similar to the other operator<< in this file.

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-02-06 08:42:03 -08:00
Danny Al-Gaaf
0327cbaa2a include/buffer.h: fix operator=
Fix operator=: return "iterator&" instead of 'iterator'. Check if 'this'
equals 'other' before set anything.

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-02-06 08:42:03 -08:00
Danny Al-Gaaf
ad526c0e44 obj_bencher.cc: use vector instead of VLA's
Fix "variable length array of non-POD element type" error. (-Wvla)

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-02-06 08:42:03 -08:00
Danny Al-Gaaf
a4042cc3c9 ceph_crypto.cc: remove unused shutdown() outside crypto ifdef's
Fix "out-of-line declaration of a member must be a definition
[-Wout-of-line-declaration]". Remove ceph::crypto::shutdown() outside
the crypto related ifdef's. Without nss or cryptopp configure will
fail anyways.

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-02-06 08:42:03 -08:00
Danny Al-Gaaf
42682963cc WorkQueue.h: fix cast
Replace C-style pointer casting with correct static_cast<>().

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-02-06 08:42:03 -08:00
Sage Weil
9eff2ee13d Merge remote-tracking branch 'gh/wip-osd-commands'
Reviewed-by: Dan Mick <dan.mick@inktank.com>
Reviewed-by: Joao Luis <joao.luis@inktank.com>
2013-02-05 17:31:48 -08:00
Sage Weil
3b635423bd mon: move list_rules into CrushWrapper method
Signed-off-by: Sage Weil <sage@inktank.com>
2013-02-05 16:35:55 -08:00
Sage Weil
9f4d4ac9cc crush: add list_rules() method
Signed-off-by: Sage Weil <sage@inktank.com>
2013-02-05 16:35:43 -08:00
Gary Lowell
7f237be2e2 Makefile: Add rgw/logrotate.conf source tarball
Signed-off-by: Gary Lowell  <gary.lowell@inktank.com>
2013-02-05 16:14:10 -08:00
Gary Lowell
99ea303054 logrotate.conf: Remove unneeded loop and update new rgw version.
Remove an unneeded for loop from the ceph logrotate.conf, and
update the new rgw logrotate.conf to reload the radosgw serivce.

Signed-off-by: Gary Lowell  <gary.lowell@inktank.com>
2013-02-05 15:20:43 -08:00
Yehuda Sadeh
c8eace6ff0 rgw: create a separate logrotate file for radosgw
Fixes: #3813
Since radosgw package is separate from the ceph package,
it also needs to have a separate logrotate. The default
path for radosgw log is changed to /var/log/radosgw,
and it now has a different logrotate script.
Updating both deb and rpm packaging.

Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
2013-02-05 14:54:38 -08:00
Sage Weil
b79067a863 qa: add workunits/mon/crush_ops.sh
Test creating, listing, removing crush rules via the mon.

Signed-off-by: Sage Weil <sage@inktank.com>
2013-02-05 14:31:13 -08:00
Sage Weil
2c559a7ad1 mon: 'osd crush rule rm <name>'
Signed-off-by: Sage Weil <sage@inktank.com>
2013-02-05 14:31:13 -08:00
Sage Weil
a19cdd490a osdmap: method to check if a crush ruleset is in use
Signed-off-by: Sage Weil <sage@inktank.com>
2013-02-05 14:31:13 -08:00
Sage Weil
b9bd482d7e crush: remove_rule() method
Signed-off-by: Sage Weil <sage@inktank.com>
2013-02-05 14:31:13 -08:00
Sage Weil
43a01c9973 crush: factor out (trivial) crush_destroy_rule()
Signed-off-by: Sage Weil <sage@inktank.com>
2013-02-05 14:31:13 -08:00
Sage Weil
c370d85b42 mon: 'osd crush rules list|ls'
Signed-off-by: Sage Weil <sage@inktank.com>
2013-02-05 14:31:10 -08:00
Sage Weil
b6036a58b9 mon: 'osd crush dump'
Signed-off-by: Sage Weil <sage@inktank.com>
2013-02-05 14:30:37 -08:00
Sage Weil
9da6290c7b crush: factor out dump_rules from dump
Signed-off-by: Sage Weil <sage@inktank.com>
2013-02-05 14:30:37 -08:00
Sage Weil
a04d3f0ab1 mon: 'osd crush rule create-simple <name> <root> <failure_domain_type>'
Simple command to create simple rules.

Signed-off-by: Sage Weil <sage@inktank.com>
2013-02-05 14:30:34 -08:00
Sage Weil
1a386d6c98 crush: add_simple_rule() command
Method to create a very generic rule the distributes objects across the
specified failure domain type underneath the given root.

Signed-off-by: Sage Weil <sage@inktank.com>
2013-02-05 14:29:13 -08:00
Sage Weil
d7ada58a69 crush: fix get_rule_id() return value
There are 0 callers, yay!

Signed-off-by: Sage Weil <sage@inktank.com>
2013-02-05 14:26:05 -08:00
Sage Weil
4f992ea344 crush: add rule_exists()
Signed-off-by: Sage Weil <sage@inktank.com>
2013-02-05 14:26:04 -08:00
Sage Weil
3105700d87 mon: 'osd find <osd-id>' command
Simple command to find the ip, host, rack, etc. for an OSD.  This is better
than 'ceph osd dump | grep ^osd.NNN\ '.

Signed-off-by: Sage Weil <sage@inktank.com>
2013-02-05 14:26:04 -08:00
Greg Farnum
b3ffc718c9 Merge branch 'wip-2753-fsync-errors'
Reviewed-by: Sage Weil <sage@inktank.com>
2013-02-05 13:48:49 -08:00
Gregory Farnum
13e22262c0 Merge pull request #39 from dachary/master
Relax Throttle::_reset_max conditions and associated unit tests
2013-02-05 11:13:49 -08:00
Loic Dachary
64ded02c99 Relax Throttle::_reset_max conditions and associated unit tests
Removes a condition in Throttle::_reset_max by which the waiting queue is only
Signal()ed if the new maximum is lower than the current maximum.
There is no evidence of a use case where such a restriction would be
useful. In addition waking up a thread when the maximum increases
gives it a chance to immediately continue the suspended process
instead of waiting for the next put().

Create a new test file covering 100% of src/Throttle.{cc,h} lines of code.
The following methods are tested:

* Throttle::Throttle with and without a maximum
* Throttle::~Throttle when each pending Cond is deleted
* Throttle::take
* Throttle::get when updating the maximum ( lower or higher ),
  when going to sleep waiting for the count to lower under
  the maximum, when going to sleep because another thread is
  already asleep waiting
* Throttle::get_or_fail when there is no maximum,
  when requesting a count that is larger than the maximum, either
  when the current value is under the maximum or above the maximum.
* Throttle::wait when used to reset the maximum and wake up
  another thread asleep waiting

All asserts checking the arguments sanity are exercised ( negative argument
for Throttle::take etc. ).
Adds the LGPLv2+ licensing terms to COPYING along with the others.
Adds a Contributors section to the AUTHORS file.

Notes:
Testing asserts outputs verbose error messages that should be silenced
but it does not seem possible.

Signed-off-by: Loic Dachary <loic@dachary.org>
2013-02-05 20:06:04 +01:00
Sage Weil
ca2d6459f8 os: default to 'journal aio = true'
Hooray, testing indicates this is a win!

Signed-off-by: Sage Weil <sage@inktank.com>
2013-02-05 10:29:11 -08:00
Gregory Farnum
e43a13c056 Merge pull request #36 from cmello/master
libexpat dependency
2013-02-05 10:20:18 -08:00
John Wilkins
26f7db11c1 Merge pull request #38 from alram/master
Fixes in ./docs/radosgw/config.rst
2013-02-05 09:57:32 -08:00
Alexandre Marangone
d41b541189 Edit endpoint-create in ./doc/radosgw/config.rst
internalurl and adminurl are mandatory. Typo in publicurl.
2013-02-04 21:20:07 -08:00
Alexandre Marangone
6e60330132 Edit rgw keystone url in ./doc/radosgw/config.rst
Won't work with the public port, it needs to be the admin port.
2013-02-04 21:14:54 -08:00