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

Fix Health Client Formatting#6995

Open
rob05c wants to merge 1 commit intoapache:masterfrom
rob05c:fix-hc-idioms
Open

Fix Health Client Formatting#6995
rob05c wants to merge 1 commit intoapache:masterfrom
rob05c:fix-hc-idioms

Conversation

@rob05c
Copy link
Copy Markdown
Member

@rob05c rob05c commented Jul 29, 2022

Fixes Health Client formatting and Go idioms, primarily GoDoc comments and naming conventions.

Which Traffic Control components are affected by this PR?

  • Traffic Control Health Client (tc-health-client)

What is the best way to verify this PR?

Run Health Client, verify behavior is unchanged.

If this is a bugfix, which Traffic Control versions contained the bug?

Not a bug fix

PR submission checklist

  • [x] This PR has tests no code behavior change, existing behavior is covered by tests
  • [x] This PR has documentation no docs, no interface change
  • [x] This PR has a CHANGELOG.md entry No changelog, no interface change
  • This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

@rob05c rob05c added low impact affects only a small portion of a CDN, and cannot itself break one tech debt rework due to choosing easy/limited solution tc-health-client Traffic Control Health Client labels Jul 29, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

low impact affects only a small portion of a CDN, and cannot itself break one tc-health-client Traffic Control Health Client tech debt rework due to choosing easy/limited solution

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant