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

Fix or remove a few unhelpful debug logs#2953

Merged
elsloo merged 1 commit into
apache:masterfrom
rawlinp:fix-debugln-format
Oct 29, 2018
Merged

Fix or remove a few unhelpful debug logs#2953
elsloo merged 1 commit into
apache:masterfrom
rawlinp:fix-debugln-format

Conversation

@rawlinp
Copy link
Copy Markdown
Contributor

@rawlinp rawlinp commented Oct 23, 2018

What does this PR do?

Fixes or removes a few unhelpful debug logs that I've found.

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?

cd trafficcontrol/traffic_ops/traffic_ops_golang
go test -v origin/*
go test -v server/*

Start up traffic_ops_golang with debug logging enabled and send it any request, grep the output for DEBUG plugins.OnRequest calling.

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 includes a database migration (ensure that migration sequence is correct)
  • This PR fixes a serious security flaw. Read more: www.apache.org/security

@rawlinp rawlinp added new feature A new feature, capability or behavior Traffic Ops related to Traffic Ops trivial labels Oct 23, 2018
@asfgit
Copy link
Copy Markdown
Contributor

asfgit commented Oct 23, 2018

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

@elsloo elsloo self-assigned this Oct 29, 2018
@elsloo elsloo self-requested a review October 29, 2018 15:38
@elsloo elsloo merged commit abc8c1e into apache:master Oct 29, 2018
@rawlinp rawlinp deleted the fix-debugln-format branch October 29, 2018 20:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

new feature A new feature, capability or behavior Traffic Ops related to Traffic Ops

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants