nodemanager to honor max_connections

[deepakverma]

It seems like that nodemanager doesn't honor max_connections set on a ClusterConnectionPool.

pool = ClusterConnectionPool(startup_nodes=[{'host': host, 'port': 6380}],
                             password=password,
                             max_connections_per_node=True,
                             max_connections=30,
                             socket_timeout=20,
                             socket_connect_timeout=10,
                             socket_keepalive=True,
                             )

With this pull requests Nodemanager would start passing the max_connections to StrictRedis

[coveralls]

Coverage remained the same at 87.421% when pulling 77fd25f on deepakverma:unstable into fad035a on Grokzen:unstable.

[Grokzen]

@deepakverma Question: Would it not make more sense to pass down the variable self.max_connections_per_node ??

[Grokzen]

@deepakverma I merged this manually and added a line in the release notes. Find the commit here 2d3006c

Thanks for finding and reporting this :)