-
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
Add filter to alerting flyout in logs UI #65011
Comments
Things to talk about: do we need this in logs if we can just set it as part of the condition? Can metrics fake this with a condition as well? |
I wanted to come back to this due to a recent conversation with a solutions architect. With regards to just adding a filter, we made the right decision to shelf this as conditions can mimic all of that functionality. It's not needed (for logs at least). However, an idea I'd suggested in a recent(ish) meeting was "reusable filters", filters that could be shared between Observability alert types. Again, we (correctly) shelved the idea due to there being no concrete user story behind this. After speaking with an SA though, it was clear there's a real use case here:
Do we think this is worth exploring? If we still think this has no legs that's also fine, just wanted to share the new information 😄 |
Interesting! So filters would be more than just free text but also have a crud aspect for saved filter items? |
Yeah, in my head I imagine this working a little like "Saved views" in Metrics. You could either enter things in the input to make a new filter, or apply a pre-saved one (even cooler if at the alerting top level you could set some sort of default). Those filters would most likely live within a saved object. I don't know the technicalities around sharing saved objects across plugins, so I'm sure this would need R&D if we wanted to pursue it. |
Kibana has the concept of saved queries by now (https://github.com/elastic/kibana/tree/16a51a18f82a8d289e56563d4ff452d11882a193/src/plugins/data/public/query/saved_query). Maybe we could integrate with that? |
related: #53123 |
@weltenwort Oh, that's nice. It looks like this would work for the use case, and if so +1 to using that service. |
@Kerry350 can you write up a new ticket describing what you'd like to see for this and how it could leverage saved queries? Then just add it to the Logs project board (no column) and it'll appear in the triage list for us to think through how to prioritize. Thanks!!! |
Describe the feature:
Current alerting mock up as described here allows users to define an alert but you can not specify a filter condition to restrict the alert condition evaluation. Cases when you'd like to evaluate alert condition when certain conditions are met - only specific index, specific hosts or service names etc.
Filter should look similar to alerting flyout on metrics UI
The text was updated successfully, but these errors were encountered: