On Sat, Jul 16, 2016 at 5:33 AM, Andrew Savchenko <birc...@gentoo.org> wrote:
>
> On Fri, 15 Jul 2016 18:03:30 +0000 Robin H. Johnson wrote:
>>
>> The tolerances are presently set to:
>> - 5 seconds of clock drift.
>
> Set it for a minute or two. This will protect from commits from
> really out-of-sync systems (like 14 days mentioned above) and will
> keep usablity hight for others.

I'll defer to infra on how much they can accept, but I tend to think
that we can afford to be a bit more liberal.  However, I don't think
we want to accept things like systems coming out of suspend that are
off by hours.

>
>> - 'git push' must be completed in 60 seconds.
>
> Why?! What is wrong if push will take 120 seconds? I often commit
> from quite an old box and git push takes 20-40 seconds, while this
> is within your limits, the margin is not safe.
>
> What if someone needs to commit via 2G GPRS or similar slow network
> link? Afaik we have developers on quite slow and unstable links.
>
> Just set this limit to 5 minutes to make it a sane protection of a
> stale push.
>

Somebody can correct me if I'm wrong, but I'm pretty sure that only
one person can be pushing anything at time.  So, regardless of any
rsync limitations, I'm not sure we really want developers to be
spending 5 minutes doing a push.  That means that if anybody else does
a commit during that 5 minutes they're going to have to rebase it.
For repos that don't get heavy use I think we could be more liberal.


-- 
Rich

Reply via email to