caches: optimized defaults for safer more reliable behaviour

vcsserver/lib/rc_cache/backends.py

@@ -29,6 +29,8 b' from dogpile.cache.backends import redis'
 from dogpile.cache.backends.file import NO_VALUE, compat, FileLock
 from dogpile.cache.util import memoized_property
 
+from pyramid.settings import asbool
+
 from vcsserver.lib.memory_lru_dict import LRUDict, LRUDictDebug
 
 
@@ -194,11 +196,11 b' class BaseRedisBackend(redis_backend.Red'
     def __init__(self, arguments):
         super(BaseRedisBackend, self).__init__(arguments)
         self._lock_timeout = self.lock_timeout
-        self._lock_auto_renewal = arguments.pop("lock_auto_renewal", False)
+        self._lock_auto_renewal = asbool(arguments.pop("lock_auto_renewal", True))
 
         if self._lock_auto_renewal and not self._lock_timeout:
             # set default timeout for auto_renewal
-            self._lock_timeout = 60
+            self._lock_timeout = 30
 
     def _create_client(self):
         args = {}