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

Issues with alert statuses UI #87055

Closed
mikecote opened this issue Dec 30, 2020 · 2 comments · Fixed by #98135
Closed

Issues with alert statuses UI #87055

mikecote opened this issue Dec 30, 2020 · 2 comments · Fixed by #98135
Assignees
Labels
discuss Feature:Alerting Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams)

Comments

@mikecote
Copy link
Contributor

I've noticed two strange behaviours around alert statuses that I thought would be worth discussing.

  1. When the alert is disabled, it still displays the status from the last execution. I found this strange when it was "error" as the last execution status since it would never go away until the alert is re-enabled. There is also no clear indication in the alert list if the alert is disabled (to be solved in Visual indication for disabled or muted alerts #80310).
  2. When the alert is enabled, it takes some time for the status to be updated. In the meantime, it still contains the previous execution status (ex: error that was set before disabling the alert). Should this change to "pending" like when the alert is created?
@mikecote mikecote added discuss Feature:Alerting Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) labels Dec 30, 2020
@elasticmachine
Copy link
Contributor

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

@pmuellr
Copy link
Member

pmuellr commented Jan 13, 2021

  1. I think the UI should take into account the enabled state of the alert - we do want the execution status to be sticky, I think, for diagnostic purposes. So I think the fix would be to change the UI to display "disabled" for alerts that are disabled, rather than display the stored status.

  2. Makes sense to me to clear the execution status back to pending (and clear the other fields like error message, etc) when an alert transitions from disabled to enabled.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discuss Feature:Alerting Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams)
Projects
None yet
4 participants