-
Notifications
You must be signed in to change notification settings - Fork 13
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 canary paused / failed metrics #56
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.
This pull request contains a valid label.
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 contains a valid label.
Nice work 👍 |
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 contains a valid label.
worth keeping in mind that the |
Codecov Report
@@ Coverage Diff @@
## master #56 +/- ##
==========================================
- Coverage 37.54% 36.65% -0.89%
==========================================
Files 29 29
Lines 1537 1574 +37
==========================================
Hits 577 577
- Misses 886 923 +37
Partials 74 74
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
What does this PR do?
Adds a new
eds_status_canary_paused
gauge metric to identify canary in paused state along with the reason.Adds a new
eds_status_canary_failed
gauge metric to identify canary in failed state along with the reason.Motivation
Canary metrics
Describe your test plan
Check the metrics are exported correctly for canaries in active and paused state.