Author db3l
Recipients LorenzMende, ammar2, db3l, eryksun, heckad, miss-islington, paul.moore, skoslowski, steve.dower, tim.golden, vstinner, zach.ware
Date 2019-10-02.21:05:45
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1570050346.04.0.29744272019.issue36670@roundup.psfhosted.org>
In-reply-to
Content
Oh, I agree it's just a warning, and I suspect few people look into warnings, but since it's not from an actual test, I'm not sure the overall build should be flagged.

The manual typeperf looks fine, but there's no way I could tell visually how the I/O is being done under the covers.  My bet is typeperf is probably issuing multiple output calls, one for the leading timestamp and then one for each value.  So regrtest just happens to interleave its own read in between the two.  The subsequent read would then get the trailing ",####" which the current code would parse normally as it only cares about the second field.  In which case it's not even affecting the load monitoring, other than the warning message.

I suspect it's happening on the Windows 10 buildbot as the machine is reasonably fast.  But to your earlier comment about things just working previously, I'm not sure there's that much value in adding code to deal with this case.

I'd probably just remove the warning to restore the earlier behavior.  The current code still ignores other issues like an actual read failure in read_output.  Or, if there's a way to generate the message without it being considered an overall test build warning, that would work too.
History
Date User Action Args
2019-10-02 21:05:46db3lsetrecipients: + db3l, paul.moore, vstinner, tim.golden, zach.ware, eryksun, steve.dower, ammar2, miss-islington, LorenzMende, skoslowski, heckad
2019-10-02 21:05:46db3lsetmessageid: <1570050346.04.0.29744272019.issue36670@roundup.psfhosted.org>
2019-10-02 21:05:46db3llinkissue36670 messages
2019-10-02 21:05:45db3lcreate