summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Build: Change build to always use system leveldbwip-system-leveldbGary Lowell2013-02-265-20/+6
| | | | | | | | Dynamically link to the leveldb installed on the system rather than statically linking ceph copy. Remove the --with-system-leveldb config option, and add a requirement for leveldb libraries for rpm and debian packages. Bug 3945. Signed-off-by: Gary Lowell <gary.lowell@inktank.com>
* configure.ac: Add test for c++ compiler.Gary Lowell2013-02-251-0/+4
| | | | | | | | | | | The AC_PROG_CXX macro sets a flag if a C++ compiler is found but does not fail if one is not found, it left to application to test the flags as needed. This fix will issue an error when a c++ compiler is not found. Bug 3955. Signed-off-by: Gary Lowell <gary.lowell@inktank.com> it's not installed, this fix adds an error message for a
* test_lock_fence.sh, rbdrw.py: rbd lock/fence testDan Mick2013-02-253-1/+85
| | | | | | | | | | | | | | | qa/workunits/rbd/test_lock_fence.sh runs using test/rbdrw.py rbdrw.py creates an image, locks it, and runs an I/O loop; test_lock_fence.sh runs it, waits, and then blacklists that client, which causes rbdrw.py to get ESHUTDOWN on operations thereafter. Currently doesn't work with rbd caching enabled. rbd.py gets new exception type for ESHUTDOWN Fixes: #3190 Signed-off-by: Dan Mick <dan.mick@inktank.com> Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
* Merge remote-tracking branch 'gh/wip-4249-master'Sage Weil2013-02-253-27/+15
|\
| * librbd: remove unused internal methodJosh Durgin2013-02-252-11/+0
| | | | | | | | | | | | get_snap_size() has been replaced by get_image_size(snap_id) everywhere. Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
| * Merge branch 'wip-4249' into wip-4249-masterJosh Durgin2013-02-251-16/+15
| |\ | | | | | | | | | | | | | | | | | | Make snap_rollback() only take a read lock on snap_lock, since it does not modify snapshot-related fields. Conflicts: src/librbd/internal.cc
| | * librbd: drop snap_lock before invalidating cacheJosh Durgin2013-02-251-16/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Writeback will take the snap_lock, so read everything we need under it before invalidating the cache. This avoids a recursive lock when writeback uses snap_lock while snap_rollback() was holding it. Remove a not-very-useful debugging message that depended on snap_lock being held. Fixes: #4249 Backport: bobtail Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
* | | Merge remote-tracking branch 'gh/wip-4252'Sage Weil2013-02-252-7/+7
|\ \ \
| * | | test_rbd: move flatten tests back into TestCloneJosh Durgin2013-02-251-6/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | They need the same setup, and it's easy enough to run specific subtests. Making them a separate subclass accidentally duplicated tests from TestClone. Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
| * | | qa: enable watch-notify dependent testJosh Durgin2013-02-251-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | This works now that watch-notify has been reworked a bit. Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
| * | | test_rbd: close image before removing itJosh Durgin2013-02-251-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This error was masked before by watch notify not differentiating between watches from the same client with different cookies. Reopen the image at the end of this test so teardown works. Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
* | | | mon: PaxosService: remove lingering uses of paxos getters and wait methodsSage Weil2013-02-255-16/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We should use the PaxosServices getters, setters, and wait methods when and wherever possible. These must have fallen through the cracks during the merge. Signed-off-by: Joao Eduardo Luis <joao.luis@inktank.com>
* | | | Merge remote-tracking branch 'gh/wip-4147'Sage Weil2013-02-254-14/+20
|\ \ \ \
| * | | | systest: restrict list error acceptanceJosh Durgin2013-02-251-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Only ignore errors after the midway point if the midway_sem_post is defined. Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
| * | | | systest: fix race with pool deletionJosh Durgin2013-02-254-13/+19
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The second test have pool deletion and object listing wait on the same semaphore to connect and start. This led to errors sometimes when the pool was deleted before it could be opened by the listing process. Add another semaphore so the pool deletion happens only after the listing has begun. Fixes: #4147 Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
* | | | doc: Added subnet example and verbiage to network settings.John Wilkins2013-02-251-19/+58
| | | | | | | | | | | | | | | | | | | | | | | | fixes: #4049 Signed-off-by: John Wilkins <john.wilkins@inktank.com>
* | | | doc: Added content to remove REJECT rules from iptables.John Wilkins2013-02-251-1/+22
| | | | | | | | | | | | | | | | Signed-off-by: John Wilkins <john.wilkins@inktank.com>
* | | | doc: Added a small ref section for osd config reference.John Wilkins2013-02-251-0/+20
| | | | | | | | | | | | | | | | Signed-off-by: John Wilkins <john.wilkins@inktank.com>
* | | | doc: Updated osd configuration reference.John Wilkins2013-02-251-168/+457
|/ / / | | | | | | | | | Signed-off-by: John Wilkins <john.wilkins@inktank.com>
* | | qa: output rados test names as they runJosh Durgin2013-02-251-1/+1
|/ / | | | | | | | | | | | | So we don't have to figure out which test is running from the output, which can be difficult with the system tests. Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
* | Merge pull request #71 from dalgaaf/wip-da-sca-c_strSage Weil2013-02-252-3/+3
|\ \ | | | | | | | | | | | | fix some c_str() usage Reviewed-by: Sage Weil <sage@inktank.com>
| * | Client.cc: don't pass c_str() if std::string is expectedDanny Al-Gaaf2013-02-251-1/+1
| | | | | | | | | | | | | | | | | | | | | Don't pass c_str() to _lookup(). The function expect a std::string as second parameter. Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
| * | Paxos.h: fix dangerouse use of c_str()Danny Al-Gaaf2013-02-251-2/+2
| | | | | | | | | | | | | | | | | | | | | No need to use c_str() in get_statename(), simply return a std::strin instead. Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
* | | Merge pull request #72 from dalgaaf/wip-da-comp-sign-unsignSage Weil2013-02-251-1/+1
|\ \ \ | | | | | | | | | | | | | | | | Monitor.cc: fix -Wsign- Reviewed-by: Sage Weil <sage@inktank.com>
| * | | Monitor.cc: fix -Wsign-compareDanny Al-Gaaf2013-02-251-1/+1
| |/ / | | | | | | | | | | | | | | | Fix -Wsign-compare, make 'i' unsigned int. Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
* | | Merge pull request #70 from dalgaaf/wip-da-fix-specSage Weil2013-02-254-0/+6
|\ \ \ | | | | | | | | | | | | | | | | Add missing files to spec and debian files Reviewed-by: Sage Weil <sage@inktank.com>
| * | | debian: add new filesDanny Al-Gaaf2013-02-253-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | Add new (installed) files to debian install files. Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
| * | | ceph.spec.in: add new filesDanny Al-Gaaf2013-02-251-0/+3
| |/ / | | | | | | | | | | | | | | | Add new files to spec file since they get installed. Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
* | | doc: Moved admonition to kernel mount.John Wilkins2013-02-252-8/+14
| | | | | | | | | | | | | | | | | | fixes: #4146 Signed-off-by: John Wilkins <john.wilkins@inktank.com>
* | | doc: Added verbiage to describe single host deadlocks.John Wilkins2013-02-251-0/+35
|/ / | | | | | | | | | | fixes: #3076 Signed-off-by: John Wilkins <john.wilkins@inktank.com>
* | debian: make gdisk, parted requirements, not recommendations.Sage Weil2013-02-241-2/+2
| | | | | | | | | | | | ceph-prepare-disk (and thus ceph-deploy) need this. Signed-off-by: Sage Weil <sage@inktank.com>
* | Merge remote-tracking branch 'gh/next'Sage Weil2013-02-243-14/+21
|\ \ | |/
| * mds: reencode MDSMap in MMDSMap if MDSENC feature is not presentSage Weil2013-02-231-1/+2
| | | | | | | | | | | | | | | | | | | | | | In some cases the MMDSMap message from mon -> client passes from leader -> peon -> client, and the leader doesn't encode with the correct feature bits. As with MMOSDMap, we reencode the nested MDSMap based on the features if relevant bits are not present. We forgot to include this with the mds encoding changes. Signed-off-by: Sage Weil <sage@inktank.com>
| * qa/run_xfstests.sh: use $TESTDIR instead of /tmp/cephtestSage Weil2013-02-231-11/+12
| | | | | | | | Signed-off-by: Sage Weil <sage@inktank.com>
| * osd: an interval can't go readwrite if its acting is emptySage Weil2013-02-231-1/+2
| | | | | | | | | | | | | | | | Let's not forget that min_size can be zero. Fixes: #4159 Signed-off-by: Sage Weil <sage@inktank.com> (cherry picked from commit 4277265d99647c9fe950ba627e5d86234cfd70a9)
| * configuration parsing: give better error for missing =Dan Mick2013-02-211-2/+7
| | | | | | | | | | | | | | | | | | | | A ceph.conf line with "key" and no "= value" currently shows "unexpected character while parsing putative key value, at char N line M". There's no reason it can't be clearer. Fixes: #4229 Signed-off-by: Dan Mick <dan.mick@inktank.com> Reviewed-by: Sage Weil <sage@inktank.com>
* | Minor wording change.Neil Levine2013-02-241-1/+1
| | | | | | | | Signed-off-by: Neil Levine <neil.levine@inktank.com>
* | Grammar typoNeil Levine2013-02-241-1/+1
| | | | | | | | Signed-off-by: Neil Levine <neil.levine@inktank.com>
* | Changes to the OS support, multi-data center, and hypervisor questions.Neil Levine2013-02-241-23/+28
| | | | | | | | Signed-off-by: Neil Levine <neil.levine@inktank.com>
* | mkcephfs: create mon data dir prior to ceph-mon --mkfsSage Weil2013-02-221-1/+2
| | | | | | | | | | | | ceph-mon now expects this directory to already exist. Signed-off-by: Sage Weil <sage@inktank.com>
* | doc: Added a lot of info to OSD troubleshooting.John Wilkins2013-02-221-92/+457
| | | | | | | | Signed-off-by: John Wilkins <john.wilkins@inktank.com>
* | doc: Added mention of Admin Socket interface and brief description.John Wilkins2013-02-221-1/+21
| | | | | | | | Signed-off-by: John Wilkins <john.wilkins@inktank.com>
* | doc: Changed title to OSD and PG, indicating both subjects are covered.John Wilkins2013-02-221-1/+1
| | | | | | | | Signed-off-by: John Wilkins <john.wilkins@inktank.com>
* | doc: Added references from monitoring OSD to troubleshooting OSD.John Wilkins2013-02-221-7/+14
| | | | | | | | Signed-off-by: John Wilkins <john.wilkins@inktank.com>
* | doc: set maxdepth to 2, so TOC isn't so long with new OSD troubleshooting.John Wilkins2013-02-221-1/+2
| | | | | | | | Signed-off-by: John Wilkins <john.wilkins@inktank.com>
* | client: use 4MB f_bsize and f_frsize for statfsSage Weil2013-02-221-5/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Old stat(1) reports: Block size: 1048576 Fundamental block size: 1048576 and the df(1) arithmetic works out. New stat(1) reports: Block size: 1048576 Fundamental block size: 4096 which is what we are shoving into statvfs, but we have the b_size and fr_size arithmetic swapped. However, doing the *correct* reporting would then break the old stat by making both sizes appear to be 4KB (or whatever). Sidestep the issue by making *both* values 4MB.. which is both large enough to report large FS sizes, and also the default stripe size and thus a "reasonable" value to report for a block size. Perhaps in the future, when we no longer care about old userland, we can report the page size for f_bsize, which is probably the "most correct" thing to do. Fixes: #3794. See also #3793. Signed-off-by: Sage Weil <sage@inktank.com> Reviewed-by: Greg Farnum <greg@inktank.com>
* | test/librados/watch_notify: fix warningSage Weil2013-02-221-1/+1
| | | | | | | | | | | | | | | | | | | | In file included from test/librados/watch_notify.cc:8:0: ../src/gtest/include/gtest/gtest.h: In function ‘testing::AssertionResult testing::internal::CmpHelperEQ(const char*, const char*, const T1&, const T2&) [with T1 = long unsigned int, T2 = int]’: ../src/gtest/include/gtest/gtest.h:1300:30: instantiated from ‘static testing::AssertionResult testing::internal::EqHelper::Compare(const char*, const char*, const T1&, const T2&) [with T1 = long unsigned int, T2 = int, bool lhs_is_null_literal = false]’ test/librados/watch_notify.cc:67:224: instantiated from here warning: ../src/gtest/include/gtest/gtest.h:1263:3: comparison between signed and unsigned integer expressions [-Wsign-compare] Signed-off-by: Sage Weil <sage@inktank.com>
* | ceph-object-corpus: re-updateSage Weil2013-02-221-0/+0
| | | | | | | | | | | | | | | | This was set by 9af94eea209fc2555f66214f01f3edddc35d4209, then single paxos merge, then accidentally reverted by the next commit 6cb53740f2c356768adfbd3cb55c007d187309d3. Signed-off-by: Sage Weil <sage@inktank.com>
* | PG::proc_replica_log: oinfo.last_complete must be *before* first entry in ↵Samuel Just2013-02-221-4/+17
| | | | | | | | | | | | | | omissing Fixes: #4189 Signed-off-by: Samuel Just <sam.just@inktank.com>
* | Merge remote-tracking branch 'gh/wip-rbd-flatten-deadlock'Sage Weil2013-02-227-160/+167
|\ \ | | | | | | | | | Reviewed-by: Sage Weil <sage@inktank.com>
View Lorry Controller Status