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/01/13 19:22:17 UTC
[GitHub] [trafficcontrol] rawlinp opened a new issue #6512: Remove Traffic Monitor dependency on CRConfig data
rawlinp opened a new issue #6512:
URL: https://github.com/apache/trafficcontrol/issues/6512
## This Improvement request (usability, performance, tech debt, etc.) affects these Traffic Control components:
- Traffic Monitor
## Current behavior:
As of https://github.com/apache/trafficcontrol/pull/6443, TM will no longer overwrite its monitoring.json snapshot data with corresponding CRConfig snapshot data if the monitoring.json contains IP addresses for the TMs, meaning it now has parity with the CRConfig data. However, it still needs the CRConfig in order to filter out delivery services from monitoring.json which aren't present in the CRConfig. That was fixed in #6443 in that the monitoring.json no longer contains delivery services from other CDNs to which the TM doesn't belong.
## New behavior:
Once ATC 6.1 is released with the changes from #6443, we should be able to remove this fallback behavior from TM (and possibly its reliance on the CRConfig altogether) in the next major/minor release. That is because monitoring.json snapshots as of TO 6.1 will contain all the necessary data and will no longer need to rely on the CRConfig snapshot.
--
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
[GitHub] [trafficcontrol] mitchell852 commented on issue #6512: Remove Traffic Monitor dependency on CRConfig data
Posted by GitBox <gi...@apache.org>.
mitchell852 commented on issue #6512:
URL: https://github.com/apache/trafficcontrol/issues/6512#issuecomment-1015602859
@rawlinp any idea what the impact and effort of this improvement is?
--
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
[GitHub] [trafficcontrol] ocket8888 closed issue #6512: Reduce Traffic Monitor dependency on CRConfig data
Posted by GitBox <gi...@apache.org>.
ocket8888 closed issue #6512:
URL: https://github.com/apache/trafficcontrol/issues/6512
--
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