-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
Rule Type implementors can't prevent Creation of their alert types in Rules Management without also preventing Editing #89162
Comments
Pinging @elastic/kibana-alerting-services (Team:Alerting Services) |
Note that stack monitoring alerts may eventually move to pre-configured alerts which would solve the issue for them (prevent creation & have singleton alerts). |
Yup, now that @chrisronline ran through their rationale with me, it makes sense. |
similar scenario to when alerts can only be edited within the application (ex: detection alerts). |
Moving from |
Is this much work to support? I'd be happy to do it if it's a relatively small effort. |
@chrisronline I think it's relatively small, it would be great if you can help. We would just need to figure out what proposal works best for such an approach, but feel free to suggest something or let us know if you just need a direction. |
Hi @chrisronline Do you know of issues if user creates a new alert using one of the stack monitoring alert types? The reason I ask is if we can handle this issue in 2 steps:
|
This is the original issue: #75414 It looks like the original issue (the screenshot where you couldn't configure the CPU usage alert) is now fixed: We can change it so users can do the above screenshot if you want. I'm worried that creating more of these alerts will lead to confusion, because they can't really change the index pattern or add any filters. LMK |
@chrisronline , in 7.11 we have an indication in the create flow that this is stack monitoring related. Can you confirm? I would still like to pursue the change where we have granular control over create and edit and then we can disable the create action. |
I don't think so. The screenshot above is master and I don't think anything has changed between 7.11 and master. The alert is just called (for example) |
There are use cases where user smay want to alert when CPU is 75% for last 5 minutes and call that a warning and send an email. When its 85% for last 10 minutes they call that an error and want to send a pagerduty alert. There is extension to the use case where a user want to create a new alert and apply to only some of the objects as well. |
Exactly. There isn't a monitoring-related alert type in the list. It sounds like we want to change that and allow users to create and edit stack monitoring alerts from the Alerts Management UI. Is that true? |
I created 91145 to change the behaviour for stack monitoring alerts. |
we going close this issue please be our guess to re-open if need it |
@elastic/stack-monitoring have prevented the Creation and Editing of their Alert Types in Alerts Management, but they would actually like to allow Editing - it's only creation that they actually want to block.
It would be nice to allow editing of their alerts in Alerts Management, especially considering we do list these alerts and display their details.
The text was updated successfully, but these errors were encountered: