[GitHub] [trafficserver] maskit commented on a diff in pull request #9285: Do not track config file periodically

[GitBox <gi...@apache.org>]

maskit commented on code in PR #9285:
URL: https://github.com/apache/trafficserver/pull/9285#discussion_r1061141887


##########
mgmt/RecordsConfig.cc:
##########
@@ -263,6 +263,8 @@ static const RecordElement RecordsConfig[] =
   ,
   {RECT_CONFIG, "proxy.config.admin.api.restricted", RECD_INT, "0", RECU_RESTART_TM, RR_NULL, RECC_INT, "[0-1]", RECA_NULL}
   ,
+  {RECT_CONFIG, "proxy.config.periodic_track_config_files", RECD_INT, "0", RECU_RESTART_TM, RR_NULL, RECC_INT, "[0-1]", RECA_NULL}

Review Comment:
   I understand this is not going to be a small change, but I feel like `config_tracking_interval` that accepts 0 (no tracking) or N (second) is a more common way of configuring this kind of things on ATS.



-- 
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: github-unsubscribe@trafficserver.apache.org

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