<div dir="ltr">Great, thanks! Yes I was just checking after dinner and saw that it looks to be rerunning just fine.<div><br></div><div>Okay I am likely to just strip out the serial test runner strategy at this point. One less thing to keep running.</div><div><br></div><div>Thanks!</div><div><br></div><div>-Todd</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Dec 17, 2015 at 5:41 PM, Ying Chen <span dir="ltr"><<a href="mailto:chying@google.com" target="_blank">chying@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Hi Todd,</div><div><br></div>Seems the problem is fixed by using parallel runner.<div>This test exits normally after rerun.</div><div><a href="http://lab.llvm.org:8011/builders/lldb-x86_64-ubuntu-14.04-android/builds/4357/steps/test7/logs/stdio" target="_blank">http://lab.llvm.org:8011/builders/lldb-x86_64-ubuntu-14.04-android/builds/4357/steps/test7/logs/stdio</a><br></div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Dec 17, 2015 at 2:47 PM, Todd Fiala <span dir="ltr"><<a href="mailto:todd.fiala@gmail.com" target="_blank">todd.fiala@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Hi Ying,</div><div><br></div>I just put this change in that reverted the aarch64 and arm removal from test-rerun eligibility:<div><br></div><div>r255935.</div><div><br></div><div>I'll watch <a href="http://lab.llvm.org:8011/builders/lldb-x86_64-ubuntu-14.04-android/" target="_blank">this builder</a> now and see what happens. If it hangs on reruns, I'll revert r255935.</div><div><br></div><div>Thanks!</div><div><br></div><div>-Todd</div></div><div class="gmail_extra"><span><br><div class="gmail_quote">On Thu, Dec 17, 2015 at 11:18 AM, Todd Fiala <span dir="ltr"><<a href="mailto:todd.fiala@gmail.com" target="_blank">todd.fiala@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Excellent. I'll try this in the afternoon. I need to run out now but I'll check in what we discussed later on when I get back.<div><br></div><div>Thanks!</div></div><div class="gmail_extra"><div><div><br><div class="gmail_quote">On Thu, Dec 17, 2015 at 10:51 AM, Ying Chen <span dir="ltr"><<a href="mailto:chying@google.com" target="_blank">chying@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Yes, you could use android builder to run that experiment.<div>Please watch test 7 of <a href="http://lab.llvm.org:8011/builders/lldb-x86_64-ubuntu-14.04-android/" target="_blank">this builder</a> after your change goes in(Another test for aarch64 which was previously timed out has been disabled for offline debugging of other unrelated problems).</div><div><br></div><div>Thanks,</div><div>Ying</div></div><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Dec 17, 2015 at 10:39 AM, Todd Fiala <span dir="ltr"><<a href="mailto:todd.fiala@gmail.com" target="_blank">todd.fiala@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">(And, as an aside, I may just nuke the serial test runner anyway, since we can do it with a multi-worker runner with a single worker just fine, and reduce the code size --- I really don't see a good reason to keep the serial test runner strategy anymore except for a purely theoretical sense).</div><div class="gmail_extra"><div><div><br><div class="gmail_quote">On Thu, Dec 17, 2015 at 10:37 AM, Todd Fiala <span dir="ltr"><<a href="mailto:todd.fiala@gmail.com" target="_blank">todd.fiala@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi Ying,<div><br></div><div>I am speculating that the rerun logic issue where we saw the hang may be more of a serial test runner issue. Would you mind if I re-enabled the arm/aarch64 inclusion in the rerun logic now that I made a change based on this speculation? It would be a relatively quick way to check if the serial test runner is the issue, since now the rerun logic will not use the serial test runner but rather the normal parallel runner with a single worker (so, the same intent but expressed another way, using the test runners we use all the time). If we still hit the issue, it is unrelated to the serial test runner strategy. If we don't see the issue, then: (1) great, we have a solution, and (2) I know I need to look into the serial test runner strategy which may need some updates for recent changes.</div><div><br></div><div>How does that sound? If I enable it and it times out, I'll just revert the change and we'll go back to normal. (And I'll know more about the issue, albeit with more investigation necessary). If it works just fine, we'll leave it this way (and I'll know I need to look into the serial test runner).</div><span><font color="#888888"><div><div><br></div>-- <br><div><div dir="ltr">-Todd</div></div>
</div></font></span></div>
</blockquote></div><br><br clear="all"><div><br></div></div></div><span><font color="#888888">-- <br><div><div dir="ltr">-Todd</div></div>
</font></span></div>
</blockquote></div><br></div>
</div></div></blockquote></div><br><br clear="all"><div><br></div></div></div><span><font color="#888888">-- <br><div><div dir="ltr">-Todd</div></div>
</font></span></div>
</blockquote></div><br><br clear="all"><div><br></div></span><span><font color="#888888">-- <br><div><div dir="ltr">-Todd</div></div>
</font></span></div>
</blockquote></div><br></div>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr">-Todd</div></div>
</div>