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.

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

Filtering actions on alerts without any actions #58362

Closed
alexfrancoeur opened this issue Feb 24, 2020 · 17 comments
Closed

Filtering actions on alerts without any actions #58362

alexfrancoeur opened this issue Feb 24, 2020 · 17 comments
Labels
bug Fixes for quality problems that affect the customer experience Feature:Alerting Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) UX

Comments

@alexfrancoeur
Copy link

Feb-24-2020 10-54-48

Steps to reproduce:

  • Filter by an action on a list of actions that do not have any actions

I think we need another empty state that says "nothing matches filters" or something along those lines. I imagine we have a common empty state for a filtered list view, right?

cc: @mdefazio

@alexfrancoeur alexfrancoeur added bug Fixes for quality problems that affect the customer experience Feature:Alerting Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) labels Feb 24, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-alerting-services (Team:Alerting Services)

@mdefazio
Copy link
Contributor

Does it make sense to have the filter dropdown only list the currently used action types? So in the case above, there would only be Slack in the dropdown. We could probably go a step further and have that be disabled or preselected, or something, since it wouldn't actually change anything. This way we can avoid an empty state altogether.

@pmuellr
Copy link
Member

pmuellr commented Feb 25, 2020

This way we can avoid an empty state altogether.

I can imagine a situation where I might want to "confirm" that no alerts are using some particular action/actionType, which would be a case where I'd want to see an empty state (if empty state means the list is empty).

@pmuellr
Copy link
Member

pmuellr commented Feb 25, 2020

I think on our sync yesterday, it seemed like having an additional filter of "has no actions" along with the other filters would work. Presumably can be multi-selected with other filters. So you could filter on "email" or "has no actions" - not that it would make a lot of sense, but would avoid having to special case a search on "no actions".

@mdefazio
Copy link
Contributor

mdefazio commented Feb 25, 2020

I can imagine a situation where I might want to "confirm" that no alerts are using some particular action/actionType, which would be a case where I'd want to see an empty state (if empty state means the list is empty).

Maybe it makes sense to have a filter on the Connector tab that is 'has no alert' to help solve this use case.

@mdefazio
Copy link
Contributor

Would an alert have no actions because I deleted an action that it was using? Wouldn't this be returning an error of some sort then?

@pmuellr
Copy link
Member

pmuellr commented Feb 28, 2020

You could have a scenario where a customer has an alert with an action, and decides to replace the existing action with another. They would likely delete the existing one, and then add the new one. Imagine they got interrupted after deleting the existing one, before adding the new one (even though both operations could be done together, atomically, since it's just modifying the actions structure in the alert structure). The alert would then be in a state with no actions.

I imagine there may be some other scenarios where someone may want to delete an action in an alert if the alert was being annoying or something. Even though there are other ways they could "mute" the alert, deleting the action may be the first thing that comes to mind, and I think we should certainly allow it. I'd be pretty frustrated as a customer if I couldn't delete an action on an alert because of a constraint that we imposed that every alert must have at least one action.

And there are the current SIEM alerts, which (right now) have no actions associated with them. :-)

@mdefazio
Copy link
Contributor

mdefazio commented Mar 2, 2020

As mentioned in #58493, I've updated the filter dropdowns to help avoid the empty state, add added an extra filter option when no alert has an action assigned to it

image

I'm also attaching a mockup for an empty state since we need one regardless of the above.

Alerts--Main-Empty

@mikecote
Copy link
Contributor

As I'm working through license checks (#54946). I'm wondering what the action type filter would look like in the following two scenarios?

  • Action type is disabled by the Kibana configuration
  • Action type is disabled by the license

Both of those may have alerts or connectors assigned to them if ever they were created before the action type got disabled.

Currently we can easily tell in the connectors list how many connectors use an action type. We currently don't know in the alerts list how many because it is tricky to aggregate. The main reasons are due to feature controls / security not applying to aggregation queries (for now).

@mdefazio
Copy link
Contributor

So if I understand the last comment, we can show a filter menu for the connector table that is pared down based on the action types used, but we can't on the alerting tab. Is it a total count that is difficult, or if its used at all?

@mikecote
Copy link
Contributor

@mdefazio that correct and we can't on the alerting tab because we can't easily tell if an action type is used at all.

With the feature controls work we're planning to do, we may be able to revisit that answer later.

@mdefazio
Copy link
Contributor

So the above screens work for the connector tab? Where the thinking is that we show a type if there is a connector of that type, even if it's disabled (assuming its still visible in the table to the user). Otherwise the type would be disabled in the filter menu.

So then for 7.7 for the alerting table, we just show all action types in the filter menu? And we need the empty state for when there are no alerts with that type.

@mikecote
Copy link
Contributor

So the above screens work for the connector tab? Where the thinking is that we show a type if there is a connector of that type, even if it's disabled (assuming its still visible in the table to the user). Otherwise the type would be disabled in the filter menu.

That's correct

So then for 7.7 for the alerting table, we just show all action types in the filter menu?

That's correct

And we need the empty state for when there are no alerts with that type.

This piece we won't be able to do for now due to not knowing how many alerts use that type.

@mikecote mikecote added bug Fixes for quality problems that affect the customer experience and removed bug Fixes for quality problems that affect the customer experience labels Mar 20, 2020
@mikecote mikecote added the UX label Sep 9, 2020
@ymao1
Copy link
Contributor

ymao1 commented Dec 16, 2020

Current state of the alerts list:

  • all action types are shown in the filter menu, whether or not there's an associated alert
  • empty state when action type with no associated alerts selected

Dec-16-2020 11-08-08

It seems like the original issue has been addressed?

@mdefazio
Copy link
Contributor

Not sure if the discussion was resolved around a disabled action type? Do we show action types in the filter menu if the user does not have access/disabled?

Also, might not be for this issue, but a few thoughts if we are closing this.

  • Should we sort the actions alphabetically in the filter dropdown?
  • Should we go ahead and add search to this filter dropdown?
  • Can the label for this dropdown simply be 'Actions' to match the column name?

@ymao1
Copy link
Contributor

ymao1 commented Dec 16, 2020

Not sure if the discussion was resolved around a disabled action type? Do we show action types in the filter menu if the user does not have access/disabled?

It looks like they still show up in the filter menu if they are disabled. Based on this comment it seems like that was the expected behavior (at least for 7.7). Have there been changes since 7.7 that would allows us to remove them from the filter menu if they are disabled?

@ymao1
Copy link
Contributor

ymao1 commented Jun 8, 2021

Closing this as the original issue (no empty state when filtering rules) has been addressed. Created follow up issue for additional UX improvements to the action type filter: #101597

@ymao1 ymao1 closed this as completed Jun 8, 2021
@kobelb kobelb added the needs-team Issues missing a team label label Jan 31, 2022
@botelastic botelastic bot removed the needs-team Issues missing a team label label Jan 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience Feature:Alerting Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) UX
Projects
None yet
Development

No branches or pull requests

7 participants