From: Sudhakar Panneerselvam <sudhakar.panneerselvam@oracle.com>
To: jes@trained-monkey.org
Cc: linux-raid@vger.kernel.org
Subject: [PATCH 1/5] tests: remove raid0 tests for 0.90 metadata version
Date: Thu, 22 Jul 2021 18:27:33 +0000 [thread overview]
Message-ID: <20210722182733.GA24996@oracle.com> (raw)
Some of the test cases fail because raid0 creation fails with the error,
"0.90 metadata does not support layouts for RAID0" added by commit,
329dfc28debb. Removing the 0.90 test to make these test cases to work.
removed 04r0update test completely as it is specific to raid0 for 0.9
version.
'04update-metadata' and '03r0assem' still fails at a different place after
this change which will be fixed in an upcoming commit.
Signed-off-by: Sudhakar Panneerselvam <sudhakar.panneerselvam@oracle.com>
---
tests/00raid0 | 11 --------
tests/00readonly | 4 +++
tests/03r0assem | 72 -------------------------------------------------
tests/04r0update | 20 --------------
tests/04update-metadata | 2 +-
5 files changed, 5 insertions(+), 104 deletions(-)
delete mode 100644 tests/04r0update
diff --git a/tests/00raid0 b/tests/00raid0
index 8bc18985f91a..e0b6e325b4c9 100644
--- a/tests/00raid0
+++ b/tests/00raid0
@@ -6,12 +6,6 @@ check raid0
testdev $md0 3 $mdsize2_l 512
mdadm -S $md0
-# now with version-0.90 superblock
-mdadm -CR $md0 -e0.90 -l0 -n4 $dev0 $dev1 $dev2 $dev3
-check raid0
-testdev $md0 4 $mdsize0 512
-mdadm -S $md0
-
# now with no superblock
mdadm -B $md0 -l0 -n5 $dev0 $dev1 $dev2 $dev3 $dev4
check raid0
@@ -22,11 +16,6 @@ mdadm -S $md0
# now same again with different chunk size
for chunk in 4 32 256
do
- mdadm -CR $md0 -e0.90 -l raid0 --chunk $chunk -n3 $dev0 $dev1 $dev2
- check raid0
- testdev $md0 3 $mdsize0 $chunk
- mdadm -S $md0
-
# now with version-1 superblock
mdadm -CR $md0 -e1.0 -l0 -c $chunk -n4 $dev0 $dev1 $dev2 $dev3
check raid0
diff --git a/tests/00readonly b/tests/00readonly
index 28b0fa13f815..39202487f614 100644
--- a/tests/00readonly
+++ b/tests/00readonly
@@ -4,6 +4,10 @@ for metadata in 0.9 1.0 1.1 1.2
do
for level in linear raid0 raid1 raid4 raid5 raid6 raid10
do
+ if [[ $metadata == "0.9" && $level == "raid0" ]];
+ then
+ continue
+ fi
mdadm -CR $md0 -l $level -n 4 --metadata=$metadata \
$dev1 $dev2 $dev3 $dev4 --assume-clean
check nosync
diff --git a/tests/03r0assem b/tests/03r0assem
index 6744e3221062..9df7ba6a7613 100644
--- a/tests/03r0assem
+++ b/tests/03r0assem
@@ -63,75 +63,3 @@ echo " metadata=0.90 devices=$dev0,$dev1,$dev2" >> $conf
mdadm --assemble --scan --config=$conf $md2
$tst
mdadm -S $md2
-
-
-### Now for version 0...
-
-mdadm --zero-superblock $dev0 $dev1 $dev2
-mdadm -CR $md2 -l0 --metadata=0.90 -n3 $dev0 $dev1 $dev2
-check raid0
-tst="testdev $md2 3 $mdsize0 512"
-$tst
-
-uuid=`mdadm -Db $md2 | sed 's/.*UUID=//'`
-mdadm -S $md2
-
-mdadm -A $md2 $dev0 $dev1 $dev2
-$tst
-mdadm -S $md2
-
-mdadm -A $md2 -u $uuid $devlist
-$tst
-mdadm -S $md2
-
-mdadm --assemble $md2 --super-minor=2 $devlist #
-$tst
-mdadm -S $md2
-
-conf=$targetdir/mdadm.conf
-{
- echo DEVICE $devlist
- echo array $md2 UUID=$uuid
-} > $conf
-
-mdadm -As -c $conf $md2
-$tst
-mdadm -S $md2
-
-{
- echo DEVICE $devlist
- echo array $md2 super-minor=2
-} > $conf
-
-mdadm -As -c $conf $md2
-$tst
-mdadm -S $md2
-
-
-{
- echo DEVICE $devlist
- echo array $md2 devices=$dev0,$dev1,$dev2
-} > $conf
-
-mdadm -As -c $conf $md2
-$tst
-
-echo "DEVICE $devlist" > $conf
-mdadm -Db $md2 >> $conf
-mdadm -S $md2
-
-mdadm --assemble --scan --config=$conf $md2
-$tst
-mdadm -S $md2
-
-echo " metadata=1 devices=$dev0,$dev1,$dev2" >> $conf
-mdadm --assemble --scan --config=$conf $md2
-$tst
-mdadm -S $md2
-
-# Now use incremental assembly.
-mdadm -I --config=$conf $dev0
-mdadm -I --config=$conf $dev1
-mdadm -I --config=$conf $dev2
-$tst
-mdadm -S $md2
diff --git a/tests/04r0update b/tests/04r0update
deleted file mode 100644
index 73ee3b9fed91..000000000000
--- a/tests/04r0update
+++ /dev/null
@@ -1,20 +0,0 @@
-
-# create a raid0, re-assemble with a different super-minor
-mdadm -CR -e 0.90 $md0 -l0 -n3 $dev0 $dev1 $dev2
-testdev $md0 3 $mdsize0 512
-minor1=`mdadm -E $dev0 | sed -n -e 's/.*Preferred Minor : //p'`
-mdadm -S /dev/md0
-
-mdadm -A $md1 $dev0 $dev1 $dev2
-minor2=`mdadm -E $dev0 | sed -n -e 's/.*Preferred Minor : //p'`
-mdadm -S /dev/md1
-
-mdadm -A $md1 --update=super-minor $dev0 $dev1 $dev2
-minor3=`mdadm -E $dev0 | sed -n -e 's/.*Preferred Minor : //p'`
-mdadm -S /dev/md1
-
-case "$minor1 $minor2 $minor3" in
- "0 0 1" ) ;;
- * ) echo >&2 "ERROR minors should be '0 0 1' but are '$minor1 $minor2 $minor3'"
- exit 1
-esac
diff --git a/tests/04update-metadata b/tests/04update-metadata
index 232fc1ffff4b..08c14af7ed29 100644
--- a/tests/04update-metadata
+++ b/tests/04update-metadata
@@ -8,7 +8,7 @@ set -xe
dlist="$dev0 $dev1 $dev2 $dev3"
-for ls in raid0/4 linear/4 raid1/1 raid5/3 raid6/2
+for ls in linear/4 raid1/1 raid5/3 raid6/2
do
s=${ls#*/} l=${ls%/*}
mdadm -CR --assume-clean -e 0.90 $md0 --level $l -n 4 -c 64 $dlist
--
1.8.3.1
next reply other threads:[~2021-07-22 18:27 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-07-22 18:27 Sudhakar Panneerselvam [this message]
2021-08-02 14:34 ` [PATCH 1/5] tests: remove raid0 tests for 0.90 metadata version Jes Sorensen
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20210722182733.GA24996@oracle.com \
--to=sudhakar.panneerselvam@oracle.com \
--cc=jes@trained-monkey.org \
--cc=linux-raid@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox