Commit Graph

25453 Commits

Author SHA1 Message Date
Sage Weil
7f19915f30 Merge pull request #186 from dalgaaf/wip-da-pylint
Fix smaller python issues
2013-04-02 17:41:16 -07:00
Sage Weil
04fff0dc71 Merge pull request #187 from imjustmatthew/imjustmatthew_docs2
Adds "mds fail 0" command to operations commmand reference.
2013-04-02 17:40:50 -07:00
Dan Mick
17a55660e8 Merge pull request #188 from dmick/wip-test-config-key
test_mon_config_key.py: fix 'del' to clean up correctly internally

Reviewed-by: Joao Eduardo Luis <joao.luis@inktank.com>
2013-04-02 16:38:12 -07:00
Dan Mick
cfd32fff7c test_mon_config_key.py: fix 'del' to clean up correctly internally 2013-04-02 15:09:43 -07:00
Greg Farnum
3842ff7d67 Merge remote-tracking branch 'origin/wip-4619'
Reviewed-by: Greg Farnum <greg@inktank.com>
2013-04-02 14:38:44 -07:00
Sage Weil
5cf0c492e7 mds: initialize tableservers/clients on mds creation
The handle_mds_recovery(who) path initializes the anchorclients by having
the server send a 'ready' message on recovery when the server is active
and a peer becomes active.  Similarly, recovery_done() does the same when
the server becomes active.  However, this misses the creation path.  Handle
that explicitly in boot_create.

Fixes: #4619
Signed-off-by: Sage Weil <sage@inktank.com>
2013-04-02 13:05:49 -07:00
Sage Weil
1054c87c19 Revert "mds: trigger tableserver active/recovery hook even for self"
This reverts commit 968c6c0c94.

This will trigger the 'ready' message twice when we restart, because we
will trigger in both recovery_done() and handle_mds_recovery().

Signed-off-by: Sage Weil <sage@inktank.com>
2013-04-02 13:05:46 -07:00
Dan Mick
8a6438fefa README: fix dependency lists
1) automake appeared twice
2) make apt-get command match the list
2013-04-02 13:01:04 -07:00
Matthew Roy
0c16b31db7 Adds "mds fail 0" command to operations commmand reference.
Partially fixes #2206, though better documentation will eventually be needed.
2013-04-02 13:57:53 -04:00
Sage Weil
6072f8f7c0 Merge pull request #184 from dachary/wip-4617
explain what an inline xattr is and how it relates to omap
2013-04-02 09:35:05 -07:00
Sage Weil
968c6c0c94 mds: trigger tableserver active/recovery hook even for self
The tableserver now sends a READY message to clients when they go active;
we need to do this even for our own local tableclients, or else they do
not initialize and hang on first use after bringing up a fresh cluster.

Signed-off-by: Sage Weil <sage@inktank.com>
Reviewed-by: Greg Farnum <greg@inktank.com>
2013-04-02 08:58:35 -07:00
Danny Al-Gaaf
c4eb7e6ddd ceph-disk: fix some (local) variable names
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-04-02 17:54:53 +02:00
Danny Al-Gaaf
3dd8b46121 ceph-disk: fix naming of local variable in is_mounted()
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-04-02 17:36:37 +02:00
Danny Al-Gaaf
eaf31bf9f9 ceph-disk: merge twice defined function is_mounted(dev)
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-04-02 17:33:08 +02:00
Danny Al-Gaaf
4c6d6442a8 ceph-disk: define exception type
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-04-02 17:26:12 +02:00
Danny Al-Gaaf
058eb923c5 ceph-disk: fix Redefining name 'uuid' from outer scope
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-04-02 17:17:38 +02:00
Danny Al-Gaaf
0080d1df7c ceph-disk: add missing space after comma
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-04-02 17:14:23 +02:00
Danny Al-Gaaf
b99b8cc8a2 rgw/rgw_user.cc: reduce scope of variable
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-04-02 17:01:07 +02:00
Danny Al-Gaaf
ee7eae3699 rgw/rgw_user.h: remove twice initialized purge_keys
Remove twice initialized purge_keys from RGWUserAdminOpState();

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-04-02 16:50:41 +02:00
Danny Al-Gaaf
6e9f4781d3 test_cors.cc: prefer prefix ++operator for iterator
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-04-02 16:36:17 +02:00
Danny Al-Gaaf
b4845a6975 test_cors.cc: use static_cast instead of C-Style cast
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-04-02 16:30:57 +02:00
Danny Al-Gaaf
e2c3511b7a test_cors.cc: use %u to format unsigned in sprintf()
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-04-02 16:25:22 +02:00
Danny Al-Gaaf
bda21f7264 rgw/rgw_user.cc: use 'true' directly instead of variable
Instead of passing 'true' via bool defer_user_update variable
in RGWUser::execute_modify() to keys.add() use it directly.

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-04-02 16:17:52 +02:00
Danny Al-Gaaf
84b1914b78 rgw/rgw_user.cc: reduce scope of same_email in execute_modify()
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-04-02 16:15:27 +02:00
Danny Al-Gaaf
13abf321c2 rgw/rgw_user.cc: remove some unused std::string variables
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-04-02 16:10:22 +02:00
Danny Al-Gaaf
4839cf2a21 rgw/rgw_cors_swift.h: prefer prefix ++operator for iterator
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-04-02 16:05:58 +02:00
Danny Al-Gaaf
620ca6c266 rgw/rgw_op.cc: prefer prefix ++operator for iterator
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-04-02 16:03:50 +02:00
Danny Al-Gaaf
275be1051b rgw/rgw_op.cc: remove unsused variable
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-04-02 16:03:10 +02:00
Danny Al-Gaaf
4a35cfdf07 rgw/rgw_op.cc: use static_cast instead of C-Style cast
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-04-02 16:02:30 +02:00
Danny Al-Gaaf
c662733a51 rgw/rgw_cors_s3.cc: prefer prefix ++operator for iterator
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-04-02 16:00:12 +02:00
Danny Al-Gaaf
9850ed1477 rgw/rgw_cors_s3.cc: remove unused variable
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-04-02 15:57:37 +02:00
Danny Al-Gaaf
21361708d9 rgw/rgw_cors_s3.cc: use static_cast instead of C-Style cast
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-04-02 15:55:51 +02:00
Loic Dachary
69f60d2644 explain what an inline xattr is and how it relates to omap
The logic of the configuration flags related to xattr is clarified to define what an inline xattr is and when storing in the object map is preferred.

