[lvm2-commits] main - tests: use aux clear_dev

[Zdenek Kabelac]

Gitweb:       
https://sourceware.org/git/?p=lvm2.git;a=commitdiff;h=b2360437e027fc4735a...
Commit:        b2360437e027fc4735afb58d193dc53b61c592d9
Parent:        c331a3b1da280f78aefcfc8dad82eb042022bc84
Author:        Zdenek Kabelac <zkabelac(a)redhat.com&gt;
AuthorDate:    Fri Apr 21 13:16:22 2023 +0200
Committer:     Zdenek Kabelac <zkabelac(a)redhat.com&gt;
CommitterDate: Fri Apr 21 14:52:43 2023 +0200

tests: use aux clear_dev

Use wrapper over 'blkdiscard -z'
---
 test/shell/integrity-caching.sh          |  6 +-----
 test/shell/integrity-syncaction.sh       |  4 +---
 test/shell/integrity.sh                  |  6 +-----
 test/shell/metadata-bad-text.sh          | 21 +++++----------------
 test/shell/metadata-old.sh               |  7 ++-----
 test/shell/process-each-duplicate-pvs.sh |  3 +--
 6 files changed, 11 insertions(+), 36 deletions(-)

diff --git a/test/shell/integrity-caching.sh b/test/shell/integrity-caching.sh
index 5539ac575..23ade6838 100644
--- a/test/shell/integrity-caching.sh
+++ b/test/shell/integrity-caching.sh
@@ -39,11 +39,7 @@ dd if=/dev/urandom of=randC bs=512K count=4
 _prepare_vg() {
 	# zero devs so we are sure to find the correct file data
 	# on the underlying devs when corrupting it
-	dd if=/dev/zero of="$dev1" bs=1M oflag=direct || true
-	dd if=/dev/zero of="$dev2" bs=1M oflag=direct || true
-	dd if=/dev/zero of="$dev3" bs=1M oflag=direct || true
-	dd if=/dev/zero of="$dev4" bs=1M oflag=direct || true
-	dd if=/dev/zero of="$dev5" bs=1M oflag=direct || true
+	aux clear_devs "$dev1" "$dev2" "$dev3" "$dev4"
"$dev5"
 	vgcreate $SHARED $vg "$dev1" "$dev2" "$dev3"
"$dev4" "$dev5" "$dev6"
 	pvs
 }
diff --git a/test/shell/integrity-syncaction.sh b/test/shell/integrity-syncaction.sh
index e404f6b6d..32d7a43a5 100644
--- a/test/shell/integrity-syncaction.sh
+++ b/test/shell/integrity-syncaction.sh
@@ -34,9 +34,7 @@ awk 'BEGIN { while (z++ < 16384) printf "C" }' >
fileC
 _prepare_vg() {
 	# zero devs so we are sure to find the correct file data
 	# on the underlying devs when corrupting it
-	dd if=/dev/zero of="$dev1" bs=1M oflag=direct || true
-	dd if=/dev/zero of="$dev2" bs=1M oflag=direct || true
-	dd if=/dev/zero of="$dev3" bs=1M oflag=direct || true
+	aux clear_devs "$dev1" "$dev2" "$dev3"
 	vgcreate $SHARED $vg "$dev1" "$dev2" "$dev3"
 	pvs
 }
diff --git a/test/shell/integrity.sh b/test/shell/integrity.sh
index a7dd5b565..290090e2d 100644
--- a/test/shell/integrity.sh
+++ b/test/shell/integrity.sh
@@ -39,11 +39,7 @@ dd if=/dev/urandom of=randC bs=512K count=4
 _prepare_vg() {
 	# zero devs so we are sure to find the correct file data
 	# on the underlying devs when corrupting it
-	dd if=/dev/zero of="$dev1" bs=1M oflag=direct || true
-	dd if=/dev/zero of="$dev2" bs=1M oflag=direct || true
-	dd if=/dev/zero of="$dev3" bs=1M oflag=direct || true
-	dd if=/dev/zero of="$dev4" bs=1M oflag=direct || true
-	dd if=/dev/zero of="$dev5" bs=1M oflag=direct || true
+	aux clear_devs "$dev1" "$dev2" "$dev3" "$dev4"
"$dev5"
 	vgcreate $SHARED $vg "$dev1" "$dev2" "$dev3"
"$dev4" "$dev5"
 	pvs
 }
