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

CiaB: added log path of trafficops in variables.env#3666

Merged
dangogh merged 1 commit intoapache:masterfrom
Shihta:CIAB_TO_LOG
Jun 27, 2019
Merged

CiaB: added log path of trafficops in variables.env#3666
dangogh merged 1 commit intoapache:masterfrom
Shihta:CIAB_TO_LOG

Conversation

@Shihta
Copy link
Copy Markdown
Contributor

@Shihta Shihta commented Jun 6, 2019

What does this PR (Pull Request) do?

added log path of trafficops in variables.env

Which Traffic Control components are affected by this PR?

  • CDN in a Box

What is the best way to verify this PR?

  1. Start CiaB
  2. You should get TO log by docker logs

Since there is no any new feature so I didn't provide test

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

master

The following criteria are ALL met by this PR

  • This PR includes tests OR I have explained why tests are unnecessary
  • This PR includes documentation OR I have explained why documentation is unnecessary
  • This PR includes an update to CHANGELOG.md OR such an update 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)

@asfgit
Copy link
Copy Markdown
Contributor

asfgit commented Jun 6, 2019

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3833/
Test PASSed.

@mitchell852 mitchell852 added the cdn-in-a-box related to the Docker-based CDN-in-a-Box system label Jun 6, 2019
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.

tested by adjusting some of the values in variables.env and building/running CiaB - works as advertised.

@dangogh dangogh self-assigned this Jun 27, 2019
@dangogh dangogh merged commit 5132621 into apache:master Jun 27, 2019
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.

6 participants