Skip to content
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

[Metrics UI] Test Threshold Alert Condition #65830

Closed
sorantis opened this issue May 8, 2020 · 4 comments · Fixed by #67684
Closed

[Metrics UI] Test Threshold Alert Condition #65830

sorantis opened this issue May 8, 2020 · 4 comments · Fixed by #67684
Assignees
Labels
enhancement New value added to drive a business result Feature:Metrics UI Metrics UI feature Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services

Comments

@sorantis
Copy link

sorantis commented May 8, 2020

As a DevOps engineer I want to be able to test alert conditions to see how many times the alert would’ve been triggered in the past.

alert_test_trigger_mockup

@sorantis sorantis added enhancement New value added to drive a business result Feature:Metrics UI Metrics UI feature Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services labels May 8, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/logs-metrics-ui (Team:logs-metrics-ui)

@phillipb phillipb assigned phillipb and unassigned phillipb May 14, 2020
@sgrodzicki sgrodzicki added this to the Metrics UI 7.9 milestone May 25, 2020
@Zacqary Zacqary self-assigned this May 26, 2020
@Zacqary
Copy link
Contributor

Zacqary commented May 26, 2020

Not sure 12 results found in the past week is super clear. Maybe something like This alert would have fired 12 times in the past week across 1000 hosts?

@Zacqary
Copy link
Contributor

Zacqary commented May 29, 2020

@simianhacker I was able to implement this for Metric Threshold by having the Elasticsearch query retrieve an hour/week/day/month's worth of buckets, and then evaluate the alert on all of them. Inventory alerts, on the other hand, use the Snapshot API, and I'm not sure how to get that same kind of data from it in a similarly efficient manner. Any ideas?

@simianhacker
Copy link
Member

@Zacqary I think you need a way to force the bucket size and the lookback? I had a branch that added that functionality (for adding charts to the inventory alerts) but I never submitted a PR for it. I can create a "draft" PR and we could look to see if that would satisfy your needs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New value added to drive a business result Feature:Metrics UI Metrics UI feature Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants