On Tue, Aug 01, 2017 at 08:34:01AM +0200, Markus Armbruster wrote: > Jeff Cody <jc...@redhat.com> 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!