[GitHub] [trafficcontrol] jrushford commented on a change in pull request #6341: Changes the t3c process locking so that no two simultaneous t3c instances run.

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

jrushford commented on a change in pull request #6341:
URL: https://github.com/apache/trafficcontrol/pull/6341#discussion_r747569131



##########
File path: cache-config/t3c-apply/t3c-apply.go
##########
@@ -69,6 +69,9 @@ func main() {
 	} else if cfg == (config.Cfg{}) { // user used the --help option
 		os.Exit(Success)
 	}
+	if !lock.GetLock("/var/run/t3c.lock") {

Review comment:
       Not in my opinion.  The idea with the locking is limit t3c processes and logins to TO.  Why allow a t3c process running in ReportOnly, it still has to login to TO.




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