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] Alert preview produces confusing results #90848

Closed
Zacqary opened this issue Feb 9, 2021 · 3 comments
Closed

[Metrics UI] Alert preview produces confusing results #90848

Zacqary opened this issue Feb 9, 2021 · 3 comments
Labels
Feature:Metrics UI Metrics UI feature R&D Research and development ticket (not meant to produce code, but to make a decision) Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services

Comments

@Zacqary
Copy link
Contributor

Zacqary commented Feb 9, 2021

UPDATE: This is an R&D ticket to document why the alert preview was returning inaccurate and/or confusing results before we removed it in 7.15, in this ticket: #107946

AC:

  • Document the problems with the "Test Alert Condition" preview functionality and what is causing these problems
  • File a new ticket to fix the problems and/or create an RFC with @elastic/kibana-alerting-services

ORIGINAL TICKET DESCRIPTION BELOW:

From #90070 (review)

I have 10 hosts reporting an average around 12-20%, then I have one host that is pushing 100% (which averages out to about 50% (ish). When I set the threshold to 50% I would expect the preview to say something like

There were 60 instances that satisfied the conditions of this alert across 1 kubernetes.pod.uids in the last hour.

but instead it says

There were 60 instances that satisfied the conditions of this alert across 10 kubernetes.pod.uids in the last hour.

It feels like we are counting too many UUIDs for the match. When the alert runs, it only matches on one UUID.

Screen Shot 2021-02-09 at 2 43 32 PM

This isn't so much a bug as a UX issue. The alert preview is designed to tell the user that it's looking at 10 pod UIDs total, but it doesn't tell you how many of those alerted.

We should try to achieve something like:

There were 60 instances that satisfied the conditions of this alert on 1 out of 10 kubernetes.pod.uids in the last hour.

@Zacqary Zacqary added bug Fixes for quality problems that affect the customer experience 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 Feb 9, 2021
@elasticmachine
Copy link
Contributor

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

@jasonrhodes
Copy link
Member

In 7.15 we are going to remove this feature from the alert flyout until we understand it better. I'd love for @Zacqary and @simianhacker to sync up when you are both around and figure out what exactly is going wrong with it, so we have it documented how it would be fixed. We can then decide if it's a good idea to re-enable it here or to try to submit this as an RFC to the alerting framework.

@jasonrhodes jasonrhodes changed the title [Metrics UI] Alert preview language about number of matched nodes/groups is confusing [Metrics UI] Alert preview produces confusing results Aug 10, 2021
@jasonrhodes jasonrhodes added R&D Research and development ticket (not meant to produce code, but to make a decision) and removed bug Fixes for quality problems that affect the customer experience labels Aug 10, 2021
@smith
Copy link
Contributor

smith commented Jun 14, 2023

Not planning on implementing this.

@smith smith closed this as not planned Won't fix, can't repro, duplicate, stale Jun 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Metrics UI Metrics UI feature R&D Research and development ticket (not meant to produce code, but to make a decision) Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services
Projects
No open projects
Development

No branches or pull requests

4 participants