summaryrefslogtreecommitdiff
path: root/tests/11spare-migration
diff options
context:
space:
mode:
authorCzarnowska, Anna <anna.czarnowska@intel.com>2011-02-08 11:44:50 +1100
committerNeilBrown <neilb@suse.de>2011-02-08 11:44:50 +1100
commitf6539edf2e8322de3bdf88e2e6354411dd24d229 (patch)
tree6b764f3ef5dc6824668b8df0d811571aa07236b3 /tests/11spare-migration
parent21f3b9cc44091f05d0d55a9c9afb64d73068d04b (diff)
Spare migration tests updated
Added tests for cases when: 0 - there is no config file at all 0a - config file has no domains defined 9a - spare is in global domain 15 - spare is in global domain for $platform metadata Test 4 pass condition changed for imsm metadata. disk_policy only adds controller domain for imsm when config_rules_has_path=1. If there are any domains in config then every disk will have also a controller domain. As a result the array needing spares has at least one controller domain even when it doesn't have any other domains specified explicitly. In this case any spare in the same controller domain matches when it has no other domains from config. (This behaviour is different from the case when there is no paths in config at all as array domain then is null and no spare matches) Signed-off-by: Anna Czarnowska <anna.czarnowska@intel.com> Signed-off-by: NeilBrown <neilb@suse.de>
Diffstat (limited to 'tests/11spare-migration')
-rw-r--r--tests/11spare-migration72
1 files changed, 69 insertions, 3 deletions
diff --git a/tests/11spare-migration b/tests/11spare-migration
index c8ef3625..3567883f 100644
--- a/tests/11spare-migration
+++ b/tests/11spare-migration
@@ -54,6 +54,33 @@ monitor(){
[ "$verbose" != "yes" ] || echo $mddevs $monitorpid
}
+test0()
+{
+dsc "Test 0: No config file, no spare should be moved"
+setupdevs 0 0 1 $platform
+setupdevs 1 3 4 $platform
+monitor 0 1
+mdadm -a /dev/$c1 $dev2
+mdadm --fail /dev/$v0 $dev0
+# check that spare loop2 was not moved from container c1 to container c0
+chksparemoved $c1 $c0 $dev2 n
+tidyup
+}
+
+test0a()
+{
+dsc "Test 0a: No domains in config file, no spare should be moved"
+setupdevs 0 0 1 $platform
+setupdevs 1 3 4 $platform
+createconfig a
+monitor 0 1
+mdadm -a /dev/$c1 $dev2
+mdadm --fail /dev/$v0 $dev0
+# check that spare loop2 was not moved from container c1 to container c0
+chksparemoved $c1 $c0 $dev2 n
+tidyup
+}
+
test1()
{
dsc "Test 1: Common domain, add disk to one container and fail first one in another container, spare should be moved"
@@ -117,7 +144,7 @@ tidyup
test4()
{
-dsc "Test 4: One domain holds one container, fail a disk in domain, and add disk to a container not described by domain, spare loop5 should not be moved"
+dsc "Test 4: One domain holds one container, fail a disk in domain, and add disk to a container not described by domain, move if metadata allows"
setupdevs 0 0 1 $platform
setupdevs 1 3 4 $platform
createconfig a
@@ -125,7 +152,9 @@ createconfig domain-$platform $platform spare 0 1 2
monitor 0 1
mdadm --fail /dev/$v0 $dev1
mdadm -a /dev/$c1 $dev5
-chksparemoved $c1 $c0 $dev5 n
+unset shouldmove
+[ "$platform" == "imsm" ] || shouldmove="n"
+chksparemoved $c1 $c0 $dev5 $shouldmove
tidyup
}
@@ -236,7 +265,7 @@ tidyup
test9()
{
-dsc "Test 9: imsm and native 1.2 - spare should be shared"
+dsc "Test 9: imsm and native 1.2 - one domain, no metadata specified, spare should be moved"
setupdevs 0 10 11 imsm
setupdevs 1 8 9 1.2
createconfig a
@@ -248,6 +277,22 @@ chksparemoved $c1 $c0 $dev12
tidyup
}
+test9a()
+{
+dsc "Test 9a: imsm and native 1.2 - spare in global domain, should be moved"
+setupdevs 0 10 11 imsm
+setupdevs 1 8 9 1.2
+createconfig a
+createconfig domain-global noplatform spare 8 9 10 11 12
+createconfig domain-1.2 1.2 spare 8 9
+createconfig domain-imsm imsm spare 10 11
+monitor 0 1
+mdadm -a /dev/$c1 $dev12
+mdadm --fail /dev/$v0 $dev10
+chksparemoved $c1 $c0 $dev12
+tidyup
+}
+
test10()
{
dsc "Test 10: Two arrays on the same devices in container"
@@ -335,8 +380,27 @@ chksparemoved $c1 $c0 $d2
tidyup
}
+test15()
+{
+dsc "Test 15: spare in global domain for $platform metadata, should be moved"
+# this is like 9a but only one metadata used
+setupdevs 0 10 11 $platform
+setupdevs 1 8 9 $platform
+createconfig a
+createconfig domain-global $platform spare 8 9 10 11 12
+createconfig domain-1 $platform spare 8 9
+createconfig domain-2 $platform spare 10 11
+monitor 0 1
+mdadm -a /dev/$c1 $dev12
+mdadm --fail /dev/$v0 $dev10
+chksparemoved $c1 $c0 $dev12
+tidyup
+}
+
try()
{
+test0
+test0a
test1
test1a
test2
@@ -351,6 +415,7 @@ if [ "$platform" != "1.2" ]; then
fi
test8
test9
+test9a
if [ "$platform" != "1.2" ]; then
# we can't create two subarrays on the same devices for native (without
# partitions)
@@ -360,6 +425,7 @@ test11
test12
test13
test14
+test15
}
try_failed()