Re: [PATCH 0/3] mdadm/test: Refactor the codes of 'test' script

[Jes Sorensen <jes.sorensen@gmail.com>]

On 06/07/2017 05:31 AM, Zhilong Liu wrote:
> Hi, Jes;
>    This patchset mainly focus on improving the 'test' script, rip it
> apart and put it together in a more coherent form.
>    Over 3 months passed, the RFC patch of improving 'test' hadn't received
> any ideas, thus I post this sample version that I have tested many times.
>    It's just the beginning to improve the test part of mdadm, any questions
> is very welcome.
> 
>    Sorry for that, I haven't added the function suggested from Guoqing, cut
> the words from the email tree of
> "ANNOUNCE: mdadm 4.0 - A tool for managing md Soft RAID under Linux":
> 
> Guoqing asked:
> ...
> Is current test can run the test for different raid level, say,
> "./test --raidtype=raid1"
> could execute all the *r1* tests, does it make sense to do it if we don't
> support it now.
> ...
> 
>    Just list this suggestion here, waiting for ideas.
> 
> RFC patch mail-tree:
> [PATCH RFC] test: revise 'test' and make it easier to understand
> Acked-by: Neil Brown
> And suggest to split those changing into smaller patches from Guoqing.

The whole test infrastructure is something I would like to see rewamped 
and modernized, so you starting to work on this is excellent.

With regard to your question, then having an option to say 'run all 
raid1 tests' is probably good, we should have that for all raid levels, 
imsm, and ddf.

I also would like to see a more flexible way to configure the tests, 
like it should be possible to specify a bunch of physical devices for 
the tests, instead of relying on loop devices.

Cheers,
Jes