Skip to content
This repository was archived by the owner on Nov 24, 2025. It is now read-only.

Conversation

@srijeet0406
Copy link
Contributor

Closes: #6197


Which Traffic Control components are affected by this PR?

  • Traffic Ops

What is the best way to verify this PR?

Make sure all the tests pass.
Run TO locally and make sure that a call to /deliveryservices/:id/routing returns the data for traffic routers in only the requested delivery service's CDN.

If this is a bugfix, which Traffic Control versions contained the bug?

  • master

PR submission checklist

@ocket8888 ocket8888 added bug something isn't working as intended Traffic Ops related to Traffic Ops labels Oct 15, 2021
@srijeet0406 srijeet0406 force-pushed the CDN-14366-TR-CDN-specific branch from bb16d41 to 39d710b Compare October 18, 2021 15:41
Copy link
Contributor

@mattjackson220 mattjackson220 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! code looks good, tests pass, tested locally and works as expected.

@mattjackson220 mattjackson220 merged commit 82e8db6 into apache:master Oct 26, 2021
@zrhoffman zrhoffman added this to the 6.0.1 milestone Nov 5, 2021
zrhoffman pushed a commit that referenced this pull request Nov 5, 2021
* Make only CDN specific queries for FQDNs

* fix ISE

(cherry picked from commit 82e8db6)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug something isn't working as intended Traffic Ops related to Traffic Ops

Projects

None yet

Development

Successfully merging this pull request may close these issues.

TO /deliveryservices/:id/routing makes requests to all TRs instead of by CDN

4 participants