From: Loic Dachary <ldachary@redhat.com>
Date: Wed, 1 Apr 2015 13:01:25 +0000 (+0200)
Subject: tests: ceph-disk.sh dmcrypt tests use test_setup_dev_and_run
X-Git-Tag: v9.0.0~44^2^2~2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=8db870edf1db43d25de9100138db1cd29d7f2227;p=ceph.git

tests: ceph-disk.sh dmcrypt tests use test_setup_dev_and_run

Instead of duplicating the device construction / destruction logic for
dmcrypt tests, use test_setup_dev_and_run to do it. It is now able to
recover from devmapper leftover which may occur when a cryptsetup test
fails.

Signed-off-by: Loic Dachary <ldachary@redhat.com>
---

diff --git a/src/test/ceph-disk.sh b/src/test/ceph-disk.sh
index 326c65da35fa..c9f7169639a1 100755
--- a/src/test/ceph-disk.sh
+++ b/src/test/ceph-disk.sh
@@ -482,20 +482,6 @@ function test_setup_dev_and_run() {
     return $status
 }
 
-function destroy_dmcrypt_dev() {
-    local name=$1
-    local dev=$2
-    local uuid=$3
-
-    for partition in 1 2 3 4 ; do
-        umount /dev/mapper/$uuid || true
-	/sbin/cryptsetup remove /dev/mapper/$uuid || true
-	dmsetup remove /dev/mapper/$uuid || true
-    done
-    losetup --detach $dev
-    rm $name
-}
-
 function activate_dmcrypt_dev_body() {
     local disk=$1
     local journal=$2
@@ -512,26 +498,7 @@ function activate_dmcrypt_dev_body() {
 }
 
 function test_activate_dmcrypt_dev() {
-    if test $(id -u) != 0 ; then
-        echo "SKIP because not root"
-        return 0
-    fi
-
-    loop_sanity_check || return 1
-
-    local disk=$(create_dev vdf.disk)
-    local journal=$(create_dev vdg.disk)
-    local newdisk=$(create_dev vdh.disk)
-
-    activate_dmcrypt_dev_body $disk $journal $newdisk
-    status=$?
-    test $status != 0 && teardown
-
-    destroy_dmcrypt_dev vdf.disk $disk
-    destroy_dmcrypt_dev vdg.disk $journal
-    destroy_dmcrypt_dev vdh.disk $newdisk
-
-    return $status
+    test_setup_dev_and_run activate_dmcrypt_dev_body
 }
 
 function activate_dmcrypt_plain_dev_body() {
@@ -550,23 +517,7 @@ function activate_dmcrypt_plain_dev_body() {
 }
 
 function test_activate_dmcrypt_plain_dev() {
-    if test $(id -u) != 0 ; then
-        echo "SKIP because not root"
-        return 0
-    fi
-
-    local disk=$(create_dev vdf.disk)
-    local journal=$(create_dev vdg.disk)
-    local newdisk=$(create_dev vdh.disk)
-
-    activate_dmcrypt_plain_dev_body $disk $journal $newdisk
-    status=$?
-
-    destroy_dmcrypt_dev vdf.disk $disk
-    destroy_dmcrypt_dev vdg.disk $journal
-    destroy_dmcrypt_dev vdh.disk $newdisk
-
-    return $status
+    test_setup_dev_and_run activate_dmcrypt_plain_dev_body
 }
 
 function test_find_cluster_by_uuid() {