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

[RAM] Add warning/error handling for alert throttles shorter than rule check interval #148569

Closed
Zacqary opened this issue Jan 9, 2023 · 1 comment · Fixed by #148919
Closed

[RAM] Add warning/error handling for alert throttles shorter than rule check interval #148569

Zacqary opened this issue Jan 9, 2023 · 1 comment · Fixed by #148919
Assignees
@Zacqary
Labels
Feature:Alerting/RulesManagement Issues related to the Rules Management UX Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams)

Comments

@Zacqary
Copy link
Contributor

Zacqary commented Jan 9, 2023

image

Display a warning when the alert throttle interval is shorter than the rule check interval. Also throw an error at the API level.
@Zacqary Zacqary added Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) Feature:Alerting/RulesManagement Issues related to the Rules Management UX labels Jan 9, 2023
@elasticmachine
Copy link
Contributor

Pinging @elastic/response-ops (Team:ResponseOps)

@Zacqary Zacqary mentioned this issue Jan 9, 2023
Merged
2 tasks
@Zacqary Zacqary self-assigned this Jan 13, 2023
@Zacqary Zacqary moved this from Awaiting Triage to In Progress in AppEx: ResponseOps - Rules & Alerts Management Jan 13, 2023
@Zacqary Zacqary mentioned this issue Jan 23, 2023
Merged
2 tasks
Zacqary added a commit that referenced this issue Jan 24, 2023
@Zacqary @kibanamachine
[RAM] Add error for action interval shorter than check interval (#148919
d60d537 
)

## Summary

Closes #148569

Adds form error and API error for action intervals shorter than rule
check intervals

<img width="585" alt="Screen Shot 2023-01-13 at 1 46 55 PM"
src="https://user-images.githubusercontent.com/1445834/212406181-059b53e4-a52d-44db-ba03-1a7285c676ce.png">
<img width="592" alt="Screen Shot 2023-01-13 at 1 46 42 PM"
src="https://user-images.githubusercontent.com/1445834/212406188-9b59fc05-45c0-4f93-b68c-c1540d46cd2b.png">


### Checklist

Delete any items that are not applicable to this PR.

- [x] Any text added follows [EUI's writing
guidelines](https://elastic.github.io/eui/#/guidelines/writing), uses
sentence case text and includes [i18n
support](https://github.com/elastic/kibana/blob/main/packages/kbn-i18n/README.md)
- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

Co-authored-by: Kibana Machine <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Zacqary Zacqary

Labels
Feature:Alerting/RulesManagement Issues related to the Rules Management UX Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams)
Projects
No open projects
AppEx: ResponseOps - Rules & Alerts Management
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[RAM] Add error for action interval shorter than check interval Zacqary/kibana
2 participants
@Zacqary @elasticmachine