From: Mariusz Tkaczyk <mariusz.tkaczyk@linux.intel.com>
To: Hannes Reinecke <hare@suse.de>
Cc: Logan Gunthorpe <logan@eideticom.com>, Coly Li <colyli@suse.de>,
Lukasz Florczak <lukasz.florczak@linux.intel.com>,
linux-raid@vger.kernel.org
Subject: Re: Python tests
Date: Fri, 26 Aug 2022 10:12:42 +0200 [thread overview]
Message-ID: <20220826101242.000055d0@linux.intel.com> (raw)
In-Reply-To: <f40217bd-5db4-65e9-0829-5d652281f3f2@suse.de>
On Thu, 25 Aug 2022 08:15:14 +0200
Hannes Reinecke <hare@suse.de> wrote:
> On 8/24/22 22:40, Logan Gunthorpe wrote:
> > Hi,
> >
> > On 2022-08-24 10:23, Coly Li wrote:
> >>
> >>
> >>> 2022年8月24日 16:15,Lukasz Florczak <lukasz.florczak@linux.intel.com>
> >>> 写道:
> >>>
> >>> Hi Coly,
> >>> I want to write some mdadm tests for assemblation and incremental
> >>> regarding duplicated array names in config and I'd like to do it in
> >>> python. I've seen that some time ago[1] you said that you could try to
> >>> integrate the python tests framework into the mdadm ci. I was wondering
> >>> how is it going? Do you need any help with this subject?
> >>>
> >>> Thanks,
> >>> Lukasz
> >>>
> >>> [1] https://marc.info/?l=linux-raid&m=165277539509464&w=2
> >>
> >> Hi Lukasz,
> >>
> >> Now I just make some of the existed mdadm test scripts running, which are
> >> copied from upstream mdadm. There won’t be any conflict for the python
> >> testing code between you and me, because now I am just studying Python
> >> again and not do any useful thing yet.
> >>
> >> As I said if no one works on the testing framework, I will do it, but it
> >> may take time. How about posting out the python code once you make it,
> >> then let’s put it into mdadm-test to test mdadm-CI, and improve whole
> >> things step by step.
> >
> > I'm not sure if this is of use to anyone but we are very slowly growing
> > a testing framework written mostly in python. Its focused on raid5 at
> > the moment and still is a fairly sizable mess, but we've caught a lot of
> > bugs with it and continue to run it, clean it up and make improvements.
> >
> > https://github.com/Eideticom/raid5-tests/
> >
> > The 'md.py 'file provides a nice interface to setup an array based on
> > ram, loop or block devices and provides methods to degrade, recover or
> > grow the array. 'test3' does grow/degrade tests while running IO,
> > 'test_all' runs all the tests with an array of different settings.
> >
> > Feel free to use anything from it that you may find useful.
> >
> When developing 'md_monitor' (https://github.com/hreinecke/md_monitor)
> I've also created an extensive testsuite for it.
> The one thing which I found particularly painful is error handling once
> mdadm fails. It's really hard to figure out _what_ went wrong, and more
> often than not mdadm simply locked up on me (try to stall I/O on one
> component device while md is running and you are in a world of pain).
>
> That's when I started to split mdadm into a library, as then we could
> have a python binding and the life would so much more fun.
> So maybe I should resurrect that patchset.
>
Hi Hannes,
Will be great if you can handle this. Beside library part you did huge code
rearrangement and divides one big mdadm.h header into separate ones which is
also great improvement. mdadm.h is still growing.
It is hard to do this separately, one by one due to cross references between
files.
Thanks,
Mariusz
next prev parent reply other threads:[~2022-08-26 8:12 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-08-24 8:15 Python tests Lukasz Florczak
2022-08-24 16:23 ` Coly Li
2022-08-24 20:40 ` Logan Gunthorpe
2022-08-25 6:15 ` Hannes Reinecke
2022-08-26 8:12 ` Mariusz Tkaczyk [this message]
2022-09-26 8:19 ` Lukasz Florczak
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=20220826101242.000055d0@linux.intel.com \
--to=mariusz.tkaczyk@linux.intel.com \
--cc=colyli@suse.de \
--cc=hare@suse.de \
--cc=linux-raid@vger.kernel.org \
--cc=logan@eideticom.com \
--cc=lukasz.florczak@linux.intel.com \
/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;
as well as URLs for NNTP newsgroup(s).