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

Add separate profiles for Varnish cache#7727

Merged
limited merged 1 commit intoapache:masterfrom
AbdelrahmanElawady:varnish-profile
Sep 1, 2023
Merged

Add separate profiles for Varnish cache#7727
limited merged 1 commit intoapache:masterfrom
AbdelrahmanElawady:varnish-profile

Conversation

@AbdelrahmanElawady
Copy link
Copy Markdown
Member

Create separate profiles for Varnish cache so users won't need to change cache profiles manually.

Closes: #7721


Which Traffic Control components are affected by this PR?

  • CDN in a Box

What is the best way to verify this PR?

Run CIAB with Varnish and check Traffic Portal that the right profile is associated with both Varnish and ATS:

  • Change Dockerfile in edge server to point to varnish/Dockerfile instead of cache/Dockerfile
  • Change edge status to ONLINE (until TM integration is done).
  • Run CIAB and expose ports with docker-compose -f docker-compose.yml -f docker-compose.expose-ports.yml up --build
  • Open Traffic Portal and check profile associated with each cache server.

Note: this applies to Mid-01 and Mid-02 too. The only issue is if running all servers with ONLINE status.

Note: new profiles is the same as ATS ones with description, name and parameters that cause issues (chkconfig and package) changed. easiest way to check is to run diff for ATS and Varnish profiles,

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

PR submission checklist

@AbdelrahmanElawady AbdelrahmanElawady added low impact affects only a small portion of a CDN, and cannot itself break one cdn-in-a-box related to the Docker-based CDN-in-a-Box system improvement The functionality exists but it could be improved in some way. low difficulty the estimated level of effort to resolve this issue is low Varnish related to Varnish labels Aug 16, 2023
@zrhoffman zrhoffman removed the low difficulty the estimated level of effort to resolve this issue is low label Aug 25, 2023
@limited limited merged commit 618ddf6 into apache:master Sep 1, 2023
@AbdelrahmanElawady AbdelrahmanElawady deleted the varnish-profile branch September 1, 2023 16:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cdn-in-a-box related to the Docker-based CDN-in-a-Box system 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 Varnish related to Varnish

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Create separate profiles for Varnish cache in CIAB

3 participants