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

[Jes Sorensen <jes@trained-monkey.org>]

On 7/22/21 2:28 PM, Sudhakar Panneerselvam wrote:
> This fixes '02lineargrow' test as prior metadata causes --add operation
> to fail.
> 
> Signed-off-by: Sudhakar Panneerselvam <sudhakar.panneerselvam@oracle.com>
> ---
>  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