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.

Sign up for GitHub

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

Jump to bottom

[AO] Add functional test for Alert Summary Widget on Alerts page #148645

Closed
maryam-saeidi opened this issue Jan 10, 2023 · 1 comment · Fixed by #154219
Closed

[AO] Add functional test for Alert Summary Widget on Alerts page #148645

maryam-saeidi opened this issue Jan 10, 2023 · 1 comment · Fixed by #154219
Assignees
@maryam-saeidi
Labels
Team: Actionable Observability - DEPRECATED For Observability Alerting and SLOs use "Team:obs-ux-management", for AIops "Team:obs-knowledge" v8.7.0 v8.8.0

Comments

@maryam-saeidi
Copy link
Member

📝 Summary

We need to add a functional test to make sure the charts are shown on the page correctly.

✅ Acceptance Criteria

  • Add a functional test for checking Alert Summary Widget information on the Alerts page
@maryam-saeidi maryam-saeidi added Team: Actionable Observability - DEPRECATED For Observability Alerting and SLOs use "Team:obs-ux-management", for AIops "Team:obs-knowledge" v8.7.0 labels Jan 10, 2023
@elasticmachine
Copy link
Contributor

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

@maryam-saeidi maryam-saeidi mentioned this issue Jan 10, 2023
Merged
maryam-saeidi added a commit that referenced this issue Jan 18, 2023
@maryam-saeidi @kibanamachine
[AO] Add AlertSummaryWidget full-size on the Alerts page (#148539)
a5e14cd 
Resolves #148279

## 📝 Summary

Add AlertSummaryWidget full-size on the Alerts page.


![image](https://user-images.githubusercontent.com/12370520/211568369-7d0847ba-e90e-4229-a55a-b68bf55209f8.png)

**Note**
The functional test will be added in another ticket
(#148645)

## 🧪 How to test
- Create a rule that generates an alert and go to the observability >
alerts page, you should see the number of alerts correctly in the Alert
Summary Widget
- Change the time range to see different formats in the tooltip and the
number of buckets will also change accordingly

## 🐛 Known issue
When we don't have any rule, the `_alert_summary` API fails and we show
a failed message, I will discuss it with @XavierM, and based on the
decision, either I'll update this PR or create a separate PR.
Update: I created a ticket for it
(#148653)

Co-authored-by: kibanamachine <[email protected]>
wayneseymour pushed a commit to wayneseymour/kibana that referenced this issue Jan 19, 2023
@maryam-saeidi @kibanamachine @wayneseymour
[AO] Add AlertSummaryWidget full-size on the Alerts page (elastic#148539
e3377aa 
)

Resolves elastic#148279

## 📝 Summary

Add AlertSummaryWidget full-size on the Alerts page.


![image](https://user-images.githubusercontent.com/12370520/211568369-7d0847ba-e90e-4229-a55a-b68bf55209f8.png)

**Note**
The functional test will be added in another ticket
(elastic#148645)

## 🧪 How to test
- Create a rule that generates an alert and go to the observability >
alerts page, you should see the number of alerts correctly in the Alert
Summary Widget
- Change the time range to see different formats in the tooltip and the
number of buckets will also change accordingly

## 🐛 Known issue
When we don't have any rule, the `_alert_summary` API fails and we show
a failed message, I will discuss it with @XavierM, and based on the
decision, either I'll update this PR or create a separate PR.
Update: I created a ticket for it
(elastic#148653)

Co-authored-by: kibanamachine <[email protected]>
@fkanout fkanout added the v8.8.0 label Mar 1, 2023
@maryam-saeidi maryam-saeidi self-assigned this Mar 21, 2023
@maryam-saeidi maryam-saeidi mentioned this issue Apr 4, 2023
Merged
maryam-saeidi added a commit that referenced this issue Apr 4, 2023
@maryam-saeidi
[AO] Add functional test for alert summary widget on alerts page. (#1…
ce277ce 
…54219)

Resolves #148645

## Summary

This PR adds a functional test for the alert summary widget on the
alerts page.

Flaky test runner:
https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/2069
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@maryam-saeidi maryam-saeidi

Labels
Team: Actionable Observability - DEPRECATED For Observability Alerting and SLOs use "Team:obs-ux-management", for AIops "Team:obs-knowledge" v8.7.0 v8.8.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[AO] Add functional test for alert summary widget on alerts page. maryam-saeidi/kibana
3 participants
@fkanout @maryam-saeidi @elasticmachine