Fixed - dead Sentinel appears in logs as node added and down. #2143

redisson/src/main/java/org/redisson/connection/SentinelConnectionManager.java

@@ -511,9 +511,27 @@ public class SentinelConnectionManager extends MasterSlaveConnectionManager {
                 return;
             }
 
-            sentinels.putIfAbsent(addr, client);
+            RFuture<RedisConnection> f = client.connectAsync();
-            log.info("sentinel: {} added", addr);
+            f.onComplete((connection, ex) -> {
-            result.trySuccess(null);
+                if (ex != null) {
+                    result.tryFailure(ex);
+                    return;
+                }
+
+                RFuture<String> r = connection.async(config.getTimeout(), RedisCommands.PING);
+                r.onComplete((resp, exc) -> {
+                    if (exc != null) {
+                        result.tryFailure(exc);
+                        return;
+                    }
+                    
+                    if (sentinels.putIfAbsent(addr, client) == null) {
+                        log.info("sentinel: {} added", addr);
+                    }
+                    result.trySuccess(null);
+                });
+            });
+
         });
         return result;
     }