rawlinp commented on a change in pull request #6527:
URL: https://github.com/apache/trafficcontrol/pull/6527#discussion_r804080620
##########
File path: lib/go-tc/traffic_monitor.go
##########
@@ -438,7 +438,7 @@ type TrafficMonitor struct {
Profile string `json:"profile"`
// Location is the Name of the Cache Group to which the Traffic Monitor
// belongs - called "Location" for legacy reasons.
- Location string `json:"location"`
+ Location string `json:"cachegroup"`
Review comment:
This wasn't really clean-up, it was more of a bug. The TM snapshot uses
`cachegroup`, but TM was parsing it into this struct using `location`, so it
wasn't actually parsing it at all. I think the intention was that all fields in
the struct should have GoDoc comments.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
