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

CiaB: Use rpm command instead of yum to avoid fetching repo metadata that will go unused#4943

Merged
ocket8888 merged 2 commits intoapache:masterfrom
zrhoffman:rpm-not-yum
Aug 10, 2020
Merged

CiaB: Use rpm command instead of yum to avoid fetching repo metadata that will go unused#4943
ocket8888 merged 2 commits intoapache:masterfrom
zrhoffman:rpm-not-yum

Conversation

@zrhoffman
Copy link
Member

@zrhoffman zrhoffman commented Aug 10, 2020

What does this PR (Pull Request) do?

This PR changes CDN-in-a-Box to install the ATC components directly using rpm instead of through yum, which makes syncing yum repos unnecessary if the RPM installation layers are the only ones needing to be rebuilt. This can save several minutes building CDN-in-a-Box, depending on your Internet speed.

  • This PR is not related to any Issue

Which Traffic Control components are affected by this PR?

  • CDN in a Box

What is the best way to verify this PR?

  • Ensure CDN-in-a-Box builds successfully and that the readiness container exits successfully

The following criteria are ALL met by this PR

  • CDN-in-a-Box change, no tests necessary
  • Behavior unchanged, no additional documentation necessary
  • An update to CHANGELOG.md is not necessary
  • This PR includes any and all required license headers
  • This PR ensures that database migration sequence is correct OR 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 the cdn-in-a-box related to the Docker-based CDN-in-a-Box system label Aug 10, 2020
@zrhoffman zrhoffman requested a review from ocket8888 August 10, 2020 18:44
Copy link
Contributor

@ocket8888 ocket8888 left a comment

Choose a reason for hiding this comment

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

Builds fine, CiaB readiness container exits

@ocket8888 ocket8888 merged commit 6d13163 into apache:master Aug 10, 2020
@zrhoffman zrhoffman deleted the rpm-not-yum branch October 4, 2020 20:29
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants