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

t3c makes too many environment-level assumptions w/o any ability to override #7857

Open
mcclory opened this issue Nov 8, 2023 · 1 comment
Labels
cache-config Cache config generation improvement The functionality exists but it could be improved in some way. low impact affects only a small portion of a CDN, and cannot itself break one medium difficulty the estimated level of effort to resolve this issue is medium

Comments

@mcclory
Copy link

mcclory commented Nov 8, 2023

Current behavior:

  • When running t3c apply on a given instance running trafficserver, it makes a bunch of assumptions about pathing
  • if you use the OS-level install packages, at least on rocky 8, the plugin path is not at the pre-configured/default path of /opt/trafficserver/libexec/trafficserver and there doesn't appear to be a meaningful way to pass a value that would get passed to the t3c-check-refs calls that happen within t3c apply

Impact:

  • It's not possible to use t3c properly on a cache host (edge/mid) that's been installed with OS-level packages

New behavior:

  • Proposal would be to leverage environment variable similar to TS_HOME to set the default of this value to make it possible to pass an environment-specific/custom path where trafficserver plugins can be found
    • simplest solution would be to introduce a TS_PLUGIN_ROOT env var check which would simply read the env var in and default to the current value (/opt/trafficserver/libexec/trafficserver)
    • while not super ideal, it would at least allow for cache tier servers to use os package installations with some basic env-level overrides in order to allow for t3c to appropriately configure and validate the plugins.config file
      `
@mcclory mcclory added the improvement The functionality exists but it could be improved in some way. label Nov 8, 2023
@ocket8888
Copy link
Contributor

Related: #6205

@ocket8888 ocket8888 added low impact affects only a small portion of a CDN, and cannot itself break one cache-config Cache config generation medium difficulty the estimated level of effort to resolve this issue is medium labels Nov 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cache-config Cache config generation improvement The functionality exists but it could be improved in some way. low impact affects only a small portion of a CDN, and cannot itself break one medium difficulty the estimated level of effort to resolve this issue is medium
Projects
None yet
Development

No branches or pull requests

2 participants