[LNT] r249026 - Add support for storing hash of test binaries.

[Galina Kistanova via llvm-commits]

>Galina, this should make the buildbot master restart Tobias requested
earlier today unnecessary.

Thanks! Please let me know if you will need it.

Thanks

Galina


On Tue, Oct 13, 2015 at 3:03 PM, Kristof Beyls <kristof.beyls at arm.com>
wrote:

> I've made the commit for the default change in r250232.
>
> Galina, this should make the buildbot master restart Tobias requested
> earlier today unnecessary.
>
> Tobias, I'll revert your zorg change tomorrow if r250232 indeed
> proves to solve the issue.
>
> Thanks,
>
> Kristof
>
> > -----Original Message-----
> > From: Tobias Grosser [mailto:tobias at grosser.es]
> > Sent: 13 October 2015 22:50
> > To: Kristof Beyls; Chris Matthews
> > Cc: Chris Matthews; llvm-commits at lists.llvm.org
> > Subject: Re: [LNT] r249026 - Add support for storing hash of test
> > binaries.
> >
> > On 10/13/2015 11:39 PM, Kristof Beyls wrote:
> > > You're absolutely right in being annoyed and I apologize for the
> > > breakage. I would've reverted this, but the commit does include a
> > > change to the database schema and I'm afraid that reverting that
> > > change could cause people's database instances to get corrupted in the
> > > worst case.
> > >
> > > Thinking a bit more on how to get up-to-date clients to submit again
> > > to out-of-date servers: we could make the
> > > --exclude-stat-from-submission=hash
> > > the default value for now, until the llvm.org/perf server is updated.
> > > While this would help for working around the llvm.org/perf server
> > > issue, it would regress the functionality for people who run their own
> > > LNT servers, who will see that hash values will no longer be submitted
> > > to their servers which can process them.
> > >
> > > Putting everything in balance, I think it's a good idea in this
> > > instance to make this change until llvm.org/perf is upgraded; which
> > > would mean no clients/slaves would need to be changed and would avoid
> > > the buildbot master restart.
> >
> > Thanks for your understanding. If you decide to commit this default
> > change (which I think makes sense), feel free to revert my LNT buildbot
> > change and to inform Galina that a restart is not necessary any more.
> >
> > Best,
> > Tobias
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151013/e7e2aeda/attachment.html>