[PATCH]: [xfstests] test 032: Run mkfs.xfs only if the prior mkfs.${fs} succeded

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



This test fails when mkfs of a different filesystem fails, which is not
a failure case for this specific test.

Hence, change the test to run mkfs.xfs only if the prior mkfs.${fs} on 
different filesystem succeded.

Signed-Off-By: Chandra Seetharaman <sekharan@xxxxxxxxxx>
---

diff --git a/032 b/032
index 839b913..4261ca2 100755
--- a/032
+++ b/032
@@ -75,11 +75,15 @@ do
 	echo "    ( $preop mkfs -t $fs $preargs $SCRATCH_DEV $postargs )" >>$seq.full
 	eval $preop mkfs -t $fs $preargs $SCRATCH_DEV $postargs >>$seq.full 2>&1
 
-	# next, ensure we don't overwrite it
-	echo "=== Attempting XFS overwrite of $fs..." >>$seq.full
-	/sbin/mkfs.xfs $SCRATCH_DEV >>$seq.full 2>&1
+	if [ $? -eq 0 ] ; then
+		# next, ensure we don't overwrite it
+		echo "=== Attempting XFS overwrite of $fs..." >>$seq.full
+		/sbin/mkfs.xfs $SCRATCH_DEV >>$seq.full 2>&1
 
-	[ $? -eq 0 ] && echo "Failed - overwrote fs type ${fs}!"
+		[ $? -eq 0 ] && echo "Failed - overwrote fs type ${fs}!"
+	else
+		echo "mkfs of type ${fs} failed" >>$seq.full
+	fi
 done
 
 # success, all done


_______________________________________________
xfs mailing list
xfs@xxxxxxxxxxx
http://oss.sgi.com/mailman/listinfo/xfs


[Index of Archives]     [Linux XFS Devel]     [Linux Filesystem Development]     [Filesystem Testing]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux