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

Zoomed in metric charts disappear after switching between metric dashboards #71201

Closed
aliher1911 opened this issue Oct 6, 2021 · 2 comments
Closed
Assignees
Labels
C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior.

Comments

@aliher1911
Copy link
Contributor

aliher1911 commented Oct 6, 2021

Describe the problem

If I zoom into a timerange on the chart, change to other metric dashboard and return back to original one, the charts are gone. Other dashboards show fine and I can switch between them.
To restore charts I can pick a new time range from time dropdown.

To Reproduce

What did you do? Describe in your own words.

If possible, provide steps to reproduce the behavior:

  1. take any cluster with metrics (like cockroach demo) that run for some time
  2. go to metrics
  3. switch to past 1 h time range
    step1
  4. use mouse drag to select a part of time range on the chart
    step2
  5. switch to hardware dashboard
    step3
  6. switch back to overview
    step4

Expected behavior
Charts to stay visible.

Environment:

  • CockroachDB version 21.2 master and prior (ff7f80f)
  • Server OS: MacOS
  • Client app Chrome, Safari

Additional context
Makes it hard to correlate historical charts between metric dashboards.
Outside of production clusters, we use dashboard to view exported metrics and that means we always look on detailed metrics in the past.

Epic CRDB-8038

@aliher1911 aliher1911 added the C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior. label Oct 6, 2021
@tbg
Copy link
Member

tbg commented Oct 6, 2021

Also had this issue yesterday while debugging roachtests and it caused a good amount of friction, especially since the date picker state is 100% volatile so any need to reload the page loses complete context.

@thtruo
Copy link
Contributor

thtruo commented Oct 25, 2021

cc @nathanstilwell @Santamaura @nkodali let's tackle this alongside the other Metrics page UI bugs that we've queued up

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior.
Projects
None yet
Development

No branches or pull requests

4 participants