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

CDN-in-a-Box image rebuilding speedups#4823

Merged
mitchell852 merged 5 commits into
apache:masterfrom
zrhoffman:ciab-building-speedups
Jun 30, 2020
Merged

CDN-in-a-Box image rebuilding speedups#4823
mitchell852 merged 5 commits into
apache:masterfrom
zrhoffman:ciab-building-speedups

Conversation

@zrhoffman
Copy link
Copy Markdown
Member

@zrhoffman zrhoffman commented Jun 24, 2020

What does this PR (Pull Request) do?

  • This PR fixes is not related to any Issue

In the CDN-in-a-Box dockerfiles, RPMs should be added after dependencies are installed so you don't need to rebuild dependencies every time you rebuild the image with a new RPM. This PR accomplishes that for the Traffic Monitor, Traffic Portal, Traffic Router, and Traffic Stats CDN-in-a-Box dockerfiles.

Which Traffic Control components are affected by this PR?

  • CDN in a Box

What is the best way to verify this PR?

Make sure CDN-in-a-Box builds

The following criteria are ALL met by this PR

  • This PR does not require tests
  • This PR does not require documentation
  • An update to CHANGELOG.md is not necessary
  • This PR includes any and all required license headers
  • This PR does not include a database migration
  • This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

@mitchell852 mitchell852 added 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. labels Jun 30, 2020
@zrhoffman zrhoffman force-pushed the ciab-building-speedups branch from 2882dc4 to 3e21485 Compare June 30, 2020 20:16
@zrhoffman
Copy link
Copy Markdown
Member Author

Rebased onto latest master

Copy link
Copy Markdown
Member

@mitchell852 mitchell852 left a comment

Choose a reason for hiding this comment

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

LGTM and builds from scratch just fine.

@mitchell852 mitchell852 merged commit 9f56bae into apache:master Jun 30, 2020
@zrhoffman zrhoffman deleted the ciab-building-speedups branch July 27, 2020 04:53
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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants