Add t3c flag for local ATS version for config gen #7032
Merged
Conversation
This file contains 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
rob05c
added
new feature
rob05c
force-pushed
the
add-t3c-ats-version-flag
branch
3 times, most recently
from
e7d8c65
to
f5be26f
Compare
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
rob05c
force-pushed
the
add-t3c-ats-version-flag
branch
from
f5be26f
to
ec595dc
Compare
rob05c
force-pushed
the
add-t3c-ats-version-flag
branch
from
ec595dc
to
de1b04d
Compare
jpappa200
approved these changes
Aug 24, 2022
ocket8888
approved these changes
Aug 26, 2022
zrhoffman
pushed a commit
to zrhoffman/trafficcontrol
that referenced
this pull request
Oct 2, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Adds a new
t3c-apply--local-ats-versionflag to discover and use the local installed ATS version for config generation, rather than the Traffic Ops ServerpackageParameter.This, combined with the existing
--install-packages=falseflag, allows users to manage the ATS operating system package with an external tool (e.g. Ansible, Puppet, etc).Which Traffic Control components are affected by this PR?
t3c, formerly ORT)What is the best way to verify this PR?
Run tests. Run t3c-apply with the new
--local-ats-versionflag with a local installed Traffic server major version different from any Server Package Parameter, verify local ATS version is used to generate rather than package Parameter.If this is a bugfix, which Traffic Control versions contained the bug?
Not a bug fix.
PR submission checklist