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: catalog of timeseries charts #40218

Open
wants to merge 1 commit into
base: master
from

Conversation

@sploiselle
Copy link
Contributor

commented Aug 26, 2019

This was a request from Spencer long, long ago. This PR essentially just adds a front end experience to a server endpoint I created in #27505.

The idea here is that it's hard for users who aren't intimately familiar with a subsystem to know which metrics would be appropriate to look at for potential issues. By hand-curating an artisanal list of subsystems, metrics, and chart presets, we can introduce a little bit of transparency during the debugging process.

The UX here is that clicking any link brings you to the custom chart page with all of the requisite data pre-filled to give you a chart (or set of charts), which will ostensibly help you understand how that subsystem is performing.

Individual charts are all specified with a bullet (i.e. in a <ul>) while clicking a header will render all of the charts that are under its "overhang"--the dotted line above the header.

Here's a screenshot of the page for reference.

Screen Shot 2019-08-26 at 2 58 07 PM

The page that renders after a link click is, like I mentioned, just the custom charts page.

Release note (ui): Added a predefined catalog of timeseries charts to help troubleshooting at /#/debug/chart-catalog

ui: catalog of timeseries charts
Release note (ui): Added a predefined catalog of timeseries charts to help troubleshooting at <Admin UI URL>/#/debug/chart-catalog

@sploiselle sploiselle requested a review from cockroachdb/admin-ui-prs as a code owner Aug 26, 2019

@cockroach-teamcity

This comment has been minimized.

Copy link
Member

commented Aug 26, 2019

This change is Reviewable

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.