[GitHub] [trafficcontrol] rob05c opened a new pull request, #6809: Fix t3c peering ring strategy gen, add tests

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

rob05c opened a new pull request, #6809:
URL: https://github.com/apache/trafficcontrol/pull/6809

   I set out to add tests for the new (experimental) Peering Ring Strategy generation, and discovered it wasn't generating right for Delivery Services with Topologies.
   
   So this includes the fix to generate peering ring strategies for topologies, as well as tests for both topology and non-topology DSes.
   
   ## Which Traffic Control components are affected by this PR?
   - Traffic Control Cache Config (`t3c`, formerly ORT)
   
   ## What is the best way to verify this PR?
   Run tests. Generate config with the new experimental peering ring strategy parameter, for both DSes with and without topologies, verify strategies file contains peering ring directives.
   
   ## If this is a bugfix, which Traffic Control versions contained the bug?
   Not in a release.
   
   
   ## PR submission checklist
   - [x] This PR has tests <!-- If not, please delete this text and explain why this PR does not need tests. -->
   - ~[x] This PR has documentation~ no docs, feature is experimental <!-- If not, please delete this text and explain why this PR does not need documentation. -->
   - ~[x] This PR has a CHANGELOG.md entry~ no changelog, bug was not in a release <!-- A fix for a bug from an ATC release, an improvement, or a new feature should have a changelog entry. -->
   - [x] This PR **DOES NOT FIX A SERIOUS SECURITY VULNERABILITY** (see [the Apache Software Foundation's security guidelines](https://apache.org/security) for details)
   


-- 
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] zrhoffman merged pull request #6809: Fix t3c peering ring strategy generation for topologies, add tests

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

zrhoffman merged PR #6809:
URL: https://github.com/apache/trafficcontrol/pull/6809


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