-
Notifications
You must be signed in to change notification settings - Fork 101
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
Add snapstore bucket-related metrics #211
Conversation
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.
First thing, isn't it possible to build query around existing exposed metrics to get both of these counts? I think prometheus supports enough expressions and function to build query. Please give it a try. If it works i don't think we need this PR.
Regarding current PR i have suggested correction please address them. And also please update the metrics doc
Signed-off-by: Shreyas Rao <[email protected]>
dd393bc
to
79983c0
Compare
@swapnilgm Thanks for the review. I've addressed the comments. PTAL. |
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.
LGTM
Signed-off-by: Shreyas Rao [email protected]
What this PR does / why we need it:
This PR adds two new metrics
etcdbr_snapstore_latest_deltas_total
andetcdbr_snapstore_latest_deltas_revisions_total
, which provide information about the delta snapshots since the latest full snapshot in the snapstore (object store bucket). We assume that etcd-backup-restore process is the only handler of snapshots in the respective snapstore bucket, in order to maintain integrity of these metrics.Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note: