]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
tests: do not ignore ceph-disk.sh::test_activate_dev failure 3152/head
authorLoic Dachary <ldachary@redhat.com>
Thu, 11 Dec 2014 17:33:31 +0000 (18:33 +0100)
committerLoic Dachary <ldachary@redhat.com>
Thu, 11 Dec 2014 17:35:13 +0000 (18:35 +0100)
The status of the test commands must be tested instead of being
silently ignored.

Signed-off-by: Loic Dachary <ldachary@redhat.com>
src/test/ceph-disk.sh

index be03011438f1146a15c40d4ddd08b79121a95714..1ed38cb5b8d69c84db1c1431ea98c306bc07ad97 100755 (executable)
@@ -231,6 +231,16 @@ function test_activate_dir() {
     $rm -fr $osd_data
 }
 
+function activate_dev_body() {
+    local disk=$1
+
+    ./ceph-disk zap $disk || return 1
+    test_activate ${disk} ${disk}p1 || return 1
+    kill_daemons
+    umount ${disk}p1 || return 1
+    ./ceph-disk zap $disk || return 1
+}
+
 function test_activate_dev() {
     run_mon
 
@@ -242,11 +252,7 @@ function test_activate_dev() {
     dd if=/dev/zero of=vde.disk bs=1024k count=200
     losetup --find vde.disk
     local disk=$(losetup --associated vde.disk | cut -f1 -d:)
-    ./ceph-disk zap $disk
-    test_activate ${disk} ${disk}p1
-    kill_daemons
-    umount ${disk}p1
-    ./ceph-disk zap $disk
+    activate_dev_body $disk
     status=$?
     losetup --detach $disk
     rm vde.disk