<div dir="ltr">Great.  Actually there is a latent bug in there we hit yesterday, when there is a platform set but it doesn't start with "remote-" (we assume the match result has a first matched group and blow up).  I'll submit a patch that both fixes that up and strips out the darwin ones.  I think the rerun logic now fully covers us on the Darwin side.</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jan 20, 2016 at 5:18 AM, Pavel Labath <span dir="ltr"><<a href="mailto:labath@google.com" target="_blank">labath@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
I have removed all of our expected timeouts from dosep.py (there are<br>
still some freebsd and darwin ones left, but I don't know If anyone is<br>
looking at those), so I think we're not using any part of the old test<br>
runner at the moment. All clear for removal on our part.<br>
<span class="HOEnZb"><font color="#888888"><br>
pl<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
On 14 December 2015 at 16:06, Todd Fiala <<a href="mailto:todd.fiala@gmail.com">todd.fiala@gmail.com</a>> wrote:<br>
> Oh yeah, that's fine.  I won't take that code out.<br>
><br>
> Hmm at least some of the builds went through this weekend, I made a number<br>
> of changes Saturday morning (US Pacific time) that I saw go through the<br>
> Ubuntu 14.04 cmake bot.<br>
><br>
> On Mon, Dec 14, 2015 at 6:29 AM, Pavel Labath <<a href="mailto:labath@google.com">labath@google.com</a>> wrote:<br>
>><br>
>> Hi,<br>
>><br>
>> we've had an unrelated breaking change, so the buildbots were red over<br>
>> the weekend. I've fixed it now, and it seems to be turning green.<br>
>> We've also had power outage during the weekend and not all of the<br>
>> buildbots are back up yet, as we need to wait for MTV to wake up. I'd<br>
>> like to give this at least one more day, to give them a chance to<br>
>> stabilize. Is this blocking you from making further changes to the<br>
>> test event system?<br>
>><br>
>> pl<br>
>><br>
>> On 12 December 2015 at 00:20, Todd Fiala <<a href="mailto:todd.fiala@gmail.com">todd.fiala@gmail.com</a>> wrote:<br>
>> > Hey Pavel and/or Tamas,<br>
>> ><br>
>> > Let me know when we're definitely all clear on the expected timeout<br>
>> > support<br>
>> > I added to the (now once again) newer default test results.<br>
>> ><br>
>> > As soon as we don't need the legacy summary results anymore, I'm going<br>
>> > to<br>
>> > strip out the code that manages it.  It is quite messy and duplicates<br>
>> > the<br>
>> > content that is better handled by the test event system.<br>
>> ><br>
>> > Thanks!<br>
>> ><br>
>> > -Todd<br>
>> ><br>
>> > On Fri, Dec 11, 2015 at 2:03 PM, Todd Fiala <<a href="mailto:todd.fiala@gmail.com">todd.fiala@gmail.com</a>><br>
>> > wrote:<br>
>> >><br>
>> >> I went ahead and added the expected timeout support in r255363.<br>
>> >><br>
>> >> I'm going to turn back on the new BasicResultsFormatter as the default.<br>
>> >> We can flip this back off if it is still not doing everything we need,<br>
>> >> but I<br>
>> >> *think* we cover the issue you saw now.<br>
>> >><br>
>> >> -Todd<br>
>> >><br>
>> >> On Fri, Dec 11, 2015 at 10:14 AM, Todd Fiala <<a href="mailto:todd.fiala@gmail.com">todd.fiala@gmail.com</a>><br>
>> >> wrote:<br>
>> >>><br>
>> >>> Hi Pavel,<br>
>> >>><br>
>> >>> I'm going to adjust the new summary output for expected timeouts.  I<br>
>> >>> hope<br>
>> >>> to do that in the next hour or less.  I'll put that in and flip the<br>
>> >>> default<br>
>> >>> back on for using the new summary output.<br>
>> >>><br>
>> >>> I'll do those two changes separately, so you can revert the flip back<br>
>> >>> on<br>
>> >>> to flip it back off if we still have an issue.<br>
>> >>><br>
>> >>> Sound good?<br>
>> >>><br>
>> >>> (This can be orthogonal to the new work to mark up expected timeouts).<br>
>> >>> --<br>
>> >>> -Todd<br>
>> >><br>
>> >><br>
>> >><br>
>> >><br>
>> >> --<br>
>> >> -Todd<br>
>> ><br>
>> ><br>
>> ><br>
>> ><br>
>> > --<br>
>> > -Todd<br>
><br>
><br>
><br>
><br>
> --<br>
> -Todd<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr">-Todd</div></div>
</div>