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: metrics refresh intermittently causing visual flash on large clusters #65030

Closed
dhartunian opened this issue May 11, 2021 · 1 comment · Fixed by #88707
Closed

ui: metrics refresh intermittently causing visual flash on large clusters #65030

dhartunian opened this issue May 11, 2021 · 1 comment · Fixed by #88707
Assignees
Labels
C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior. P-2 Issues/test failures with a fix SLA of 3 months

Comments

@dhartunian
Copy link
Collaborator

dhartunian commented May 11, 2021

Describe the problem

On large clusters loading a dashboard and scrolling around will display pauses and reloads intermittently. Even when looking at a "2 week" timespan which has a window validity of 10 minutes

https://github.com/cockroachdb/cockroach/blob/master/pkg/ui/src/redux/timewindow.ts#L104-L108

we see refreshes every 10/15 seconds.

We do see the loading spinner show up while this happening.

To Reproduce

Load metrics dashboards on a large cluster > 60 nodes and scroll around and wait for a bit.

Expected behavior

Expect to see no visual flashes (ideally even to load new data, should happen smoothly)

Environment:

  • CockroachDB version: 20.2.9-alpha
  • Client app: DB Console

Additional context
This sounds like an issue that might be caused with all the React layers of indirection used when rendering graphs. For instance, we still have code in the NodeGraphs component that calls React.cloneElement

https://github.com/cockroachdb/cockroach/blob/master/pkg/ui/src/views/cluster/containers/nodeGraphs/index.tsx#L244

It's possible that this triggers a re-mounting of the component instead of an update.

Or this might be a simple issue of showing a loading spinner when in fact we shouldn't...we currently show the loading spinner when there's no data in the component

https://github.com/cockroachdb/cockroach/blob/master/pkg/ui/src/views/cluster/components/linegraph/index.tsx#L518

so it might help to track down why this could happen when we load new data. Maybe it ends up being intermittently set to empty while we're updating.

Epic CC-5280

Jira issue: CRDB-7398

@dhartunian dhartunian added C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior. T-cluster-ui labels May 11, 2021
@dhartunian
Copy link
Collaborator Author

Additionally, we should deal with the DOM sizing of the graph nodes thrashing as graphs load.

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. P-2 Issues/test failures with a fix SLA of 3 months
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants