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

Remove wrong error logs#4750

Merged
dneuman64 merged 1 commit into
apache:masterfrom
rob05c:to-fix-wrong-error-logs
Jun 2, 2020
Merged

Remove wrong error logs#4750
dneuman64 merged 1 commit into
apache:masterfrom
rob05c:to-fix-wrong-error-logs

Conversation

@rob05c
Copy link
Copy Markdown
Member

@rob05c rob05c commented Jun 2, 2020

What does this PR (Pull Request) do?

Removes debug error logs that accidentally got left in.

It's only removing log messages, nothing to test.
No docs, no changelog, no interface change.

Not actually a behavioral bug, but labelling minor bug, because it's not a feature and it's wrong.

  • This PR is not related to any other Issue

Which Traffic Control components are affected by this PR?

  • Traffic Ops

What is the best way to verify this PR?

Observe code, observe logs are not errors and should not exist.

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

  • 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)

Additional Information

@rob05c rob05c added bug something isn't working as intended low impact affects only a small portion of a CDN, and cannot itself break one labels Jun 2, 2020
Copy link
Copy Markdown
Member

@zrhoffman zrhoffman left a comment

Choose a reason for hiding this comment

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

Confirmed wrong error logs are removed.

@dneuman64 dneuman64 merged commit b85f350 into apache:master Jun 2, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug something isn't working as intended low impact affects only a small portion of a CDN, and cannot itself break one

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants