zrhoffman commented on a change in pull request #4724:
URL: https://github.com/apache/trafficcontrol/pull/4724#discussion_r442492247
##########
File path:
traffic_router/core/src/main/java/com/comcast/cdn/traffic_control/traffic_router/core/config/ConfigHandler.java
##########
@@ -454,6 +476,31 @@ private void parseCacheConfig(final JsonNode
contentServers, final CacheRegister
return deliveryServiceMap;
}
+ private void parseTopologyConfig(final JsonNode allTopologies, final
Map<String, DeliveryService> deliveryServiceMap, final CacheRegister
cacheRegister) {
+ final Map<String, List<String>> topologyMap = new HashMap<>();
+ allTopologies.fieldNames().forEachRemaining((String
topologyName) -> {
+ final List<String> nodes = new ArrayList<>();
+
allTopologies.get(topologyName).get("nodes").forEach((JsonNode cache) ->
nodes.add(cache.textValue()));
+ topologyMap.put(topologyName, nodes);
+ });
+
+ deliveryServiceMap.forEach((xmlId, ds) -> {
+ final List<DeliveryServiceReference> dsReferences = new
ArrayList<>();
+ try {
+ dsReferences.add(new
DeliveryServiceReference(ds.getId(), ds.getDomain()));
Review comment:
As of af6c35ecd, all delivery service URLs are considered when parsing
the topologies config (including initialization of stat tracking)
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
