-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
release-23.1: kvserver: Add a metric for in-progress snapshots #100421
release-23.1: kvserver: Add a metric for in-progress snapshots #100421
Conversation
Fixes: #98242 Knowing how many delegate snapshot requests are currently in-progress will be useful for detecting problems. This change adds a metric for this. It also updates the names of the previous stats to have the prefix `range.snapshots` vs `range.snapshot` to be consistent with other stats. Epic: none Release note (ops change): Adds a new stat range.snapshots.delegate.in-progress and renames two existing stats. They were never part of a release, so better to rename them before 23.1.0 is cut. range.snapshot.delegate.successes -> range.snapshots.delegate.successes range.snapshot.delegate.failures -> range.snapshots.delegate.failures
Thanks for opening a backport. Please check the backport criteria before merging:
If some of the basic criteria cannot be satisfied, ensure that the exceptional criteria are satisfied within.
Add a brief release justification to the body of your PR to justify this backport. Some other things to consider:
|
It looks like your PR touches production code but doesn't add or edit any test code. Did you consider adding tests to your PR? 🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is dev-inf. |
d8e415f
to
fff395f
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.
Reviewed 3 of 3 files at r1, all commit messages.
Reviewable status:complete! 1 of 0 LGTMs obtained (waiting on @andrewbaptist)
Backport 1/1 commits from #99843 on behalf of @andrewbaptist.
/cc @cockroachdb/release
Fixes: #98242
Knowing how many delegate snapshot requests are currently in-progress will be useful for detecting problems. This change adds a metric for this. It also updates the names of the previous stats to have the prefix
range.snapshots
vsrange.snapshot
to be consistent with other stats.Epic: none
Release note: None
Release justification: This was identified as a useful stat during the KV premortem