-
Notifications
You must be signed in to change notification settings - Fork 478
DOC-15405 Update _data/v25.4/metrics/export files #21152
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.
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
Conversation
….com/cockroachlabs/managed-service/tree/88fab7c0da5da662b90759202e04f18732c006d9/pkg/otel/assets. In available-metrics-in-metrics-list.csv, added 5 metrics. In metric-names.md, updated the check to look at shared_metrics.yaml first.
✅ Deploy Preview for cockroachdb-api-docs canceled.
|
✅ Deploy Preview for cockroachdb-interactivetutorials-docs canceled.
|
Files changed:
|
✅ Netlify Preview
To edit notification comments on pull requests, go to your Netlify project configuration. |
mikeCRL
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. I noticed sql_mem_root_current was removed from shared_metrics.yaml but doesn't appear in crdb_metrics.yaml or tenant_metrics.yaml. Intentional? If so, should we have a release note? (I'm not seeing one.)
Fixes DOC-15405
Rendered preview