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

kvprober: add metrics to the registry & catalog #61524

Merged

Conversation

joshimhoff
Copy link
Collaborator

@joshimhoff joshimhoff commented Mar 5, 2021

#61074

kvprober: add metrics to the registry & catalog

This commit adds the kvprober metrics to the registry so they are
exported at the prometheus endpoint and tracked in CRDB's time-series
DB. This commit also adds the kvprober metrics to the catalog since
that is required by a unit test.

Release justification: Auxiliary system that is off by default.
Release note: None.

@joshimhoff joshimhoff requested a review from a team as a code owner March 5, 2021 01:51
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@joshimhoff joshimhoff requested review from tbg and knz March 5, 2021 01:52
This commit adds the kvprober metrics to the registry so they are
exported at the prometheus endpoint and tracked in CRDB's time-series
DB. This commit also adds the kvprober metrics to the catalog since
that is required by a unit test.

Release justification: Auxiliary system that is off by default.
Release note: None.
@joshimhoff joshimhoff force-pushed the export_the_metrics_from_kvprober branch from 85324ec to 0dd1e55 Compare March 5, 2021 03:25
@joshimhoff joshimhoff changed the title kvprober: add metrics to the registry kvprober: add metrics to the registry & catalog Mar 5, 2021
@joshimhoff
Copy link
Collaborator Author

bors r+

@craig
Copy link
Contributor

craig bot commented Mar 5, 2021

🔒 Permission denied

Existing reviewers: click here to make joshimhoff a reviewer

@tbg
Copy link
Member

tbg commented Mar 5, 2021

bors r+

@craig
Copy link
Contributor

craig bot commented Mar 5, 2021

Build succeeded:

@craig craig bot merged commit 05a7bec into cockroachdb:master Mar 5, 2021
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.

3 participants