[PATCH 2/2] tests: backup default config before testing

From 4d7c70452ca8db359fef6a9f3a420d62e67ef3cd Mon Sep 17 00:00:00 2001
From: Anna Czarnowska
Date: Thu, 24 Mar 2011 16:24:57 +0100
Subject: [PATCH 2/2] tests: backup default config before testing
Cc: linux-raid@vger.kernel.org, Williams, Dan J , Ciechanowski, Ed

We need to have no config at all for this test so if there is one in the
system it must be renamed for the test duration.

Signed-off-by: Anna Czarnowska
---
tests/11spare-migration | 3 +++
1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/tests/11spare-migration b/tests/11spare-migration
index 58f1df9..6c2dc6d 100644
--- a/tests/11spare-migration
+++ b/tests/11spare-migration
@@ -57,6 +57,8 @@ monitor(){
test0()
{
dsc "Test 0: No config file, no spare should be moved"
+[ -f /etc/mdadm.conf ] && mv /etc/mdadm.conf /etc/_mdadm.conf
+[ -f $config ] && rm -f $config
setupdevs 0 0 1 $platform
setupdevs 1 3 4 $platform
monitor 0 1
@@ -64,6 +66,7 @@ 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
+[ -f /etc/_mdadm.conf ] && mv /etc/_mdadm.conf /etc/mdadm.conf
tidyup
}

--
1.7.1

--
To unsubscribe from this list: send the line "unsubscribe linux-raid" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH 2/2] tests: backup default config before testing

On Thu, 24 Mar 2011 21:45:56 +0000 "Czarnowska, Anna"
wrote:

> >From 4d7c70452ca8db359fef6a9f3a420d62e67ef3cd Mon Sep 17 00:00:00 2001
> From: Anna Czarnowska
> Date: Thu, 24 Mar 2011 16:24:57 +0100
> Subject: [PATCH 2/2] tests: backup default config before testing
> Cc: linux-raid@vger.kernel.org, Williams, Dan J , Ciechanowski, Ed
>
> We need to have no config at all for this test so if there is one in the
> system it must be renamed for the test duration.
>

If you want to have no config at all, then you can
--config=none
or
--config=/tmp/mdadm.conf
and make /tmp/mdadm.conf an empty file.

There is really no need to move /etc/mdadm.conf.

I have changed this patch to simple make /tmp/mdamd.conf an empty file.

NeilBrown



> Signed-off-by: Anna Czarnowska
> ---
> tests/11spare-migration | 3 +++
> 1 files changed, 3 insertions(+), 0 deletions(-)
>
> diff --git a/tests/11spare-migration b/tests/11spare-migration
> index 58f1df9..6c2dc6d 100644
> --- a/tests/11spare-migration
> +++ b/tests/11spare-migration
> @@ -57,6 +57,8 @@ monitor(){
> test0()
> {
> dsc "Test 0: No config file, no spare should be moved"
> +[ -f /etc/mdadm.conf ] && mv /etc/mdadm.conf /etc/_mdadm.conf
> +[ -f $config ] && rm -f $config
> setupdevs 0 0 1 $platform
> setupdevs 1 3 4 $platform
> monitor 0 1
> @@ -64,6 +66,7 @@ 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
> +[ -f /etc/_mdadm.conf ] && mv /etc/_mdadm.conf /etc/mdadm.conf
> tidyup
> }
>

--
To unsubscribe from this list: send the line "unsubscribe linux-raid" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html