Re: buildfarm animals and 'snapshot too old'

On 17.5.2014 19:58, Andrew Dunstan wrote:
>
> On 05/15/2014 07:47 PM, Tomas Vondra wrote:
>> On 15.5.2014 22:07, Andrew Dunstan wrote:
>>> Yes, I've seen that. Frankly, a test that takes something like 500
>>> hours is a bit crazy.
>> Maybe. It certainly is not a test people will use during development.
>> But if it can detect some hard-to-find errors in the code, that might
>> possibly lead to serious problems, then +1 from me to run them at least
>> on one animal. 500 hours is ~3 weeks, which is not that bad IMHO.
>>
>> Also, once you know where it fails the developer can run just that
>> single test (which might take minutes/hours, but not days).
>
>
>
> I have made a change that omits the snapshot sanity check for
> CLOBBER_CACHE_RECURSIVELY cases, but keeps it for all others. See
> <https://github.com/PGBuildFarm/server-code/commit/abd946918279b7683056a4fc3156415ef31a4675>

OK, thanks. Seems reasonable.

Tomas