Skip to content

Reduce default metric label cardinality#46

Merged
darthfork merged 4 commits intomainfrom
fix/issue-38-metric-cardinality
Apr 23, 2026
Merged

Reduce default metric label cardinality#46
darthfork merged 4 commits intomainfrom
fix/issue-38-metric-cardinality

Conversation

@darthfork
Copy link
Copy Markdown
Owner

Summary

  • remove high-cardinality labels from default workflow, job, push, and pull request metrics
  • switch hot-path metric updates to WithLabelValues to avoid per-update label map allocation
  • document the lower-cardinality metric contract and upgrade impact

Testing

  • go test ./...

Closes #38

@darthfork darthfork force-pushed the fix/issue-38-metric-cardinality branch from 1b4cf52 to 339e201 Compare April 23, 2026 04:09
@darthfork darthfork force-pushed the fix/issue-38-metric-cardinality branch from 339e201 to aa4d279 Compare April 23, 2026 04:11
@darthfork darthfork enabled auto-merge (squash) April 23, 2026 04:26
@darthfork darthfork merged commit 5b19941 into main Apr 23, 2026
1 check passed
@darthfork darthfork deleted the fix/issue-38-metric-cardinality branch April 23, 2026 04:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Audit and reduce high-cardinality labels in exported metrics

1 participant