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

[Alerting UI] Alert flyout doesn't have any information if alert type is not available for the current user role, just Save button is disabled. #75823

Closed
YulNaumenko opened this issue Aug 24, 2020 · 4 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)

Comments

@YulNaumenko
Copy link
Contributor

Currently user doesn't have any UI information that alert type is not allowed from Management UI for the user with disabled

Steps to reproduce:

  1. Login as user with no Stack Monitoring access.
  2. Navigate to Alerting management ui and open create alert flyout.
  3. Fill top section with name and schedule.
  4. Select CPU usage alert type (no fields to populate is required).
  5. Add ServerLog action.

Observer that Save button is disabled, but no warnings or errors is displayed:
image

If user have Stack Monitoring access CPU Usage alert type fields is available and form is valid:
image

We should provide some user info about not available alert types. At least show why Save button is disabled.

@YulNaumenko YulNaumenko added Feature:Alerting Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) labels Aug 24, 2020
@elasticmachine
Copy link
Contributor

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

@mikecote mikecote added the bug Fixes for quality problems that affect the customer experience label Aug 25, 2020
@pmuellr
Copy link
Member

pmuellr commented Aug 25, 2020

Ideally we wouldn't let customers get this far into the UI, for alert types that aren't enabled via RBAC.

Seems like we should present these as "disabled" in the alert type picker - but then we may have multiple reasons to disable - eg, gold+ actions when running with basic vs alert type not enabled due to RBAC. We'll need to provide some kind of visual cue for the "reason" - even if it's just hover, but that feels not great. Perhaps some "badge" we can add to the icons indicating why it's disabled?

@mikecote
Copy link
Contributor

@YulNaumenko mentioned this relates to #75414.

@YulNaumenko
Copy link
Contributor Author

Closed this in #77097

@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)
Projects
None yet
Development

No branches or pull requests

5 participants