<div dir="ltr"><div><div><div>><span class="">Galina</span>, this should make the buildbot master restart Tobias requested<br>
earlier today unnecessary.<br><br></div><div>Thanks! Please let me know if you will need it.<br></div></div><br>Thanks<br><br></div>Galina<br><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 13, 2015 at 3:03 PM, Kristof Beyls <span dir="ltr"><<a href="mailto:kristof.beyls@arm.com" target="_blank">kristof.beyls@arm.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I've made the commit for the default change in r250232.<br>
<br>
Galina, this should make the buildbot master restart Tobias requested<br>
earlier today unnecessary.<br>
<br>
Tobias, I'll revert your zorg change tomorrow if r250232 indeed<br>
proves to solve the issue.<br>
<span class="im HOEnZb"><br>
Thanks,<br>
<br>
Kristof<br>
<br>
> -----Original Message-----<br>
> From: Tobias Grosser [mailto:<a href="mailto:tobias@grosser.es">tobias@grosser.es</a>]<br>
</span><span class="im HOEnZb">> Sent: 13 October 2015 22:50<br>
> To: Kristof Beyls; Chris Matthews<br>
> Cc: Chris Matthews; <a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
> Subject: Re: [LNT] r249026 - Add support for storing hash of test<br>
> binaries.<br>
><br>
</span><div class="HOEnZb"><div class="h5">> On 10/13/2015 11:39 PM, Kristof Beyls wrote:<br>
> > You're absolutely right in being annoyed and I apologize for the<br>
> > breakage. I would've reverted this, but the commit does include a<br>
> > change to the database schema and I'm afraid that reverting that<br>
> > change could cause people's database instances to get corrupted in the<br>
> > worst case.<br>
> ><br>
> > Thinking a bit more on how to get up-to-date clients to submit again<br>
> > to out-of-date servers: we could make the<br>
> > --exclude-stat-from-submission=hash<br>
> > the default value for now, until the <a href="http://llvm.org/perf" rel="noreferrer" target="_blank">llvm.org/perf</a> server is updated.<br>
> > While this would help for working around the <a href="http://llvm.org/perf" rel="noreferrer" target="_blank">llvm.org/perf</a> server<br>
> > issue, it would regress the functionality for people who run their own<br>
> > LNT servers, who will see that hash values will no longer be submitted<br>
> > to their servers which can process them.<br>
> ><br>
> > Putting everything in balance, I think it's a good idea in this<br>
> > instance to make this change until <a href="http://llvm.org/perf" rel="noreferrer" target="_blank">llvm.org/perf</a> is upgraded; which<br>
> > would mean no clients/slaves would need to be changed and would avoid<br>
> > the buildbot master restart.<br>
><br>
> Thanks for your understanding. If you decide to commit this default<br>
> change (which I think makes sense), feel free to revert my LNT buildbot<br>
> change and to inform Galina that a restart is not necessary any more.<br>
><br>
> Best,<br>
> Tobias<br>
<br>
<br>
<br>
</div></div></blockquote></div><br></div></div>