elastic / kibana

Your window into the Elastic Stack
https://www.elastic.co/products/kibana
Other
19.51k stars 8.05k forks source link

[SLO] Create an embeddable alert summary #167575

Open mgiota opened 9 months ago

mgiota commented 9 months ago

Slack Thread

🍒 Summary

We should allow users to add an alerts summary to their dashboards with predefined filters. This would allow SREs to see a summary of the alerts for the SLOs they are viewing on the dashboard.

🎨 Design requirements

Acceptance criteria

Screenshot 2023-11-16 at 13 51 37
### Tasks
- [x] SLO configuration with option to select multiple SLOs
- [x] Fetch active alerts logic
- [x] SLO alert widget UI
- [x] Set initial dimensions to the panel
- [x] Display selected SLOs in the widget
- [x] Unit tests with different scenarios
- [ ] Alert Widget empty state
- [ ] Option to edit the configuration
- [ ] Link to Observability alerts page
- [ ] Should honor dashboard's time picker
- [ ] Should honor manual Refresh button
elasticmachine commented 9 months ago

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

mgiota commented 8 months ago

According to the [Slack thread](Slack thread) of this issue, untracked alerts shouldn't be included in the calculation of total alerts. Total alerts = Active + Recovered

Update: We will not include Total alerts in the widget, just Active Alerts.

elasticmachine commented 8 months ago

Pinging @elastic/obs-ux-management-team (Team:obs-ux-management)

mgiota commented 8 months ago

I summarize here the points that were discussed in the Slack Thread with @maciejforcone while refining this issue

mgiota commented 8 months ago

Update:

mgiota commented 8 months ago

@maciejforcone When user selects multiple SLOs, where one can have active alerts and the other one has no active alerts (could be either 0 alerts or no alert rules), how do we display it with current design? What color has the widget, what is the title and what is the badge color of the selected SLOs? Combining the information of multiple SLOs with current design is restrictive. What do you think? Maybe we shouldn't allow the selection of multiple SLOs? Any other suggestion?

mgiota commented 5 months ago

@maciejforcone Does it make sense to still work on this one? Since the Overview embeddable now shows the number of active alerts (for one SLO), I was thinking maybe this is redundant. I can continue working on this one, if you think it will be useful information for the user though.