-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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: anomaly detection UI #5916
Conversation
We would appreciate it if you could provide us with more info about this issue/pr! |
Build Error! No Linked Issue found. Please link an issue or mention it in the body using #<issue_id> |
fc223fe
to
9e6a0af
Compare
9e6a0af
to
486798d
Compare
Build Error! No Linked Issue found. Please link an issue or mention it in the body using #<issue_id> |
Build Error! No Linked Issue found. Please link an issue or mention it in the body using #<issue_id> |
frontend/src/container/AnomalyAlertEvaluationView/AnomalyAlertEvaluationView.tsx
Show resolved
Hide resolved
frontend/src/container/AnomalyAlertEvaluationView/AnomalyAlertEvaluationView.tsx
Show resolved
Hide resolved
@vikrantgupta25 : Will take these up in the subsequent PR's. |
|
Will make the change in the next PR. Can you please provide more details on these points
|
@srikanthccv : Will the API return only the non anomaly based alerts if the FF is disabled ? How do we want to handle this ? |
Discussed disabling the anomaly detection by default for everyone and only enable for specific tenants (like private beta), starting with saasmonitor. |
As discussed, merging the base PR. will work on the above points in next PR / subsequent PR's |
Merged ✅ |
|
|
Feature: Anomal Detection
Users should be able to setup a anomaly based alerts for metrics.
Alert Selection Change
Switch between Threshold based alert and Anomaly based alert
Define Alert Section Change:
Evaluation view rendering
Anomaly Function for all queries