<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 20 February 2018 at 10:12, Stephan Bergmann <span dir="ltr"><<a href="mailto:sbergman@redhat.com" target="_blank">sbergman@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 17.02.2018 07:54, Noel Grandin wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
at the very least we should make that wait command use a timeout, and fail the test, rather than hanging indefinitely​<br>
</blockquote>
<br></span>
As long as its unclear what the underlying problem is, what benefit does it have to add a timeout failure path in the test?  Without the added timeout, a local failed test will be ready to be inspected (and moggi now gave suggestions what exactly to inspect in another mail in this thread).  And failed tests on the tinderboxes will eventually time out either way.<div class="HOEnZb"><div class="h5"><br></div></div></blockquote><div><br></div><div>My thinking is that it's better to fail sooner rather than tying up the box for the hour or so it takes before the build timeout kicks in.</div><div><br></div><div>But yeah, it is now harder to catch the thing locally.</div></div></div></div>