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

Apache TC builder images#4752

Merged
ocket8888 merged 1 commit into
apache:masterfrom
zrhoffman:apache-builder-images
Jun 8, 2020
Merged

Apache TC builder images#4752
ocket8888 merged 1 commit into
apache:masterfrom
zrhoffman:apache-builder-images

Conversation

@zrhoffman
Copy link
Copy Markdown
Member

@zrhoffman zrhoffman commented Jun 2, 2020

What does this PR (Pull Request) do?

Docker Hub repos:
https://hub.docker.com/r/apache/traffic_docs_builder
https://hub.docker.com/r/apache/traffic_grove_builder
https://hub.docker.com/r/apache/traffic_grovetccfg_builder
https://hub.docker.com/r/apache/traffic_monitor_builder
https://hub.docker.com/r/apache/traffic_ops_builder
https://hub.docker.com/r/apache/traffic_portal_builder
https://hub.docker.com/r/apache/traffic_router_builder
https://hub.docker.com/r/apache/traffic_source_tarballer
https://hub.docker.com/r/apache/traffic_stats_builder

Which Traffic Control components are affected by this PR?

  • Build system

What is the best way to verify this PR?

  • Build RPMs for CDN-in-a-Box and verify that the RPMs that the CiaB dockerfiles expect are found and that it runs as expected
  • Verify that the RPMs that CDN-in-a-Box does not currently use are still generated
  • Verifying that the source is still tarred and gzipped

If this is a bug fix, what versions of Traffic Control are affected?

Not a bug fix.

The following criteria are ALL met by this PR

  • The builder images do not have tests
  • This PR changes image tags only, no additional documentation is necessary
  • 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)

@ocket8888 ocket8888 added build related to the build process new feature A new feature, capability or behavior process labels Jun 3, 2020
Copy link
Copy Markdown
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.

Built the RPMs, started CiaB, confirmed that the 'readiness' container exited successfully after curling DS content.

@ocket8888 ocket8888 merged commit c4d197d into apache:master Jun 8, 2020
@zrhoffman zrhoffman deleted the apache-builder-images branch June 8, 2020 22:45
@zrhoffman zrhoffman added the ci label Sep 17, 2020
@zrhoffman zrhoffman added automation related to automated testing/deployment/packaging etc. and removed ci labels Sep 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

automation related to automated testing/deployment/packaging etc. build related to the build process new feature A new feature, capability or behavior process

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants