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

[BUG] Bulk acknowledge alerts only works for first 10 alerts in a trigger #126

Closed

annie3431 opened this issue Oct 4, 2021 · 1 comment

Labels

bug

Contributor

annie3431 commented Oct 4, 2021

Describe the bug
On alerts by triggers page, when acknowledging alerts from 1 trigger it only applies to the first 10 alerts.

To Reproduce
Steps to reproduce the behavior:

Create a bucket-level monitor with 10+ alerts in 1 trigger. (Try using sample ecommerce data with count of documents below 10000 over the past 3 days)
Click on Alerts tab
Select the trigger with more than 10+ alerts.
Click the "Acknowledge" button.
Should see that "Active" column only decreased by 10, and "Acknowledged" column only adds 10.

Expected behavior
All alerts within the trigger should be acknowledge. If error occurs, should show an error notification.

Screenshots
N/A

Host/Environment (please complete the following information):

OS: [e.g. iOS]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

annie3431 added bug Beta untriaged labels

This was referenced Oct 13, 2021

Fixed a bug that was preventing the AcknowledgeAlerts API from acknowledging more than 10 alerts at once. opensearch-project/alerting#205

Merged

Fixed a bug that was preventing the AcknowledgeAlerts API from acknowledging more than 10 alerts at once. opensearch-project/alerting#208

Merged

Fixed a bug that was preventing the AcknowledgeAlerts API from acknowledging more than 10 alerts at once. opensearch-project/alerting#209

Closed

Fixed a bug that was preventing the AcknowledgeAlerts API from acknowledging more than 10 alerts at once opensearch-project/alerting#210

Merged

Collaborator

AWSHurneyt commented Nov 5, 2021

Closing this issue as it was resolved with this PR.

AWSHurneyt closed this as completed

AWSHurneyt removed the untriaged label

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment