Skip to content

[chore][receiver/tlscheck] Remove excessive "Peer Certificates" log#41705

Merged
songy23 merged 1 commit into
open-telemetry:mainfrom
Fiery-Fenix:chore/tlscheckreceiver-peer-cert-logging
Aug 1, 2025
Merged

[chore][receiver/tlscheck] Remove excessive "Peer Certificates" log#41705
songy23 merged 1 commit into
open-telemetry:mainfrom
Fiery-Fenix:chore/tlscheckreceiver-peer-cert-logging

Conversation

@Fiery-Fenix
Copy link
Copy Markdown
Contributor

Description

Switch "Peer Certificates" log line from Info to Debug level to reduce excessive logging during tlscheckreceiver scrape loop

Copy link
Copy Markdown
Contributor

@michael-burt michael-burt left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution

Copy link
Copy Markdown
Contributor

@dehaansa dehaansa left a comment

Choose a reason for hiding this comment

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

I don't think this needs a changelog, feel free to disagree whoever pops through to review the ready-to-merge tag.

@dehaansa dehaansa added ready to merge Code review completed; ready to merge by maintainers Skip Changelog PRs that do not require a CHANGELOG.md entry labels Aug 1, 2025
@songy23 songy23 merged commit f32a732 into open-telemetry:main Aug 1, 2025
201 of 202 checks passed
Dylan-M pushed a commit to Dylan-M/opentelemetry-collector-contrib that referenced this pull request Aug 5, 2025
…pen-telemetry#41705)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description
Switch "Peer Certificates" log line from Info to Debug level to reduce
excessive logging during `tlscheckreceiver` scrape loop
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready to merge Code review completed; ready to merge by maintainers receiver/tlscheck Skip Changelog PRs that do not require a CHANGELOG.md entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants