[PATCH 2/5] tests: clear the superblock before adding a device to the array.

Subsystems: the rest

STALE1843d

2 messages, 2 authors, 2021-08-02 · open the first message on its own page

[PATCH 2/5] tests: clear the superblock before adding a device to the array.

From: Sudhakar Panneerselvam <hidden>
Date: 2021-07-22 18:28:13

This fixes '02lineargrow' test as prior metadata causes --add operation
to fail.

Signed-off-by: Sudhakar Panneerselvam <redacted>
---
 test | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/test b/test
index 711a3c7a2076..39a85d77fa25 100755
--- a/test
+++ b/test
@@ -48,7 +48,7 @@ mdadm() {
 		;;
 	esac
 	case $* in
-	*-C* | *--create* | *-B* | *--build* )
+	*-C* | *--create* | *-B* | *--build* | *--add* )
 		# clear superblock every time once creating or
 		# building arrays, because it's always creating
 		# and building array many times in a test case.
@@ -59,7 +59,12 @@ mdadm() {
 					$mdadm --zero $args > /dev/null
 			}
 		done
-		$mdadm 2> $targetdir/stderr --quiet "$@" --auto=yes
+		if [[ $* == *--add* ]]
+		then
+			$mdadm 2> $targetdir/stderr --quiet "$@"
+		else
+			$mdadm 2> $targetdir/stderr --quiet "$@" --auto=yes
+		fi
 		;;
 	* )
 		$mdadm 2> $targetdir/stderr --quiet "$@"
-- 
1.8.3.1

Re: [PATCH 2/5] tests: clear the superblock before adding a device to the array.

From: Jes Sorensen <hidden>
Date: 2021-08-02 14:46:24

On 7/22/21 2:28 PM, Sudhakar Panneerselvam wrote:
quoted hunk
This fixes '02lineargrow' test as prior metadata causes --add operation
to fail.

Signed-off-by: Sudhakar Panneerselvam <redacted>
---
 test | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/test b/test
index 711a3c7a2076..39a85d77fa25 100755
--- a/test
+++ b/test
@@ -48,7 +48,7 @@ mdadm() {
 		;;
 	esac
 	case $* in
-	*-C* | *--create* | *-B* | *--build* )
+	*-C* | *--create* | *-B* | *--build* | *--add* )
 		# clear superblock every time once creating or
 		# building arrays, because it's always creating
 		# and building array many times in a test case.
@@ -59,7 +59,12 @@ mdadm() {
 					$mdadm --zero $args > /dev/null
 			}
 		done
-		$mdadm 2> $targetdir/stderr --quiet "$@" --auto=yes
+		if [[ $* == *--add* ]]
+		then
+			$mdadm 2> $targetdir/stderr --quiet "$@"
+		else
+			$mdadm 2> $targetdir/stderr --quiet "$@" --auto=yes
+		fi
 		;;
 	* )
 		$mdadm 2> $targetdir/stderr --quiet "$@"
I am not sure this is the right approach to fix this. --add has two
meanings and we would potentially want test cases where we make sure not
to add arrays with pre-existing meta-data on them. I think it's better
to fix this in 02lineargrow

Thanks,
Jes
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help