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

Conversation

@rob05c
Copy link
Member

@rob05c rob05c commented Aug 22, 2018

What does this PR do?

Fix TO tests for parameter change

Which TC components are affected by this PR?

  • Documentation
  • Grove
  • Traffic Analytics
  • Traffic Monitor
  • Traffic Ops
  • Traffic Ops ORT
  • Traffic Portal
  • Traffic Router
  • Traffic Stats
  • Traffic Vault
  • Other _________

What is the best way to verify this PR?

go test

Check all that apply

  • This PR includes tests
  • This PR includes documentation updates
  • This PR includes an update to CHANGELOG.md
  • This PR includes all required license headers
  • This PR does NOT fix a serious security flaw. Read more: www.apache.org/security

@rob05c rob05c added the tests related to tests and/or testing infrastructure label Aug 22, 2018
@dangogh dangogh self-assigned this Aug 22, 2018
Copy link
Member

@dangogh dangogh left a comment

Choose a reason for hiding this comment

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

tests pass now

@dangogh dangogh merged commit f48ae3f into apache:master Aug 22, 2018
@asfgit
Copy link
Contributor

asfgit commented Aug 22, 2018

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

@rob05c rob05c deleted the to-go-test-fix-param-change branch January 2, 2019 20:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

tests related to tests and/or testing infrastructure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants