-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
streamingest: add a metric for replication cutover progress #96537
streamingest: add a metric for replication cutover progress #96537
Conversation
3d3cd2c
to
1e6af1e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This feels reasonable to me.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewable status: complete! 1 of 0 LGTMs obtained (waiting on @adityamaru)
@lidorcarmel if you want to you can add this metric to the DBConsole page added in #96206. Can also be a followup! |
followup sg, thanks! |
We already have the progress info in the job as the percentage of ranges that were reverted. This commit adds the number of ranges that are left to be reverted as a metric. Epic: CRDB-18752 Fixes: cockroachdb#96536 Release note: None
1e6af1e
to
45faeeb
Compare
followup sg, thanks! TFTR all! |
Build succeeded: |
We already have the progress info in the job as the percentage of ranges that were reverted. This commit adds the number of ranges that are left to be reverted as a metric.
Epic: CRDB-18752
Fixes: #96536
Release note: None