http://tracker.ceph.com/issues/4617 refs #4617

Signed-off-by: Loic Dachary <loic@dachary.org>
2013-04-02 15:54:57 +02:00
Danny Al-Gaaf
7abba7b192 rgw/rgw_cors.cc: fix inefficient usage of string::find()
Fix warning from cppcheck:
 [src/rgw/rgw_cors.cc:70]: (performance) Inefficient usage of
 string::find() in condition; string::compare() would be faster.

Instead of string::find() use boost::algorithm::starts_with().

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-04-02 15:43:12 +02:00
Danny Al-Gaaf
eb815bb069 rgw/rgw_cors.cc: prefer prefix ++operator for iterator
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-04-02 14:52:06 +02:00
Danny Al-Gaaf
547972373e rgw/rgw_cors.cc: use empty() instead of size() == 0
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-04-02 14:47:54 +02:00
Danny Al-Gaaf
e1babf4734 rgw/rgw_bucket.cc: prefer prefix ++operator for iterator
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-04-02 14:43:36 +02:00
Danny Al-Gaaf
c9be9fff1b rgw/rgw_bucket.cc: remove unused variable
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-04-02 14:42:58 +02:00
Danny Al-Gaaf
42109d319b rgw/rgw_user.h: initialize some bool variables in constructor
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-04-02 14:41:56 +02:00
Danny Al-Gaaf
a5ce2461d6 rgw/rgw_user.h: move initialization in initialization list
Move initialization of some variables from constructor body to
the initialization list.

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-04-02 14:39:24 +02:00
Danny Al-Gaaf
9b5f8a4f9d rbd.cc: use static_cast instead of C-Style cast
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-04-02 14:24:28 +02:00
Danny Al-Gaaf
07be36c92b mds/Migrator.cc: prefer prefix ++operator for iterator
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-04-02 14:01:24 +02:00
Loic Dachary
aee654916b fix nspace assignment in LFNIndex::lfn_parse_object_name
The unused ns variable is assigned to the nspace data member of the
parsed hobject_t. It would be nicer to use a constructor accepting
the namespace in argument.

http://tracker.ceph.com/issues/4597 refs #4597

Signed-off-by: Loic Dachary <loic@dachary.org>
2013-04-02 13:07:24 +02:00
caleb miles
dd19d693e6 rgw: Create RESTful endpoint for user and bucket administration.
Expose the following operations through a RESTful endpoint:
    user create
    user modify
    user remove
    subuser create
    subuser modify
    subuser remove
    key create
    key remove
    bucket list
    bucket stats
    bucket link
    bucket unlink
    bucket check
    bucket remove
    remove object

building on the existing /{admin} endpoint.

Signed-off-by caleb miles <caleb.miles@inktank.com>
2013-04-01 21:44:45 -04:00
Sage Weil
aa14da20ed doc/release-notes: v0.60
Signed-off-by: Sage Weil <sage@inktank.com>
2013-04-01 18:17:27 -07:00
Gary Lowell
6ffadce67e Merge branch 'next' 2013-04-01 17:57:45 -07:00
athanatos
f861d54c17 Merge pull request #181 from ceph/wip_4510
Scrub/repair should correctly handle truncation and EIO

Fixes #4510
Reviewed-by: Greg Farnum <greg@inktank.com>
Reviewed-by: Sage Weil <sage@inktank.com>
2013-04-01 16:32:34 -07:00
Samuel Just
fc13f1111c PG::_scan_list: assert if error is neither -EIO nor -ENOENT
Signed-off-by: Samuel Just <sam.just@inktank.com>
2013-04-01 16:27:34 -07:00
Samuel Just
3fa3b676f9 FileStore: rename debug_delete_obj to debug_obj_on_delete
This should make the method intent less confusing.

Signed-off-by: Samuel Just <sam.just@inktank.com>
2013-04-01 16:27:34 -07:00
Samuel Just
40070cef3f PG: _scan_list can now handle EIO on read, stat, get_omap_header
Signed-off-by: Samuel Just <sam.just@inktank.com>
2013-04-01 16:27:34 -07:00