diff --git a/test/shell/metadata-bad-text.sh b/test/shell/metadata-bad-text.sh
index 17f79eb59..e51268823 100644
--- a/test/shell/metadata-bad-text.sh
+++ b/test/shell/metadata-bad-text.sh
@@ -29,9 +29,7 @@ _clear_online_files() {
 aux prepare_devs 3
 get_devs
 
-dd if=/dev/zero of="$dev1" || true
-dd if=/dev/zero of="$dev2" || true
-dd if=/dev/zero of="$dev3" || true
+aux clear_devs "$dev1" "$dev2" "$dev3"
 
 vgcreate $SHARED $vg "$dev1" "$dev2" "$dev3"
 
@@ -74,9 +72,7 @@ vgremove -ff $vg
 # copy of the metadata.
 #
 
-dd if=/dev/zero of="$dev1" || true
-dd if=/dev/zero of="$dev2" || true
-dd if=/dev/zero of="$dev3" || true
+aux clear_devs "$dev1" "$dev2" "$dev3"
 
 vgcreate $SHARED $vg "$dev1" "$dev2" "$dev3"
 
@@ -125,9 +121,7 @@ vgremove -ff $vg
 # makes the VG usable.
 #
 
-dd if=/dev/zero of="$dev1" || true
-dd if=/dev/zero of="$dev2" || true
-dd if=/dev/zero of="$dev3" || true
+aux clear_devs "$dev1" "$dev2" "$dev3"
 
 pvcreate "$dev1"
 pvcreate "$dev2"
@@ -182,9 +176,7 @@ vgremove -ff $vg
 # devices.
 #
 
-dd if=/dev/zero of="$dev1" || true
-dd if=/dev/zero of="$dev2" || true
-dd if=/dev/zero of="$dev3" || true
+aux clear_devs "$dev1" "$dev2" "$dev3"
 
 pvcreate "$dev1"
 pvcreate "$dev2"
@@ -258,9 +250,7 @@ if test -n "$LVM_TEST_LVMLOCKD"; then
 exit 0
 fi
 
-dd if=/dev/zero of="$dev1" || true
-dd if=/dev/zero of="$dev2" || true
-dd if=/dev/zero of="$dev3" || true
+aux clear_devs "$dev1" "$dev2" "$dev3"
 
 vgcreate $SHARED $vg "$dev1" "$dev2" "$dev3"
 
@@ -328,4 +318,3 @@ pvs "$dev3"
 
 vgchange -an $vg
 vgremove -ff $vg
-
diff --git a/test/shell/metadata-old.sh b/test/shell/metadata-old.sh
index ad7bf101e..9a0351c23 100644
--- a/test/shell/metadata-old.sh
+++ b/test/shell/metadata-old.sh
@@ -118,9 +118,7 @@ vgremove -ff $vg
 # metadata, and the second mda has current metadata.
 #
 
-dd if=/dev/zero of="$dev1" || true
-dd if=/dev/zero of="$dev2" || true
-dd if=/dev/zero of="$dev3" || true
+aux clear_devs "$dev1" "$dev2" "$dev3"
 
 pvcreate "$dev1"
 pvcreate "$dev2"
@@ -181,8 +179,7 @@ vgremove -ff $vg
 # Test when the metadata on two PVs have the same seqno
 # but different checksums.
 
-dd if=/dev/zero of="$dev1" || true
-dd if=/dev/zero of="$dev2" || true
+aux clear_devs "$dev1" "$dev2"
 
 pvcreate "$dev1"
 pvcreate "$dev2"
diff --git a/test/shell/process-each-duplicate-pvs.sh
b/test/shell/process-each-duplicate-pvs.sh
index bf86c141f..f3063ab39 100644
--- a/test/shell/process-each-duplicate-pvs.sh
+++ b/test/shell/process-each-duplicate-pvs.sh
@@ -383,8 +383,7 @@ check pv_field "$dev4" dev_size "$SIZE4"
 # Verify that devs being used by an active LV are
 # preferred over duplicates that are not used by an LV.
 
-dd if=/dev/zero of="$dev3" bs=1M oflag=direct,sync || true
-dd if=/dev/zero of="$dev4" bs=1M oflag=direct,sync || true
+aux clear_devs "$dev3" "$dev4"
 #pvscan --cache
 
 # The previous steps prevent us from nicely cleaning up