This repository was archived by the owner on Nov 24, 2025. It is now read-only.
Add Cache Config Generator for All Configs#3917
Closed
rob05c wants to merge 32 commits into
Closed
Conversation
Adds a client-side ATS config gen interceptor to ORT, with an initial parent.config gen and passing everything else to TO. Plan is to add all configs to be client-side generated.
Also adds a client function to specify server and DS IDs, as well as a missing client func for the existing deliveryservice?cdn param.
For debugging, or emergencies.
Also changes atstccfg to do things ORT needed: - changed to return the HTTP code as the exit code, on error - added CLI option to return which configs are generated (vs proxied) - added retry num option, for failed TO attempts - changed to use lib/tc-log, and take args for where to log - fixed missing license headers - added integrity check via SHA512 or Content-length headers
Adds error logging whenever the config gen inserts multiple cacheurl or cachekey plugins on the same line. This isn't supported in ATS, and will usually do the wrong thing. But, we've always generated config like this, so for now, this doesn't change the config gen, but it logs the error
Adds a client-side ATS config gen interceptor to ORT, with an initial parent.config gen and passing everything else to TO. Plan is to add all configs to be client-side generated.
Adds a client-side ATS config gen interceptor to ORT, with an initial parent.config gen and passing everything else to TO. Plan is to add all configs to be client-side generated.
Merging: to-cache-side-configs-plus-cdns to-cache-side-configs-plus-to-go-atsconfig-remap
rob05c
added
the
WIP
Contributor
|
Refer to this link for build results (access rights to CI server needed): |
rob05c
force-pushed
the
to-cache-side-configs-all
branch
from
0aeeacc to
69daa37
Compare
Contributor
|
Refer to this link for build results (access rights to CI server needed): |
Member
Author
|
This was a demonstration, for anyone interested to try out the configs before. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add Cache Config Generator for All Configs.
This is an example PR, to allow previewing the cache config generation which is coming. This PR shouldn't be merged as-is; rather, we want to review and merge smaller pieces, just for ease of reviewing, because it's so large.
But if you want to try cache-side config gen now, you can pull this PR and try it out.
What does this PR (Pull Request) do?
Which Traffic Control components are affected by this PR?
What is the best way to verify this PR?
If this is a bug fix, what versions of Traffic Control are affected?
The following criteria are ALL met by this PR