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

[vtadmin-web] Add chart for stream vreplication lag across all streams in a workflow #8331

Merged
merged 4 commits into from
Jun 15, 2021

Conversation

doeg
Copy link
Contributor

@doeg doeg commented Jun 14, 2021

Description

This PR adds a chart to the Workflow view that graphs the vreplication lag across all streams in the workflow. Many thanks to @rohit-nayak-ps for adding these stats for us in #8306!

Here's what it looks like before/after the cutover in a reshard workflow:

Screen Shot 2021-06-14 at 11 57 36 AM

And here's the MoveTables workflow from the local example, for fun:

Screen Shot 2021-06-14 at 12 04 49 PM

Related Issue(s)

Checklist

  • Tests were added or are not required
  • Documentation was added or is not required

Deployment Notes

This PR depends on #8306, which adds the VReplicationLag metric to the tablet /debug/vars endpoint. For Vitess deployments without that change, the chart will still render but no data will be shown. (More helpful error handling will be added to all charts as part of https://github.com/vitessio/vitess/projects/12#card-63086237.)

doeg added 4 commits June 14, 2021 09:40
Signed-off-by: Sara Bee <855595+doeg@users.noreply.github.com>
Signed-off-by: Sara Bee <855595+doeg@users.noreply.github.com>
Signed-off-by: Sara Bee <855595+doeg@users.noreply.github.com>
…to show

Signed-off-by: Sara Bee <855595+doeg@users.noreply.github.com>
@doeg doeg changed the title Sarabee vtadmin stream lag charts [vtadmin-web] Add chart for stream vreplication lag across all streams in a workflow Jun 14, 2021
@doeg doeg added this to In progress in VTAdmin via automation Jun 14, 2021
@doeg doeg added Component: VReplication Component: VTAdmin VTadmin interface Type: Enhancement Logical improvement (somewhere between a bug and feature) labels Jun 14, 2021
@doeg doeg marked this pull request as ready for review June 14, 2021 16:07
@ajm188 ajm188 merged commit 1290072 into vitessio:main Jun 15, 2021
VTAdmin automation moved this from In progress to Done Jun 15, 2021
@ajm188 ajm188 deleted the sarabee-vtadmin-stream-lag-charts branch June 15, 2021 20:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: VReplication Component: VTAdmin VTadmin interface Type: Enhancement Logical improvement (somewhere between a bug and feature)
Projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants