Add a metric to track unresolved DNS hosts

PrabhuHub · Aug 6, 2018 · da4a19f · da4a19f
1 parent 5cb0453
commit da4a19f
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/...ore/src/main/java/com/netflix/zuul/netty/connectionpool/NettyClientConnectionFactory.java b/...ore/src/main/java/com/netflix/zuul/netty/connectionpool/NettyClientConnectionFactory.java
@@ -16,6 +16,8 @@
 
 package com.netflix.zuul.netty.connectionpool;
 
+import com.netflix.spectator.api.Counter;
+import com.netflix.zuul.netty.SpectatorUtils;
 import com.netflix.zuul.netty.server.Server;
 import com.netflix.zuul.passport.CurrentPassport;
 import io.netty.bootstrap.Bootstrap;
@@ -40,6 +42,7 @@ public class NettyClientConnectionFactory {
 
  private final ConnectionPoolConfig connPoolConfig;
  private final ChannelInitializer<? extends Channel> channelInitializer;
+ private final Counter unresolvedDiscoveryHost;
 
  private static final Logger LOGGER = LoggerFactory.getLogger(NettyClientConnectionFactory.class);
 
@@ -48,6 +51,8 @@ public class NettyClientConnectionFactory {
  final ChannelInitializer<? extends Channel> channelInitializer) {
  this.connPoolConfig = connPoolConfig;
  this.channelInitializer = channelInitializer;
+ this.unresolvedDiscoveryHost = SpectatorUtils.newCounter("unresolvedDiscoveryHost",
+ connPoolConfig.getOriginName() == null ? "unknownOrigin" : connPoolConfig.getOriginName());
  }
 
  public ChannelFuture connect(final EventLoop eventLoop, String host, final int port, CurrentPassport passport) {
@@ -79,6 +84,7 @@ public ChannelFuture connect(final EventLoop eventLoop, String host, final int p
  ZuulBootstrap zuulBootstrap = new ZuulBootstrap(bootstrap);
  if (!zuulBootstrap.getResolver(eventLoop).isResolved(socketAddress)) {
  LOGGER.warn("NettyClientConnectionFactory got an unresolved server address, host: " + host + ", port: " + port);
+ unresolvedDiscoveryHost.increment();
  }
  return bootstrap.connect();
  }