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

[AO] Alerts page: refresh does not work properly on the alert summary widget #157245

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

[AO] Alerts page: refresh does not work properly on the alert summary widget #157245

maryam-saeidi opened this issue May 10, 2023 · 1 comment · Fixed by #160159
Assignees
Labels
bug Fixes for quality problems that affect the customer experience Team: Actionable Observability - DEPRECATED For Observability Alerting and SLOs use "Team:obs-ux-management", for AIops "Team:obs-knowledge" v8.9.0

Comments

@maryam-saeidi
Copy link
Member

📝 Summary

On the alerts page, by clicking on refresh, the alert summary widget data is not refreshed:
image

✅ Acceptance Criteria

  • Refresh the alert summary widget on the alerts page when refresh button is clicked
@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.9.0 labels May 10, 2023
@elasticmachine
Copy link
Contributor

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

@maryam-saeidi maryam-saeidi added the bug Fixes for quality problems that affect the customer experience label May 10, 2023
@maryam-saeidi maryam-saeidi self-assigned this Jun 21, 2023
maryam-saeidi added a commit that referenced this issue Jun 22, 2023
…160159)

Fixes #157245

## Summary

This PR updates the absolute time range when the refresh button is
clicked.

## 🧪 How to test
- On the Alerts page, click on refresh and check the time range of
`_alert_summary` API call, it should be updated with each refresh


![image](https://github.com/elastic/kibana/assets/12370520/0522d770-6cf5-4af7-8c42-bb508e614164)
kibanamachine pushed a commit to kibanamachine/kibana that referenced this issue Jun 22, 2023
…lastic#160159)

Fixes elastic#157245

## Summary

This PR updates the absolute time range when the refresh button is
clicked.

## 🧪 How to test
- On the Alerts page, click on refresh and check the time range of
`_alert_summary` API call, it should be updated with each refresh

![image](https://github.com/elastic/kibana/assets/12370520/0522d770-6cf5-4af7-8c42-bb508e614164)

(cherry picked from commit 7f906bd)
kibanamachine added a commit that referenced this issue Jun 22, 2023
…page (#160159) (#160233)

# Backport

This will backport the following commits from `main` to `8.8`:
- [[AO] Fix issue with alert summary widget refresh on the Alerts page
(#160159)](#160159)

<!--- Backport version: 8.9.7 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Maryam
Saeidi","email":"maryam.saeidi@elastic.co"},"sourceCommit":{"committedDate":"2023-06-22T08:36:41Z","message":"[AO]
Fix issue with alert summary widget refresh on the Alerts page
(#160159)\n\nFixes #157245\r\n\r\n## Summary\r\n\r\nThis PR updates the
absolute time range when the refresh button is\r\nclicked.\r\n\r\n## 🧪
How to test\r\n- On the Alerts page, click on refresh and check the time
range of\r\n`_alert_summary` API call, it should be updated with each
refresh\r\n\r\n\r\n![image](https://github.com/elastic/kibana/assets/12370520/0522d770-6cf5-4af7-8c42-bb508e614164)","sha":"7f906bd0c1bddba64e23fdcdfeb61304542fa9c8","branchLabelMapping":{"^v8.9.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:
Actionable
Observability","backport:prev-minor","v8.9.0"],"number":160159,"url":"https://github.com/elastic/kibana/pull/160159","mergeCommit":{"message":"[AO]
Fix issue with alert summary widget refresh on the Alerts page
(#160159)\n\nFixes #157245\r\n\r\n## Summary\r\n\r\nThis PR updates the
absolute time range when the refresh button is\r\nclicked.\r\n\r\n## 🧪
How to test\r\n- On the Alerts page, click on refresh and check the time
range of\r\n`_alert_summary` API call, it should be updated with each
refresh\r\n\r\n\r\n![image](https://github.com/elastic/kibana/assets/12370520/0522d770-6cf5-4af7-8c42-bb508e614164)","sha":"7f906bd0c1bddba64e23fdcdfeb61304542fa9c8"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v8.9.0","labelRegex":"^v8.9.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/160159","number":160159,"mergeCommit":{"message":"[AO]
Fix issue with alert summary widget refresh on the Alerts page
(#160159)\n\nFixes #157245\r\n\r\n## Summary\r\n\r\nThis PR updates the
absolute time range when the refresh button is\r\nclicked.\r\n\r\n## 🧪
How to test\r\n- On the Alerts page, click on refresh and check the time
range of\r\n`_alert_summary` API call, it should be updated with each
refresh\r\n\r\n\r\n![image](https://github.com/elastic/kibana/assets/12370520/0522d770-6cf5-4af7-8c42-bb508e614164)","sha":"7f906bd0c1bddba64e23fdcdfeb61304542fa9c8"}}]}]
BACKPORT-->

Co-authored-by: Maryam Saeidi <maryam.saeidi@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience Team: Actionable Observability - DEPRECATED For Observability Alerting and SLOs use "Team:obs-ux-management", for AIops "Team:obs-knowledge" v8.9.0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants