Commit Graph

24798 Commits

Author SHA1 Message Date
Danny Al-Gaaf
89ffc64043 mds/CInode.h: prefer prefix ++operator for iterators
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-03-13 17:17:28 +01:00
Danny Al-Gaaf
bd4f1a3705 msg/Messenger.h: prefer prefix ++operator for iterators
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-03-13 16:59:40 +01:00
Danny Al-Gaaf
95749b3821 MonMap.h: prefer prefix ++operator for iterators
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-03-13 16:58:40 +01:00
Danny Al-Gaaf
6ae03459ee buffer.h: prefer prefix ++operator for iterators
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-03-13 16:58:05 +01:00
Danny Al-Gaaf
5d401b0a9e types.h: prefer prefix ++operator for iterators
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-03-13 16:47:32 +01:00
Danny Al-Gaaf
81e00c14e9 auth/Crypto.cc prefer prefix ++operator for iterators
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-03-13 16:40:44 +01:00
John Wilkins
af3b16349a doc: Fixed syntax error.
Signed-off-by: John Wilkins <john.wilkins@inktank.com>
2013-03-12 17:50:26 -07:00
John Wilkins
be6f7ce68f doc: Added tcp and bind settings and cleaned up syntax a bit.
Signed-off-by: John Wilkins <john.wilkins@inktank.com>
2013-03-12 17:44:10 -07:00
John Wilkins
77ecdc7260 doc: Setting should use network not single IP.
Signed-off-by: John Wilkins <john.wilkins@inktank.com>
2013-03-12 15:43:33 -07:00
John Wilkins
669d2ba552 doc: Fixed usage and added note for copying keyring to client. Added hyperlinks.
fixes: #4373

Signed-off-by: John Wilkins <john.wilkins@inktank.com>
2013-03-12 14:39:46 -07:00
John Wilkins
7f66ebe9a5 doc: Addressed confusion with version numbering.
fixes: #4411

Signed-off-by: John Wilkins <john.wilkins@inktank.com>
2013-03-12 14:25:44 -07:00
John Wilkins
4df44bb1bb doc: Added troubleshooting entry for single OSD and single replica.
fixes: #3327

Signed-off-by: John Wilkins <john.wilkins@inktank.com>
2013-03-12 14:15:51 -07:00
David Zafman
9464b5e965 Merge remote-tracking branch 'origin/wip-fix-librados-headers' 2013-03-12 12:03:43 -07:00
Gary Lowell
a5156fb74c Merge pull request #97 from Elbandi/master
Fix FTBFS

Reviewed-by: Gary Lowell <gary.lowell@inktank.com>
2013-03-12 11:15:00 -07:00
David Zafman
36c672dcb2 Fix typo in ceph usage output
Signed-off-by: David Zafman <david.zafman@inktank.com>
2013-03-12 11:07:54 -07:00
Sage Weil
6e2b33b460 Merge pull request #103 from dalgaaf/wip-da-fix-c_str
mon/Monitor.h: return string instead of 'char *' from get_state_name()

Reviewed-by: Sage Weil <sage@inktank.com>
2013-03-12 11:00:08 -07:00
Danny Al-Gaaf
60639ab67e mon/Monitor.h: return string instead of 'char *' from get_state_name()
Return a string instead of 'char *' to avoid usage of std::string:c_str()
to return a 'char *' from get_state_name().

Returning result of c_str() from a function is dangerous since the
result gets (may) invalid after the related string object gets
destroyed or out of scope (which is the case with return). So you may
end up with garbage in this case.

