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

ui: add lease preference metrics to replication dashboard #116709

Merged
merged 1 commit into from Dec 20, 2023

Conversation

kvoli
Copy link
Collaborator

@kvoli kvoli commented Dec 18, 2023

Lease preference metrics were added in #107505, however were only
visible in DB Console via custom metrics. This commit adds the
leases.preferences.violating and leases.preferences.less-preferred
metrics to the replication dashboard under a new lease preferences
graph.

Resolves: #116678

Release note (ui change): Introduce a new lease preferences graph on the
replication dashboard. The lease preferences graph will display when the
current leaseholder is not the first lease preference and where the
current leaseholder satisfies no applied lease preference.

@kvoli kvoli self-assigned this Dec 18, 2023
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@kvoli kvoli force-pushed the 231218.ui-lease-pref-repl-dashboard branch 2 times, most recently from 7b8a272 to 5f38adf Compare December 19, 2023 15:37
@kvoli
Copy link
Collaborator Author

kvoli commented Dec 19, 2023

image

Lease preference metrics were added in cockroachdb#107505, however were only
visible in DB Console via custom metrics. This commit adds the
`leases.preferences.violating` and `leases.preferences.less-preferred`
metrics to the replication dashboard under a new lease preferences
graph.

Resolves: cockroachdb#116678

Release note (ui change): Introduce a new lease preferences graph on the
replication dashboard. The lease preferences graph will display when the
current leaseholder is not the first lease preference and where the
current leaseholder satisfies no applied lease preference.
@kvoli kvoli marked this pull request as ready for review December 19, 2023 17:54
@kvoli kvoli requested a review from a team as a code owner December 19, 2023 17:54
@kvoli kvoli force-pushed the 231218.ui-lease-pref-repl-dashboard branch from 5f38adf to 19fa4ba Compare December 19, 2023 20:07
Copy link
Collaborator

@andrewbaptist andrewbaptist left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

@kvoli
Copy link
Collaborator Author

kvoli commented Dec 20, 2023

TYFTR

bors r=andrewbaptist

@craig
Copy link
Contributor

craig bot commented Dec 20, 2023

Build succeeded:

@craig craig bot merged commit d590dd0 into cockroachdb:master Dec 20, 2023
9 checks passed
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.

ui: add lease preference metrics to db console replication dashboard
3 participants