-
Notifications
You must be signed in to change notification settings - Fork 25
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 Prometheus metrics verification to reconciler test #195
Conversation
c484f9f
to
e8aee47
Compare
/merge |
🚂 MergeQueue: pull request added to the queue The median merge time in Use |
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 pull request does not contain a valid label. Please add one of the following labels: bug, enhancement, refactoring, documentation, tooling, dependencies
e8aee47
to
0229b72
Compare
/merge |
🚂 MergeQueue: pull request added to the queue The median merge time in Use |
What does this PR do?
PR adds verification for metrics in reconciler test.
wantPromMetrics
are just baselines for current tests and isn't verified for correctness (first three are interesting since they are all 0).Motivation
Reviewing #187.
Additional Notes
Anything else we should know when reviewing?
Describe your test plan
Write there any instructions and details you may have to test your PR.