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

Alerting Summary widget doesn't show total number of alerts #170603

Closed
mgiota opened this issue Nov 6, 2023 · 7 comments
Closed

Alerting Summary widget doesn't show total number of alerts #170603

mgiota opened this issue Nov 6, 2023 · 7 comments
Assignees
Labels
Team: Actionable Observability - DEPRECATED For Observability Alerting and SLOs use "Team:obs-ux-management", for AIops "Team:obs-knowledge" v8.12.0

Comments

@mgiota
Copy link
Contributor

mgiota commented Nov 6, 2023

Slack thread

🍒 Summary

The total number of alerts shown in the alert summary widget in the Observability alerts page doesn't match the actual number of alerts that gets displayed in the alerts table below.

number_of_alerts

In the Video below you can see that total number of alerts is 9, but it displays 5 instead. The untracked alerts are not taken into consideration in the calculation of total number of alerts in the summary widget.

Screen.Recording.2023-11-06.at.01.12.12.mov

I did one more round of testing to verify what happens with the Recovered alerts and if they are taken into consideration or not in the calculation of total numbers. It seems that recovered alerts work as expected.

Screen.Recording.2023-11-06.at.01.14.34.mov

✅ Acceptance criteria

  • Total number of alerts in the alerts summary widget should be the total of active + recovered + untracked
  • Without knowing the implementation details, I assume this should be fixed in the triggersActionsUi plugin (need to verify)
@mgiota mgiota added the Team: Actionable Observability - DEPRECATED For Observability Alerting and SLOs use "Team:obs-ux-management", for AIops "Team:obs-knowledge" label Nov 6, 2023
@mgiota mgiota self-assigned this Nov 6, 2023
@elasticmachine
Copy link
Contributor

Pinging @elastic/actionable-observability (Team: Actionable Observability)

@maryam-saeidi
Copy link
Member

@mgiota Is this ticket only about adding the number of untracked alerts to the alert summary widget?

In the current implementation, we don't include untracked alerts in the widget, I wonder if it is better to keep it this way or include the untracked alerts there. @maciejforcone What do you think?

@mgiota
Copy link
Contributor Author

mgiota commented Nov 6, 2023

@maryam-saeidi For me it was confusing to see different number of total alerts in the table versus in the summary widget. I am not sure if untracked alerts was left out by design or if it is a bug.

adding the number of untracked alerts

So this ticket is about updating the number of total alerts when Show all is selected including untracked alerts as well, if that makes sense by design point of view. Mainly for consistency reasons with the total number of alerts in the table below. I don't mean adding an extra text, if that's what you mean.

4
Untracked

@maryam-saeidi
Copy link
Member

@mgiota Yes, exactly. My question for @maciejforcone is whether adding the number of untracked alerts to the total number for the chart and the number of alerts would be the best way to show this information on the widget.

@mgiota
Copy link
Contributor Author

mgiota commented Nov 6, 2023

@maciejforcone The same applies to what I am currently working on, the SLO Alerts embeddable (#167575). I want to know if I should include in the total number of alerts the untracked alerts as well. Because currently we don't and I want to be consistent with what we display on SLO alert embeddable and Alerts page.

@maryam-saeidi
Copy link
Member

maryam-saeidi commented Nov 6, 2023

Update from @maciejforcone:

  1. Should we show untracked information in the alert summary widget? Or keep it as it is?
    Answer: I think it makes sense, we can treat untracked as irrelevant alerts, so we don't need to count them into the widget.

  2. Should we handle mismatch between table and widget numbers?

    Answer: I think that untracked alerts shouldn't be shown on the list by default, user would need to select "untracked" on top to see them.

@mgiota
Copy link
Contributor Author

mgiota commented Nov 7, 2023

According to our discussion in the Slack thread, I am closing this issue in favor of this one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team: Actionable Observability - DEPRECATED For Observability Alerting and SLOs use "Team:obs-ux-management", for AIops "Team:obs-knowledge" v8.12.0
Projects
None yet
Development

No branches or pull requests

3 participants