-
Notifications
You must be signed in to change notification settings - Fork 9.8k
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
mvcc: add store revision metrics #11126
Conversation
Add experimental metrics etcd_debugging_mvcc_current_revision and etcd_debugging_mvcc_compact_revision.
Have we seen this functional test error before? {"level":"warn","ts":"2019-09-05T22:55:57.643Z","caller":"etcdmain/etcd.go:176","msg":"failed to start etcd","error":"cannot fetch cluster info from peer urls: could not retrieve cluster information from the given URLs"} |
cc @wenjiaswe |
@jingyih No, all functional test passes when running locally. |
lgtm, thanks! |
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 too. I believe this should be added in the changelog though. Thanks @jingyih !!
Thanks for reminding! |
By the way, I think we could cherry pick metrics back to previous versions right? Could we cherry pick it to 3.2, 3.3. and 3.4? |
Sounds good. |
CHANGELOG-3.5: update from #11126
…6-origin-release-3.4 Automated cherry pick of #11126 on release-3.4
…6-origin-release-3.3 Automated cherry pick of #11126 on release-3.3
…6-origin-release-3.2 Automated cherry pick of #11126 on release-3.2
Add experimental metrics
etcd_debugging_mvcc_current_revision
andetcd_debugging_mvcc_compact_revision
.