Passing tests

Yes, that was a bug discovered in OutlierDetection
(EnvoyServerProtoData). And yes, now XdsDepManager combines EAGs
into a single endpoint.

Author: ejona86

xds/src/main/java/io/grpc/xds/EnvoyServerProtoData.java

@@ -328,7 +328,7 @@ static OutlierDetection fromEnvoyOutlierDetection(
         Integer minimumHosts = envoyOutlierDetection.hasSuccessRateMinimumHosts()
             ? envoyOutlierDetection.getSuccessRateMinimumHosts().getValue() : null;
         Integer requestVolume = envoyOutlierDetection.hasSuccessRateRequestVolume()
-            ? envoyOutlierDetection.getSuccessRateMinimumHosts().getValue() : null;
+            ? envoyOutlierDetection.getSuccessRateRequestVolume().getValue() : null;
 
         successRateEjection = SuccessRateEjection.create(stdevFactor, enforcementPercentage,
             minimumHosts, requestVolume);

xds/src/main/java/io/grpc/xds/XdsDependencyManager.java

@@ -41,6 +41,7 @@
 import io.grpc.xds.client.XdsClient;
 import io.grpc.xds.client.XdsClient.ResourceWatcher;
 import io.grpc.xds.client.XdsResourceType;
+import java.net.SocketAddress;
 import java.net.URI;
 import java.net.URISyntaxException;
 import java.util.ArrayList;
@@ -83,7 +84,7 @@ private enum TrackedWatcherTypeEnum {
 
   private static final int MAX_CLUSTER_RECURSION_DEPTH = 16; // Specified by gRFC A37
 
-  static boolean enableLogicalDns = false;
+  static boolean enableLogicalDns = true;
 
   private final String listenerName;
   private final XdsClient xdsClient;
@@ -394,10 +395,13 @@ private static StatusOr<XdsEndpointResource.EdsUpdate> dnsToEdsUpdate(
       return StatusOr.fromStatus(dnsData.getStatus());
     }
 
-    List<Endpoints.LbEndpoint> endpoints = new ArrayList<>();
+    List<SocketAddress> addresses = new ArrayList<>();
     for (EquivalentAddressGroup eag : dnsData.getValue()) {
-      endpoints.add(Endpoints.LbEndpoint.create(eag, 1, true, dnsHostName, ImmutableMap.of()));
+      addresses.addAll(eag.getAddresses());
     }
+    EquivalentAddressGroup eag = new EquivalentAddressGroup(addresses);
+    List<Endpoints.LbEndpoint> endpoints = ImmutableList.of(
+        Endpoints.LbEndpoint.create(eag, 1, true, dnsHostName, ImmutableMap.of()));
     LocalityLbEndpoints lbEndpoints =
         LocalityLbEndpoints.create(endpoints, 1, 0, ImmutableMap.of());
     return StatusOr.fromValue(new XdsEndpointResource.EdsUpdate(