Related warning from cppcheck:
 [src/mon/Monitor.h:172]: (error) Dangerous usage of c_str(). The value
 returned by c_str() is invalid after this call.

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-03-12 18:21:40 +01:00
Sage Weil
34f198487f Merge pull request #102 from dalgaaf/wip-da-sca-style-4-v2
Fixes for more cppcheck issues (v2 of #101)

Reviewed-by: Sage Weil <sage@inktank.com>
2013-03-12 09:57:44 -07:00
Danny Al-Gaaf
72c62269f0 client/hadoop/CephFSInterface.cc: prefer prefix ++operator for iterators
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-03-12 16:59:45 +01:00
Danny Al-Gaaf
5ad24601ff cls/rgw/cls_rgw.cc: prefer prefix --operator for iterators
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-03-12 16:59:44 +01:00
Danny Al-Gaaf
7a310854f2 client/Inode.cc: prefer prefix ++operator for iterators
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-03-12 16:59:44 +01:00
Danny Al-Gaaf
822c67ef84 SyntheticClient.h: prefer prefix ++operator for iterators
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-03-12 16:59:44 +01:00
Danny Al-Gaaf
0f48638ee4 CephxKeyServer.h: prefer prefix ++operator for iterators
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-03-12 16:59:44 +01:00
Danny Al-Gaaf
b1437a8bf9 auth/Auth.h: prefer prefix ++operator for non-primitive types
Prefer prefix ++operator for non-primitive types like iterators for
performance reasons. Prefix ++/-- operators avoid creating a temporary
copy.

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-03-12 16:59:44 +01:00
Danny Al-Gaaf
64dca11e4a tools/rest_bench.cc: remove unused variable
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-03-12 16:59:44 +01:00
Danny Al-Gaaf
9b51c25fa9 test/test_filejournal.cc: use %u to format unsigned in sprintf()
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-03-12 16:59:44 +01:00
Danny Al-Gaaf
5e882db521 test/system/st_rados_list_objects.cc: remove not needed statement
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-03-12 16:59:44 +01:00
Danny Al-Gaaf
ae15b2fcaa test/osd/RadosModel.h: remove unused variables
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-03-12 16:59:44 +01:00
Danny Al-Gaaf
faab0e849a test/osd/Object.cc: catch exception by reference
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-03-12 16:59:43 +01:00
Danny Al-Gaaf
4762faccb7 test/omap_bench.cc: remove unused variables
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-03-12 16:59:43 +01:00
Danny Al-Gaaf
b77454c61d test_mon_workloadgen.cc: remove unused variables
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-03-12 16:59:43 +01:00
Danny Al-Gaaf
3cb9b35644 test/librados/aio.cc: use static_cast instead of C-Style cast
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-03-12 16:59:43 +01:00
Danny Al-Gaaf
a6bd6581c3 test/encoding/ceph_dencoder.cc: use static_cast instead of C-Style cast
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-03-12 16:59:43 +01:00
Danny Al-Gaaf
8dc0a3292d bench/dumb_backend.h: remove never executed delete statement
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-03-12 16:59:43 +01:00
Danny Al-Gaaf
3072902c00 test_object_map.cc: use %u to format unsigned in snprintf()
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-03-12 16:59:43 +01:00
Danny Al-Gaaf
a19a2a14d4 test_keyvaluedb_iterators.cc: remove unused variable
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-03-12 16:59:43 +01:00
Danny Al-Gaaf
08bf10b99b rgw/rgw_xml.cc: use static_cast instead of C-Style cast
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-03-12 16:59:42 +01:00
Danny Al-Gaaf
03e0308f3e rgw/rgw_user.cc: remove unused variables
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-03-12 16:59:42 +01:00
Danny Al-Gaaf
69cb92e37d rgw/rgw_rest_usage.cc: remove unused variables
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-03-12 16:59:42 +01:00
Danny Al-Gaaf
38942059d4 rgw/rgw_rest_s3.cc: remove unused variables
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-03-12 16:59:42 +01:00
Danny Al-Gaaf
f8a9fec87f rgw/rgw_rados.cc: perform initialization of prefix in initialization list
Perform initialization of IntentLogNameFilter:prefix in initialization list
instead of constructor body.

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-03-12 16:59:42 +01:00
Danny Al-Gaaf
a37d71dcdf rgw/rgw_rados.cc: remove unused variables
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-03-12 16:59:42 +01:00
Danny Al-Gaaf
c6becc6b58 rgw/rgw_rados.cc: use static_cast instead of C-Style cast
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
2013-03-12 16:59:42 +01:00
Joe Buck
579d4ad77d testing: update wordcount to new code layout.
Signed-off-by: Joe Buck <jbbuck@gmail.com>
2013-03-12 01:23:35 -07:00
Joe Buck
09fa711093 testing: fix hadoop-internal-tests
The hadoop-internal-tests workunit needs
to be updated in light of our moving to
using stock Hadoop with our hadoop-cephfs
jars.

Signed-off-by: Joe Buck <jbbuck@gmail.com>
2013-03-11 23:41:27 -07:00
Sage Weil
936b8f20af ceph-disk-activate: rely on default/configured keyring path
No reason to override the default or configured value here.

Signed-off-by: Sage Weil <sage@inktank.com>
2013-03-11 21:15:11 -07:00
Sage Weil
813e9fe2b4 ceph-disk-activate: don't override default or configured osd journal path
There is no reason not to rely on the default or obey any configured
value here.

Fixes: #4031
Signed-off-by: Sage Weil <sage@inktank.com>
2013-03-11 21:14:53 -07:00
Josh Durgin
82677c38b7 Merge branch 'next' 2013-03-11 16:59:05 -07:00
Josh Durgin
1ac8f6abaf Merge branch 'wip-rbd-flatten-cache' into next
Reviewed-by: Sage Weil <sage.weil@inktank.com>
2013-03-11 16:57:47 -07:00
Sage Weil
9eb0d91b86 debian: stop ceph-mds before uninstalling ceph-mds
Fixes: #4384
Signed-off-by: Sage Weil <sage@inktank.com>
2013-03-11 17:09:37 -07:00