Skip to content

Conversation

Frapschen
Copy link
Contributor

Description

fix: #39646

@iblancasa
Copy link
Contributor

iblancasa commented Apr 25, 2025

LGTM, just a nit. Also, fix CI

@Frapschen
Copy link
Contributor Author

the warn log will look like:

2025-04-26T00:10:25.806+0800    warn    [email protected]/connector.go:111  DimensionsCacheSize is deprecated, please use AggregationCardinalityLimit instead.      {"otelcol.component.id": "spanmetrics", "otelcol.component.kind": "connector", "otelcol.signal": "traces", "otelcol.signal.output": "metrics"}

@atoulme atoulme merged commit 1d452e5 into open-telemetry:main Apr 25, 2025
173 checks passed
@github-actions github-actions bot added this to the next release milestone Apr 25, 2025
@Frapschen Frapschen deleted the fix-issue-39646 branch April 25, 2025 18:20
vincentfree pushed a commit to ing-bank/opentelemetry-collector-contrib that referenced this pull request May 6, 2025
…CacheSize (open-telemetry#39648)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description
fix:
open-telemetry#39646
vincentfree pushed a commit to ing-bank/opentelemetry-collector-contrib that referenced this pull request May 20, 2025
…CacheSize (open-telemetry#39648)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description
fix:
open-telemetry#39646
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Deprecate the unused configuration DimensionsCacheSize

4 participants