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.

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

Clean up control plane metrics names #3684

Closed
zhaohuabing opened this issue Jun 27, 2024 · 2 comments · Fixed by #3727
Closed

Clean up control plane metrics names #3684

zhaohuabing opened this issue Jun 27, 2024 · 2 comments · Fixed by #3727
Assignees
Labels
area/observability Observability related issues kind/refactor
Milestone

Comments

@zhaohuabing
Copy link
Member

Original comment on #3564 (comment)

There are two different naming styles in EG:

xds_snapshot_creation_total
xds_snapshot_creation_failed
xds_snapshot_creation_success


status_update_total
status_update_failed_total
status_update_success_total
status_update_conflict_total


Prefer the first one. We also need to align "failed, failure, error" to a single term.
@zhaohuabing zhaohuabing added area/observability Observability related issues and removed triage labels Jun 27, 2024
@shawnh2 shawnh2 self-assigned this Jun 29, 2024
@shawnh2
Copy link
Contributor

shawnh2 commented Jun 29, 2024

Like https://prometheus.io/docs/practices/naming/#labels suggest, we can consider various status like failed, succeed, etc as a label value, this will reduce redundancy.

@arkodg arkodg added this to the v1.1.0-rc1 milestone Jul 1, 2024
@arkodg
Copy link
Contributor

arkodg commented Jul 1, 2024

thanks for looking into this one @shawnh2, can you revisit the naming for all our control plane metrics and update it based on best practices
cc @Xunzhuo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/observability Observability related issues kind/refactor
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants