From c60536e5272ae6c410f61842f8c1581ebe83ca38 Mon Sep 17 00:00:00 2001
From: Andy McCurdy <andy@andymccurdy.com>
Date: Mon, 26 May 2014 12:02:50 -0700
Subject: only pass explicitly defined or socket_* options to sentinel
 connections.

fixed #483
---
 redis/sentinel.py | 57 +++++++++++++++++++++++++++++++++++++++++--------------
 1 file changed, 43 insertions(+), 14 deletions(-)

diff --git a/redis/sentinel.py b/redis/sentinel.py
index 4320847..0479690 100644
--- a/redis/sentinel.py
+++ b/redis/sentinel.py
@@ -5,7 +5,7 @@ import weakref
 from redis.client import StrictRedis
 from redis.connection import ConnectionPool, Connection
 from redis.exceptions import ConnectionError, ResponseError, ReadOnlyError
-from redis._compat import xrange, nativestr
+from redis._compat import iteritems, nativestr, xrange
 
 
 class MasterNotFoundError(ConnectionError):
@@ -82,8 +82,6 @@ class SentinelConnectionPool(ConnectionPool):
         self.connection_kwargs['connection_pool'] = weakref.proxy(self)
         self.service_name = service_name
         self.sentinel_manager = sentinel_manager
-        self.master_address = None
-        self.slave_rr_counter = None
 
     def __repr__(self):
         return "%s<service=%s(%s)" % (
@@ -92,6 +90,11 @@ class SentinelConnectionPool(ConnectionPool):
             self.is_master and 'master' or 'slave',
         )
 
+    def reset(self):
+        super(SentinelConnectionPool, self).reset()
+        self.master_address = None
+        self.slave_rr_counter = None
+
     def get_master_address(self):
         master_address = self.sentinel_manager.discover_master(
             self.service_name)
@@ -124,6 +127,7 @@ class SentinelConnectionPool(ConnectionPool):
     def _checkpid(self):
         if self.pid != os.getpid():
             self.disconnect()
+            self.reset()
             self.__init__(self.service_name, self.sentinel_manager,
                           connection_class=self.connection_class,
                           max_connections=self.max_connections,
@@ -145,18 +149,35 @@ class Sentinel(object):
     ``sentinels`` is a list of sentinel nodes. Each node is represented by
     a pair (hostname, port).
 
-    Use ``socket_timeout`` to specify a timeout for sentinel clients.
-    It's recommended to use short timeouts.
+    ``min_other_sentinels`` defined a minimum number of peers for a sentinel.
+    When querying a sentinel, if it doesn't meet this threshold, responses
+    from that sentinel won't be considered valid.
+
+    ``sentinel_kwargs`` is a dictionary of connection arguments used when
+    connecting to sentinel instances. Any argument that can be passed to
+    a normal Redis connection can be specified here. If ``sentinel_kwargs`` is
+    not specified, any socket_timeout and socket_keepalive options specified
+    in ``connection_kwargs`` will be used.
 
-    Use ``min_other_sentinels`` to filter out sentinels with not enough peers.
+    ``connection_kwargs`` are keyword arguments that will be used when
+    establishing a connection to a Redis server.
     """
 
-    def __init__(self, sentinels, password=None, socket_timeout=None,
-                 min_other_sentinels=0):
-        self.sentinels = [StrictRedis(hostname, port, password=password,
-                                      socket_timeout=socket_timeout)
+    def __init__(self, sentinels, min_other_sentinels=0, sentinel_kwargs=None,
+                 **connection_kwargs):
+        # if sentinel_kwargs isn't defined, use the socket_* options from
+        # connection_kwargs
+        if sentinel_kwargs is None:
+            sentinel_kwargs = dict([(k, v)
+                                    for k, v in iteritems(connection_kwargs)
+                                    if k.startswith('socket_')
+                                    ])
+        self.sentinel_kwargs = sentinel_kwargs
+
+        self.sentinels = [StrictRedis(hostname, port, **self.sentinel_kwargs)
                           for hostname, port in sentinels]
         self.min_other_sentinels = min_other_sentinels
+        self.connection_kwargs = connection_kwargs
 
     def __repr__(self):
         sentinel_addresses = []
@@ -237,11 +258,15 @@ class Sentinel(object):
         The ``connection_pool_class`` specifies the connection pool to use.
         The SentinelConnectionPool will be used by default.
 
-        All other arguments are passed directly to the SentinelConnectionPool.
+        All other keyword arguments are merged with any connection_kwargs
+        passed to this class and passed to the connection pool as keyword
+        arguments to be used to initialize Redis connections.
         """
         kwargs['is_master'] = True
+        connection_kwargs = dict(self.connection_kwargs)
+        connection_kwargs.update(kwargs)
         return redis_class(connection_pool=connection_pool_class(
-            service_name, self, **kwargs))
+            service_name, self, **connection_kwargs))
 
     def slave_for(self, service_name, redis_class=StrictRedis,
                   connection_pool_class=SentinelConnectionPool, **kwargs):
@@ -258,8 +283,12 @@ class Sentinel(object):
         The ``connection_pool_class`` specifies the connection pool to use.
         The SentinelConnectionPool will be used by default.
 
-        All other arguments are passed directly to the SentinelConnectionPool.
+        All other keyword arguments are merged with any connection_kwargs
+        passed to this class and passed to the connection pool as keyword
+        arguments to be used to initialize Redis connections.
         """
         kwargs['is_master'] = False
+        connection_kwargs = dict(self.connection_kwargs)
+        connection_kwargs.update(kwargs)
         return redis_class(connection_pool=connection_pool_class(
-            service_name, self, **kwargs))
+            service_name, self, **connection_kwargs))
-- 
cgit v1.2.1