-
Notifications
You must be signed in to change notification settings - Fork 33
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
Add "cause based" alerts for authorino #672
Comments
@guicassolato Tagging you on this as an fyi, in case you weren't aware of it. If you need any help on constructing the prometheus queries for what you think are good alerts for limitador, or help on how to construct the alert rules, please do reach out to myself or @R-Lawton . There are examples of alert rules in here. |
I want to close this issue. The what and done do not talk about the same thing. It is not clear what is need here. |
Updated the issue was copy and paste error. The idea is to create cause based alerts for authorino. The table top is in reference to an exercise we did for limitador where a fake issue was created to see whats missing in terms of alerts that could cause inspiration for the cause based alerts. I think its probs a good idea to do a table top scenario with authorino like whats was done in limitador. Ill follow up on that and get back to you |
@Boomatang this doc puts alerts into 2 category types, one of them being 'cause based' alerts. From an authorino point of view, perhaps there are some scenarios that fall into the first category that would be helpful to have as alerts? |
@david-martin That makes sense. I am questioning is this again the right place for this issue. By what I am hearing, we wont be creating alerts, only creating examples that the user can follow. As this is targeted at authorino is the correct place for this work in the the authorino-operator? I am not seeing the work that the kuadrant-operator needs to do. |
That sounds fine to me. |
The reason I have it in the kuadrant operator is to be in line with the examples folder that we have that has the current examples. I would be in favour of keeping the alert examples in one location so that if you are a user of kuadrant you can see what types of alerts can be done |
WHAT
Create cause-based alerts for the authorino
The tabletop scenarios when applicable can be used as inspiration for the alerts
DONE
The text was updated successfully, but these errors were encountered: