Skip to content

ui: add lease preference metrics to replication dashboard#116709

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
kvoli:231218.ui-lease-pref-repl-dashboard
Dec 20, 2023
Merged

ui: add lease preference metrics to replication dashboard#116709
craig[bot] merged 1 commit intocockroachdb:masterfrom
kvoli:231218.ui-lease-pref-repl-dashboard

Conversation

@kvoli
Copy link
Copy Markdown
Contributor

@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
Copy Markdown
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
Copy Markdown
Contributor 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 review from a team, andrewbaptist and koorosh 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
Copy Markdown

@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
Copy Markdown
Contributor Author

kvoli commented Dec 20, 2023

TYFTR

bors r=andrewbaptist

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Dec 20, 2023

Build succeeded:

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