-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
feat(alerts): Add insights metrics option and alert type #76634
feat(alerts): Add insights metrics option and alert type #76634
Conversation
note: The alert rule builder doesn't fully work with Insight Metrics yet. This PR just includes the option and some types + constants. There will be a separate PR to update the builder to fully support Insights |
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.
Just need a :rubber_stamp:? 🤷 👍
Thanks for the review @nhsiehgit ! This PR relies on some changes from #76566 which is still open. I based this PR on that branch to minimize the code diff. Let me know if that changes anything for you! I plan to merge this in to |
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.
code looks good
…dd-insights-metrics-option-and-alert-type
Adds Insights Metric as an Alert type in the UI, protected behind the `insights-alerts` feature flag.
Adds Insights Metric as an Alert type in the UI, protected behind the
insights-alerts
feature flag.