Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make Traffic Router ignore ANY_MAP Delivery Services #1684

Merged
merged 1 commit into from Dec 20, 2017

Conversation

rawlinp
Copy link
Contributor

@rawlinp rawlinp commented Dec 19, 2017

Delivery Services of type ANY_MAP should be completely ignored by
Traffic Router, even if they're set to 'active'. An active ANY_MAP DS is
treated like an HTTP DS to the Traffic Router (which is bad).

In practice ANY_MAP DSes have been set to inactive (which makes them
ignored by TR but still able to add "raw remap text" lines to
remap.config), but this makes it so that even if they're set to active
they're still ignored by TR.

Delivery Services of type ANY_MAP should be completely ignored by
Traffic Router, even if they're set to 'active'. An active ANY_MAP DS is
treated like an HTTP DS to the Traffic Router (which is bad).

In practice ANY_MAP DSes have been set to inactive (which makes them
ignored by TR but still able to add "raw remap text" lines to
remap.config), but this makes it so that even if they're set to active
they're still ignored by TR.
@rawlinp
Copy link
Contributor Author

rawlinp commented Dec 19, 2017

Fixes #1121

@asfgit
Copy link
Contributor

asfgit commented Dec 19, 2017

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/811/
Test PASSed.

@mitchell852 mitchell852 added the Traffic Ops related to Traffic Ops label Dec 19, 2017
@elsloo elsloo merged commit 44124db into apache:master Dec 20, 2017
@rawlinp rawlinp deleted the tr-ignore-anymap branch April 13, 2018 21:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Traffic Ops related to Traffic Ops
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants