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: dragging-to-zoom on graph results in incorrect start time within the DB Console metrics page #84001

Closed
thtruo opened this issue Jul 7, 2022 · 2 comments · Fixed by #85809
Assignees
Labels
A-kv-observability A-observability-inf C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior.

Comments

@thtruo
Copy link
Contributor

thtruo commented Jul 7, 2022

Describe the problem

When dragging to zoom on a graph within the DB Console, the resulting graph displays the incorrect start time. This makes it challenging to investigate

To Reproduce

  1. Go to any metrics dashboard within the DB Console, e.g. Overview dashboard
  2. Click and drag to zoom in on a graph
  3. Notice that after dragging, the zoomed in graph incorrectly presents the starting time - it does not correspond to the actual start time you picked (the end time is correct)

Expected behavior
After dragging to zoom, I expect the new start and end time to reflect the time boundaries that I clicked/dragged.

Environment:

  • CockroachDB version 22.1

Jira issue: CRDB-17400

@thtruo thtruo added C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior. T-observability-inf labels Jul 7, 2022
@nvanbenschoten
Copy link
Member

I noticed this today as well. On v22.1.2, the end time of a drag-to-zoom interval is correct, but the start time is adjusted so that the interval is less than or equal to 1 hour. As a result, drag-to-zoom works correctly for intervals that are less than 1 hour, but is broken for intervals that are greater than 1 hour.

@maryliag
Copy link
Contributor

maryliag commented Aug 8, 2022

Seems duplicate of #71205

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

Successfully merging a pull request may close this issue.

4 participants