<div dir="ltr">On Fri, Feb 1, 2013 at 9:31 AM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Fri, Feb 1, 2013 at 9:14 AM, Daniel Dunbar <<a href="mailto:daniel@zuster.org">daniel@zuster.org</a>> wrote:<br>

> On Fri, Feb 1, 2013 at 8:20 AM, David Blaikie <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> wrote:<br>
>><br>
>> On Thu, Jan 31, 2013 at 5:44 PM, Daniel Dunbar <<a href="mailto:daniel@zuster.org">daniel@zuster.org</a>> wrote:<br>
>> > Author: ddunbar<br>
>> > Date: Thu Jan 31 19:44:23 2013<br>
>> > New Revision: 174139<br>
>> ><br>
>> > URL: <a href="http://llvm.org/viewvc/llvm-project?rev=174139&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=174139&view=rev</a><br>
>> > Log:<br>
>> > LitTestCommand: Add back support for max_logs argument.<br>
>> ><br>
>> > Modified:<br>
>> >     zorg/trunk/zorg/buildbot/commands/LitTestCommand.py<br>
>> ><br>
>> > Modified: zorg/trunk/zorg/buildbot/commands/LitTestCommand.py<br>
>> > URL:<br>
>> > <a href="http://llvm.org/viewvc/llvm-project/zorg/trunk/zorg/buildbot/commands/LitTestCommand.py?rev=174139&r1=174138&r2=174139&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/zorg/trunk/zorg/buildbot/commands/LitTestCommand.py?rev=174139&r1=174138&r2=174139&view=diff</a><br>

>> ><br>
>> > ==============================================================================<br>
>> > --- zorg/trunk/zorg/buildbot/commands/LitTestCommand.py (original)<br>
>> > +++ zorg/trunk/zorg/buildbot/commands/LitTestCommand.py Thu Jan 31<br>
>> > 19:44:23 2013<br>
>> > @@ -20,9 +20,11 @@ class LitLogObserver(LogLineObserver):<br>
>> >    # step results.<br>
>> >    failingCodes = set(['FAIL', 'XPASS', 'KPASS', 'UNRESOLVED'])<br>
>> ><br>
>> > -  def __init__(self):<br>
>> > +  def __init__(self, maxLogs=None):<br>
>> >      LogLineObserver.__init__(self)<br>
>> >      self.resultCounts = {}<br>
>> > +    self.maxLogs = maxLogs<br>
>> > +    self.numLogs = 0<br>
>><br>
>> What's the particular motivation for this feature?<br>
><br>
><br>
> This is mainly just to protect the buildbot and web UI. In cases where there<br>
> is a catastrophic failure of some kind, we don't want buildbot to be showing<br>
> 13k tests and logs in the web UI.<br>
><br>
> The primary goal of the extracted logs is to make it easy for users to jump<br>
> examine a specific failure. This is substantially less useful as more test<br>
> failures are present so it makes sense to cap it at some point.<br>
><br>
>> Lit does the same<br>
>> thing - logging every failure & it's not terribly usable there, it<br>
>> seems much more usable in a web UI where you can scroll around a list,<br>
>> click on the failures, etc. If it's really a problem, shouldn't we<br>
>> just fix lit to do this instead?<br>
><br>
><br>
> I don't think it is a problem in the lit console based context.<br>
<br>
</div></div>Why not? It seems easier to navigate in a web UI than a console.<br></blockquote><div><br></div><div style>That's just my opinion, tempered by experience. The console has one log that scrolls up and down, it doesn't blow up as bad with tons of failures. The buildbot UI doesn't scale in the same way.</div>
<div style><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
> Again, please don't just drop support for a feature without understanding<br>
> why it was there.<br>
<br>
</div>I asked whether anyone wanted to pre-commit review my changes - the<br>
general opinion was that post-commit review was just as good because<br>
it'd be done before anyone updated their buildmasters anyway.<br></blockquote><div><br></div><div style>My understanding was that you were just rewriting things, not changing functionality.</div><div style><br></div>
<div style> - Daniel</div><div style><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888"><br>
- David<br>
</font></span></blockquote></div><br></div></div>