rawlinp opened a new issue #1831: Traffic Monitor shouldn't serve stale Delivery Services data in /publish/CrStates URL: https://github.com/apache/incubator-trafficcontrol/issues/1831 When a Delivery Service is deleted from Traffic Ops and a snapshot is taken, Traffic Monitor should check the new snapshot for missing Delivery Services and remove them from its stored state. Currently a TM will continue to serve `CrStates` json with old Delivery Services that have been removed since the TM was last started. This can have the effect of getting different `CrStates` from TMs that were started at different times, causing a TR to repeatedly update its cache state even though no actual cache state changes have occurred. This is because the `CrStates` json file ends up being different between the two or more TMs because one contains nonexistent Delivery Services. Another related issue that can cause two TMs with the same state to produce different `CrStates` is how it handles arrays of `"disabledLocations"` (this might affect other arrays too). Sometimes `"disabledLocations"` is set to `null` rather than `[]` (an initialized empty array). For the sake of consistency the TM should set empty json array values to one of either `null` or `[]`. Priority: minor Affected TM (go) Version: b419e747f574979a16265b6e1ceaa6d417aec1ba
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services