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

Conversation

@zrhoffman
Copy link
Member

@zrhoffman zrhoffman commented Feb 7, 2024


Which Traffic Control components are affected by this PR?

  • CDN in a Box - TPv2

This PR fixes some issues from #7934 and #7936 where the paths did not consider that TPv2 was moved to the optional CiaB directory.

What is the best way to verify this PR?

cd infrastructure/cdn-in-a-box
make optional/traffic_portal_v2/traffic_portal_v2.rpm
docker-compose -f docker-compose.yml  -f optional/docker-compose.traffic-portal-v2.yml build
docker-compose -f docker-compose.yml  -f optional/docker-compose.traffic-portal-v2.yml up --no-deps

PR submission checklist

@zrhoffman zrhoffman added regression bug a bug in existing functionality introduced by a new version cdn-in-a-box related to the Docker-based CDN-in-a-Box system Traffic Portal v2 Related to the experimental Traffic Portal version 2 labels Feb 7, 2024
Copy link
Contributor

@rimashah25 rimashah25 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Ran the cmds to verify whether all the container including TPv2 run correctly.

@rimashah25 rimashah25 merged commit 8347b39 into apache:master Feb 8, 2024
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 regression bug a bug in existing functionality introduced by a new version Traffic Portal v2 Related to the experimental Traffic Portal version 2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants