You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficcontrol.apache.org by ne...@apache.org on 2016/11/07 19:30:02 UTC

[16/21] incubator-trafficcontrol git commit: Fix TM2 duplicate strings to enums

Fix TM2 duplicate strings to enums


Project: http://git-wip-us.apache.org/repos/asf/incubator-trafficcontrol/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-trafficcontrol/commit/1aa0c0ba
Tree: http://git-wip-us.apache.org/repos/asf/incubator-trafficcontrol/tree/1aa0c0ba
Diff: http://git-wip-us.apache.org/repos/asf/incubator-trafficcontrol/diff/1aa0c0ba

Branch: refs/heads/master
Commit: 1aa0c0ba41cbfb6637df76cd290130e6165087b6
Parents: 3a73422
Author: Robert Butts <ro...@gmail.com>
Authored: Thu Nov 3 09:55:03 2016 -0600
Committer: Dave Neuman <ne...@apache.org>
Committed: Mon Nov 7 12:29:08 2016 -0700

----------------------------------------------------------------------
 .../experimental/traffic_monitor/manager/monitorconfig.go     | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-trafficcontrol/blob/1aa0c0ba/traffic_monitor/experimental/traffic_monitor/manager/monitorconfig.go
----------------------------------------------------------------------
diff --git a/traffic_monitor/experimental/traffic_monitor/manager/monitorconfig.go b/traffic_monitor/experimental/traffic_monitor/manager/monitorconfig.go
index 4ff0cf4..2505b9b 100644
--- a/traffic_monitor/experimental/traffic_monitor/manager/monitorconfig.go
+++ b/traffic_monitor/experimental/traffic_monitor/manager/monitorconfig.go
@@ -190,11 +190,12 @@ func monitorConfigListen(
 
 			cacheName := enum.CacheName(srv.HostName)
 
-			if srv.Status == "ONLINE" {
+			srvStatus := enum.CacheStatusFromString(srv.Status)
+			if srvStatus == enum.CacheStatusOnline {
 				localStates.SetCache(cacheName, peer.IsAvailable{IsAvailable: true})
 				continue
 			}
-			if srv.Status == "OFFLINE" {
+			if srvStatus == enum.CacheStatusOffline {
 				continue
 			}
 			// seed states with available = false until our polling cycle picks up a result
@@ -222,7 +223,7 @@ func monitorConfigListen(
 			if srv.HostName == staticAppData.Hostname {
 				continue
 			}
-			if srv.Status != "ONLINE" {
+			if enum.CacheStatusFromString(srv.Status) != enum.CacheStatusOnline {
 				continue
 			}
 			// TODO: the URL should be config driven. -jse