diff options
| author | NeilBrown <neilb@suse.de> | 2009-04-01 13:51:57 +1100 |
|---|---|---|
| committer | NeilBrown <neilb@suse.de> | 2009-04-01 13:51:57 +1100 |
| commit | 37353437daabcf05eb389782f61b90ae181b8979 (patch) | |
| tree | 303ea43379c4e107e81a325cdc900bfb382e62fe /tests/env-09imsm-create-fail-rebuild | |
| parent | dd4ae23ad0f65ab28b812a75ef3a9923c70f78a2 (diff) | |
| download | mdadm-37353437daabcf05eb389782f61b90ae181b8979.tar.gz | |
ism-tests: fix the imsm-create-fail-rebuild test
it was testing the return status of mdadm wrongly, and
I found I needed a 'udevadm settle'.
Diffstat (limited to 'tests/env-09imsm-create-fail-rebuild')
| -rw-r--r-- | tests/env-09imsm-create-fail-rebuild | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/tests/env-09imsm-create-fail-rebuild b/tests/env-09imsm-create-fail-rebuild index 2fc90ed..b44746c 100644 --- a/tests/env-09imsm-create-fail-rebuild +++ b/tests/env-09imsm-create-fail-rebuild @@ -1,5 +1,5 @@ imsm_check_hold() { - if [ mdadm --remove $1 $2 -eq 0 ]; then + if mdadm --remove $1 $2; then echo "$2 removal from $1 should have been blocked" >&2 cat /proc/mdstat >&2 mdadm -E $2 @@ -8,7 +8,7 @@ imsm_check_hold() { } imsm_check_removal() { - if [ mdadm --remove $1 $2 -ne 0 ]; then + if ! mdadm --remove $1 $2 ; then echo "$2 removal from $1 should have succeeded" >&2 cat /proc/mdstat >&2 mdadm -E $2 @@ -17,6 +17,7 @@ imsm_check_removal() { } imsm_check() { + udevadm settle case $1 in container ) grep -s "$(((418 * $2)/2)) blocks super external:imsm" /proc/mdstat > /dev/null || { |