summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* rgw: add encode_jsonwip-json-decodeYehuda Sadeh2013-02-124-17/+49
| | | | | | | | | | dump() just dumps the internal content of an object, encode_json() create the object inside its own section. Note that there are cases where we don't want an object to be surrounded by a section, e.g., when an object represents an array. In such a case we'd need to override the encode_json() for this object type. Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
* moved rgw/rgw_json.* common/ceph_json.*Yehuda Sadeh2013-02-117-27/+26
| | | | Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
* rgw: rgw_jsonparser changesYehuda Sadeh2013-02-111-11/+61
| | | | | | Now testing json_decode functionality. Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
* rgw: decode RGWUserInfo using decode_jsonYehuda Sadeh2013-02-116-385/+527
| | | | | | | RGWUserInfo can be decoded using json_decode, and fix the ->dump() method to match that decoding. Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
* rgw: introduce decode_json to easily decode json structuresYehuda Sadeh2013-02-112-4/+193
| | | | | | | Introducing a new way to define the json structure of an object, similar to what we do with bufferlist encode/decode. Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
* rgw: get bucket_owner from policyYehuda Sadeh2013-02-071-13/+2
| | | | | | | We already read the bucket policy, we can get the bucket owner from there. Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
* rgw: fix bucket_owner assignmentYehuda Sadeh2013-02-071-1/+1
| | | | | | s->bucket_acl may be null, so reverting to old behavior. Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
* Feature 3667: Support extra canned acls.caleb miles2013-02-077-21/+70
| | | | | | | | 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>
* modified the script to run on both argonaut and bobtail.tamil2013-02-072-9/+17
| | | | Signed-off-by: tamil <tamil.muthamizhan@inktank.com>
* mds: rename mds_traceless_replies to mds_inject_traceless_reply_probabilityGreg Farnum2013-02-072-4/+5
| | | | | | | | 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. 08b82b3ef6b43283e35fd4e56eb5c78651345bea
* qa: fix iogen scriptSage Weil2013-02-061-0/+5
| | | | | | Wait 10 minutes and then stop. Signed-off-by: Sage Weil <sage@inktank.com>
* client: trigger the completion in _flush when short-cuttingGreg Farnum2013-02-061-0/+2
| | | | | | | | | | | | We missed a shortcut return from _flush() when doing e9a6694d0151b79c3a3b44cee5df8e3d4dcbfc2c, 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>
* mds: add "mds traceless replies" debug optionGreg Farnum2013-02-063-4/+15
| | | | | | | | | | | | 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>
* logrotate.conf: Silence rgw logrotate some moreGary Lowell2013-02-061-3/+3
| | | | | | | Apply the same change as commit d02340d90c9d30d44c962bea7171db3fe3bfba8e to the radosgw logrotate.conf. Signed-off-by: Gary Lowell <gary.lowell@inktank.com>
* silence logrotate some moreAlexandre Oliva2013-02-061-3/+3
| | | | | | | | | | | 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>
* Merge remote-tracking branch 'gh/wip-danny-cleanups'Sage Weil2013-02-069-20/+25
|\ | | | | | | Reviewed-by: Sage Weil <sage@inktank.com>
| * rbd-fuse: fix for loop in open_rbd_image()Danny Al-Gaaf2013-02-061-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | 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>
| * msg/Message.h: fix C-style pointer castingDanny Al-Gaaf2013-02-061-2/+2
| | | | | | | | | | | | Replace C-style pointer casting with correct static_cast<>(). Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
| * messages/MOSDRepScrub.h: initialize member variable in constructorDanny Al-Gaaf2013-02-061-1/+4
| | | | | | | | | | | | | | Initialize chunky and deep bool member variables in the constructor with false. Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
| * include/xlist.h: fix C-style pointer castingDanny Al-Gaaf2013-02-061-3/+3
| | | | | | | | | | | | Replace C-style pointer casting with correct static_cast<>(). Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
| * include/types.h: change operator<< function parameterDanny Al-Gaaf2013-02-061-1/+1
| | | | | | | | | | | | | | 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>
| * include/buffer.h: fix operator=Danny Al-Gaaf2013-02-061-3/+5
| | | | | | | | | | | | | | 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>
| * obj_bencher.cc: use vector instead of VLA'sDanny Al-Gaaf2013-02-061-6/+7
| | | | | | | | | | | | Fix "variable length array of non-POD element type" error. (-Wvla) Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
| * ceph_crypto.cc: remove unused shutdown() outside crypto ifdef'sDanny Al-Gaaf2013-02-061-1/+0
| | | | | | | | | | | | | | | | | | 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>
| * WorkQueue.h: fix castDanny Al-Gaaf2013-02-061-2/+2
| | | | | | | | | | | | Replace C-style pointer casting with correct static_cast<>(). Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
* | Merge remote-tracking branch 'gh/wip-3768'Sage Weil2013-02-067-10/+36
|\ \
| * | Makefile: Add rgw/logrotate.conf source tarballGary Lowell2013-02-051-0/+1
| | | | | | | | | | | | Signed-off-by: Gary Lowell <gary.lowell@inktank.com>
| * | logrotate.conf: Remove unneeded loop and update new rgw version.Gary Lowell2013-02-052-7/+15
| | | | | | | | | | | | | | | | | | | | | 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>
| * | rgw: create a separate logrotate file for radosgwYehuda Sadeh2013-02-054-1/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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>
| * | cli test: add pg deep-scrub option to testGary Lowell2013-02-041-0/+1
| | | | | | | | | | | | Signed-off-by: Gary Lowell <gary.lowell@inktank.com>
| * | install: remove perl dependencyGary Lowell2013-02-043-3/+2
| | | | | | | | | | | | | | | | | | | | | Change the filter in logrotate to use sed instead of perl, and remove the package dependency on perl. Signed-off-by: Gary Lowell <gary.lowell@inktank.com>
* | | test: fix Throttle unit test.Greg Farnum2013-02-061-2/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | 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>
* | | osd: do not spam system log on successful read_logSage Weil2013-02-061-1/+2
| |/ |/| | | | | Signed-off-by: Sage Weil <sage@inktank.com>
* | Merge remote-tracking branch 'gh/wip-osd-commands'Sage Weil2013-02-058-5/+275
|\ \ | | | | | | | | | | | | Reviewed-by: Dan Mick <dan.mick@inktank.com> Reviewed-by: Joao Luis <joao.luis@inktank.com>
| * | mon: move list_rules into CrushWrapper methodSage Weil2013-02-051-5/+1
| | | | | | | | | | | | Signed-off-by: Sage Weil <sage@inktank.com>
| * | crush: add list_rules() methodSage Weil2013-02-052-0/+10
| | | | | | | | | | | | Signed-off-by: Sage Weil <sage@inktank.com>
| * | qa: add workunits/mon/crush_ops.shSage Weil2013-02-051-0/+23
| | | | | | | | | | | | | | | | | | Test creating, listing, removing crush rules via the mon. Signed-off-by: Sage Weil <sage@inktank.com>
| * | mon: 'osd crush rule rm <name>'Sage Weil2013-02-051-0/+46
| | | | | | | | | | | | Signed-off-by: Sage Weil <sage@inktank.com>
| * | osdmap: method to check if a crush ruleset is in useSage Weil2013-02-052-0/+10
| | | | | | | | | | | | Signed-off-by: Sage Weil <sage@inktank.com>
| * | crush: remove_rule() methodSage Weil2013-02-052-0/+15
| | | | | | | | | | | | Signed-off-by: Sage Weil <sage@inktank.com>
| * | crush: factor out (trivial) crush_destroy_rule()Sage Weil2013-02-052-1/+7
| | | | | | | | | | | | Signed-off-by: Sage Weil <sage@inktank.com>
| * | mon: 'osd crush rules list|ls'Sage Weil2013-02-051-0/+27
| | | | | | | | | | | | Signed-off-by: Sage Weil <sage@inktank.com>
| * | mon: 'osd crush dump'Sage Weil2013-02-051-0/+11
| | | | | | | | | | | | Signed-off-by: Sage Weil <sage@inktank.com>
| * | crush: factor out dump_rules from dumpSage Weil2013-02-052-1/+7
| | | | | | | | | | | | Signed-off-by: Sage Weil <sage@inktank.com>
| * | mon: 'osd crush rule create-simple <name> <root> <failure_domain_type>'Sage Weil2013-02-051-0/+42
| | | | | | | | | | | | | | | | | | Simple command to create simple rules. Signed-off-by: Sage Weil <sage@inktank.com>
| * | crush: add_simple_rule() commandSage Weil2013-02-052-0/+43
| | | | | | | | | | | | | | | | | | | | | 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>
| * | crush: fix get_rule_id() return valueSage Weil2013-02-051-1/+1
| | | | | | | | | | | | | | | | | | There are 0 callers, yay! Signed-off-by: Sage Weil <sage@inktank.com>
| * | crush: add rule_exists()Sage Weil2013-02-051-2/+5
| | | | | | | | | | | | Signed-off-by: Sage Weil <sage@inktank.com>
| * | mon: 'osd find <osd-id>' commandSage Weil2013-02-051-0/+32
| | | | | | | | | | | | | | | | | | | | | 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>
* | | Merge branch 'wip-2753-fsync-errors'Greg Farnum2013-02-052-14/+53
|\ \ \ | |/ / |/| | | | | Reviewed-by: Sage Weil <sage@inktank.com>