-
Notifications
You must be signed in to change notification settings - Fork 740
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 New Relic as a metrics provider #691
Conversation
6cc5d56
to
04dd365
Compare
e3598c3
to
ab28006
Compare
ab28006
to
2c249e2
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.
@fpetkovski can you please add a "New Relic" section to the docs https://github.com/weaveworks/flagger/blob/master/docs/gitbook/usage/metrics.md
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.
Accept suggested changes
Co-authored-by: Stefan Prodan <stefan.prodan@gmail.com>
Thanks for the pointer, will add the section with the proposed queries and get back to you |
f130523
to
c81e19c
Compare
2da2268
to
563b1cd
Compare
Co-authored-by: Stefan Prodan <stefan.prodan@gmail.com>
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
Thanks @fpetkovski 🥇
The pull requests adds New Relic as an available metrics provider. The implementation follows the one from the datadog provider.