Re: [PATCH 0/3] iotests: enable logging prior to notrun() invocation

[John Snow]

On 5/15/20 5:57 AM, Kevin Wolf wrote:
> Am 14.05.2020 um 22:16 hat John Snow geschrieben:
>> Hi, you can take just patch 1.
> 
> Thanks, I'm doing that now.
> 
>> patches 2-3 admittedly don't do a whole heck of a lot, because I
>> didn't realize that ./check discards *all* output from either stdout
>> or stderr.
> 
> It doesn't discard it, but it compares it to the reference output and
> prints a diff if it's non-empty.
> 

notrun cases do effectively discard that output, though. I guess that's
fine if it gets logged as output otherwise.

> At least for bash scripts, stderr contains important test output. Not
> sure what the *_log() functions in iotests.py do, maybe they redirect
> the stderr output from tools to stdout.
> 
>> The changes are tiny, though, and maybe still worth doing in the long
>> run? Hm. They are archived on the list now, anyway.
> 
> We can still take these patches for a later pull request if you think we
> should, though I'm not sure if they are useful given how things actually
> work. I don't have a strong opinion either way (except that I don't want
> to see more than a single line per test without -d, but these patches
> don't try to do this even though you wrote them under the assumption
> that this is how things work).
> 

Nah, it's fine. Bigger fish to fry. Thanks!