Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Implement cardinality limits for metrics streams #1065

Open
lzchen opened this issue May 17, 2023 · 4 comments
Open

Implement cardinality limits for metrics streams #1065

lzchen opened this issue May 17, 2023 · 4 comments
Assignees
@cijothomas
Labels
A-metrics Area: issues related to metrics

Comments

@lzchen
Copy link
Contributor

lzchen commented May 17, 2023

From spec: open-telemetry/opentelemetry-specification#2960

Implement ability to specify cardinality limits per metric stream to help protect against excessive data production from a single instrument.
@lzchen lzchen mentioned this issue May 19, 2023
Merged
@TommyCpp TommyCpp added the A-metrics Area: issues related to metrics label May 25, 2023
@cijothomas
Copy link
Member

Can be closed with #1066

@lzchen
Copy link
Contributor Author

lzchen commented May 26, 2023 •
edited
Loading

Can't be closed yet as the feature to configure the limit has not been implemented yet.

@lalitb lalitb mentioned this issue Dec 21, 2023
Closed
@lalitb lalitb added this to the Metrics SDK Stable milestone Feb 1, 2024
bnjjj added a commit to bnjjj/opentelemetry-rust that referenced this issue Jun 18, 2024
@bnjjj
Implement cardinality limits for metrics streams open-telemetry#1065
166d217 
Signed-off-by: Benjamin Coenen <[email protected]>
@bnjjj bnjjj mentioned this issue Jun 18, 2024
Closed
4 tasks
@cijothomas cijothomas self-assigned this Jul 22, 2024
@cijothomas cijothomas mentioned this issue Jul 22, 2024
Open
@frittentheke frittentheke mentioned this issue Dec 12, 2024
Open
@cijothomas
Copy link
Member

Tracked via #1951 closing this.

@cijothomas cijothomas reopened this Jan 22, 2025
@cijothomas cijothomas removed this from the Metrics SDK Stable milestone Jan 22, 2025
@cijothomas
Copy link
Member

Reopening this issue, and using to track enabling this feature.

@cijothomas cijothomas mentioned this issue Feb 10, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cijothomas cijothomas

Labels
A-metrics Area: issues related to metrics
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Implement cardinality limits for metrics streams bnjjj/opentelemetry-rust
4 participants
@lalitb @cijothomas @lzchen @TommyCpp