Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CIAB: Fix the environment variable and cron jobs #3508

Merged
merged 1 commit into from
Apr 26, 2019

Conversation

Shihta
Copy link
Contributor

@Shihta Shihta commented Apr 19, 2019

What does this PR (Pull Request) do?

  1. Fix the environment variable in trafficops-go. SSL keys can not be set without X509-related environment variable.
  2. Fix the traffic_ops_ort path from /usr/bin/traffic_ops_ort to /usr/local/bin/traffic_ops_ort because the python version is upgraded

No procedural change, so no tests or doc or CHANGELOG changes needed.

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. The cached demo page can be retrieved.
  3. Enter mid or edge container. There should not any error in /var/log/ort.log

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

  • master
  • 3.0.0
  • Basically every active branch

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
Contributor

asfgit commented Apr 19, 2019

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

@mitchell852 mitchell852 added the cdn-in-a-box related to the Docker-based CDN-in-a-Box system label Apr 19, 2019
@dangogh dangogh merged commit 2cef0d0 into apache:master Apr 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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