feat(otelcol): Add otelcol.processor.cardinality_guardian component - #6855
Open
bhargav2427 wants to merge 3 commits into
Open
feat(otelcol): Add otelcol.processor.cardinality_guardian component#6855bhargav2427 wants to merge 3 commits into
bhargav2427 wants to merge 3 commits into
Conversation
Contributor
Signed commits report1 of 3 commits between
This repository requires all commits to be signed. See GitHub docs on commit signature verification. |
Author
|
Hi @clayton-cornell, could you please help move this PR forward when you get a chance? |
Contributor
|
Hi @bhargav2427 thanks for the PR. A few things,
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Brief description of Pull Request
Adds
otelcol.processor.cardinality_guardian, a wrapper over the upstream OpenTelemetry Collector Contribcardinalityguardianprocessor, at Experimental stability. The processor detects metric labels with abnormal cardinality growth (delta-based, using HyperLogLog++ sketches) and can tag, replace, or strip and reaggregate the offending attributes.Since the upstream processor was first released in opentelemetry-collector-contrib v0.154.0, this PR also updates all OpenTelemetry Collector and Collector Contrib dependencies from v0.153.0 to v0.155.0 (stable modules to v1.61.0, matching the versions already partially pinned in go.mod), and regenerates the Alloy OTel Collector distribution.
The dependency update required one component change: contrib v0.155.0 added
max_broker_write_bytesto the Kafka producer configuration with a 100 MiB franz-go minimum, sootelcol.exporter.kafkanow exposes amax_broker_write_bytesattribute (default104857600) in itsproducerblock, with its converter, tests, and documentation updated accordingly.Also included for the new component: registration in
internal/component/all, an otelcolconvert converter with test fixtures, reference documentation, and an entry in the OTel components list.Pull Request Details
Issue(s) fixed by this Pull Request
Notes to the Reviewer
PR Checklist
One heads-up for your human-written sections: this PR bundles a dependency bump + a kafka exporter change + a new component, which may violate the "one logical change per PR" rule in AGENTS.md — reviewers may ask you to split the OTel v0.155.0 bump (with the kafka fix) from the new component.