<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, May 19, 2015 at 10:28 AM, Chris Matthews <span dir="ltr"><<a href="mailto:chris.matthews@apple.com" target="_blank">chris.matthews@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">If “flooding” is the issue, the only long term solution is to A) have a bot channel that can be flooded or B) curate the list of bots which notify IRC.<br></blockquote><div><br>Alternatively: The phased builder approach Apple was upstreaming for a while that seemed to get stalled at some point.<br><br>Or a pseudo-phased system: Have builders that do "interesting" things not fail (but simply warn/no-result) when early steps fail (I've wanted/meant to do this with the GDB buildbot for example: I don't care if LLVM fails to compile on that buildbot, I know other bots will catch it - so I'd be happy to mark that as a warning/do-not-continue rather than error/do-not-continue, then just carry on with the next run). You miss the efficiency gains of a phased builder, but you get the same reduced noise/spam.<br><br>- David<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
A more fun solution would be to have the bots implement a first failure policy for notifications.  We move the bots to their own channel, then have an IRC bot watch that channel for failures, parse the rev, and only notify if it is the first to find a failure at that rev.<br>
<br>
I agree that long blame lists are useless and should be removed. URL shortening would add a dependence on an external service which is a new source of failure and trust.<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
<br>
> On May 19, 2015, at 9:50 AM, Justin Bogner <<a href="mailto:mail@justinbogner.com">mail@justinbogner.com</a>> wrote:<br>
><br>
> Jonathan Roelofs <<a href="mailto:jonathan@codesourcery.com">jonathan@codesourcery.com</a>> writes:<br>
>> On 5/19/15 8:58 AM, Renato Golin wrote:<br>
>>> On 19 May 2015 at 15:48, Jonathan Roelofs <<a href="mailto:jonathan@codesourcery.com">jonathan@codesourcery.com</a>> wrote:<br>
>>>> Also, perhaps the URLs should be shortened?<br>
>>><br>
>>> That's a good idea.<br>
>>><br>
>>><br>
>>>> The format could be: [botname]: [buildername] [short_url]<br>
>>>> ("Passed"|"Failed:" [usernames])<br>
>>><br>
>>> The only reason to show "Passed" results is if they were failing<br>
>>> before, as a confirmation that whatever you did to fix, worked.<br>
>>> Otherwise, they're just noise.<br>
>><br>
>> Good point.<br>
>><br>
>> [botname]: [buildername] [short_url] [revision] ("Fixed:"|"FAILED:")<br>
>> [usernames]<br>
><br>
> Most (I thought all) of the bots already do the fixed failed thing.<br>
> There's something to be said for a consistent format though, and it'd be<br>
> nice if more of the bots mentioned the revision (green-dragon does, but<br>
> llvmbb and bb-chapuni don't).<br>
><br>
> The other thing that would help is limiting the number of names that can<br>
> show up on a blamelist. If there are a hundred names on the blamelist<br>
> the message is way too long and the notifications don't help much. The<br>
> green-dragon bot just says "[n] people on blamelist" instead in these<br>
> cases.<br>
><br>
>> Should the bots blame people for fixes?<br>
><br>
> I don't think so - if you fixed it intentionally you usually know it,<br>
> and if not it's just another beeping window to ignore.<br>
> _______________________________________________<br>
> cfe-dev mailing list<br>
> <a href="mailto:cfe-dev@cs.uiuc.edu">cfe-dev@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev</a><br>
<br>
<br>
_______________________________________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@cs.uiuc.edu">cfe-dev@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev</a><br>
</div></div></blockquote></div><br></div></div>