[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [Qemu-devel] [PATCH v2 for-2.11 3/3] qemu-iotests: add
From: |
Jeff Cody |
Subject: |
Re: [Qemu-block] [Qemu-devel] [PATCH v2 for-2.11 3/3] qemu-iotests: add option to save temp files on error |
Date: |
Tue, 1 Aug 2017 17:42:06 -0400 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
On Tue, Aug 01, 2017 at 08:34:01AM +0200, Markus Armbruster wrote:
> Jeff Cody <address@hidden> writes:
>
> > Now that ./check takes care of cleaning up after each tests, it
> > can also selectively not clean up. Add option to leave all output from
> > tests intact if that test encountered an error.
> >
> > Note: this currently only works for bash tests, as the python tests
> > still clean up after themselves manually.
>
> Should we add a TODO comment for that?
>
Couldn't hurt!
> Much appreciated work, by the way. You might want to mention in one of
> your commit messages that this is also a step towards running iotests in
> parallel.
>
Thanks. I'll go ahead and spin a v3 to clean up commit messages and add a
TODO, since 2.11 won't open for a while.
> Another step towards sanity would be making $TEST_DIR instead of
> $source_iotests the current working directory for running tests.
Yep!