You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@trafficcontrol.apache.org by GitBox <gi...@apache.org> on 2022/04/21 19:45:02 UTC

[GitHub] [trafficcontrol] ocket8888 commented on a diff in pull request #6769: Document Traffic Monitor configuration files

ocket8888 commented on code in PR #6769:
URL: https://github.com/apache/trafficcontrol/pull/6769#discussion_r855538347


##########
traffic_monitor/config/config.go:
##########
@@ -149,33 +191,33 @@ func GetAccessLogWriter(cfg Config) (io.WriteCloser, error) {
 
 // DefaultConfig is the default configuration for the application, if no configuration file is given, or if a given config setting doesn't exist in the config file.
 var DefaultConfig = Config{
-	MonitorConfigPollingInterval: 5 * time.Second,
-	HTTPTimeout:                  2 * time.Second,
-	PeerOptimistic:               true,
-	PeerOptimisticQuorumMin:      0,
-	StatPolling:                  true,
-	MaxEvents:                    200,
+	CachePollingProtocol:         Both,
+	CRConfigBackupFile:           CRConfigBackupFile,
+	CRConfigHistoryCount:         20000,

Review Comment:
   ¯\\\_(ツ)\_/¯
   
   I didn't change any of these values, just sorted the lines alphabetically. I can change the defaults if these don't make sense, though.



-- 
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.

To unsubscribe, e-mail: issues-unsubscribe@trafficcontrol.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org