Just to be clear, im totally on board with the global timeout.  It's the per-test override I'm not crazy about, since it doesn't matter much for the bot, and it's non-discoverable for the user.<br><div class="gmail_quote">On Thu, Dec 18, 2014 at 8:34 AM Vince Harron <<a href="mailto:vharron@google.com">vharron@google.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><span style="font-size:13px">> And in addition to both of those, we really should be either fixing the tests or disabling them. Now that this timeout is in, i have a feeling that fixing the hanging tests will be de-prioritized. </span><br><div><span style="font-size:13px"><br></span></div></div><div dir="ltr"><div><span style="font-size:13px">Currently there are tests that only hang on the debian buildbot.  These hangs obscure all other test failures.  This patch was the only way to figure out what those tests are.  Unfortunately, they are not hanging locally for us and are therefore not at the top of our priority list.  We could mark them as XFail, but we still need the timeout in in case this happens again.</span></div><div><span style="font-size:13px"><br></span></div><div><span style="font-size:13px">Remember, without the timeout feature, if a single test hangs, you need to surgically delete the hung test or you lose all test results.</span></div><div><span style="font-size:13px"><br></span></div></div><div class="gmail_extra"></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Dec 18, 2014 at 7:50 AM, Zachary Turner <span dir="ltr"><<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>></span> wrote:<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>My concern, and the reason I brought it up, is because it seems like<br>
unnecessary complexity which only benefits a handful of people. I don't<br>
like adding complexity that isn't useful for a wide audience.<br>
<br>
If this is just for making your own local test suite terminate in a<br>
reasonable amount of time, then you can do that with an external shell<br>
script which you run when you notice it's hung. If the issue is not knowing<br>
it's hung, then we should change the test runner to print its output as the<br>
test suite runs instead of at the end (we should do this anyway actually<br>
because that's very useful).<br>
<br>
And in addition to both of those, we really should be either fixing the<br>
tests or disabling them. Now that this timeout is in, i have a feeling that<br>
fixing the hanging tests will be de-prioritized.<br>
<br>
Anyway, this patch is going in (or has gone in) for now, but I really don't<br>
want to see this timeout logic get any more complicated in its current form.<br>
<br>
<br>
</span><div><div><a href="http://reviews.llvm.org/D6669" target="_blank">http://reviews.llvm.org/D6669</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
<br>
<br>
</div></div></blockquote></div><br clear="all"><div><br></div></div><div class="gmail_extra">-- <br><div><div dir="ltr"><br><table cellspacing="0" cellpadding="0" style="font-family:'Times New Roman'"><tbody><tr style="color:rgb(85,85,85);font-family:sans-serif;font-size:small"><td nowrap style="border-top-style:solid;border-top-color:rgb(213,15,37);border-top-width:2px">Vince Harron |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(51,105,232);border-top-width:2px"> Technical Lead Manager |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(0,153,57);border-top-width:2px"> <a href="mailto:vharron@google.com" target="_blank">vharron@google.com</a> |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(238,178,17);border-top-width:2px"> 858-442-0868</td></tr></tbody></table><br></div></div>
</div></blockquote></div>