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

[Monitoring] Migrate cluster alerts from watcher to Kibana alerting #81020

Closed
3 tasks done
chrisronline opened this issue Oct 19, 2020 · 7 comments · Fixed by #87377
Closed
3 tasks done

[Monitoring] Migrate cluster alerts from watcher to Kibana alerting #81020

chrisronline opened this issue Oct 19, 2020 · 7 comments · Fixed by #87377
Assignees
Labels
Team:Monitoring Stack Monitoring team

Comments

@chrisronline
Copy link
Contributor

chrisronline commented Oct 19, 2020

Depends on elastic/elasticsearch#50032

Action items

Background

We need to leverage the new ES api to remove all existing cluster alert watches and let our new Kibana alerts work by themselves.

Right now, we fetch legacy alert data from the result of the cluster alert watches index (which is .monitoring-alerts-*). Once the watches are gone, these alerts need to fetch the data manually, but I have done this work already:

@chrisronline chrisronline added the Team:Monitoring Stack Monitoring team label Oct 19, 2020
@chrisronline chrisronline self-assigned this Oct 19, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/stack-monitoring (Team:Monitoring)

@sgrodzicki sgrodzicki added this to the Stack Monitoring UI 7.11 milestone Oct 19, 2020
@jakelandis
Copy link
Contributor

elastic/elasticsearch#50032 is almost complete..but not quite ready yet. However, the API should be pretty well sorted if you want to start to code against that.

The request will be POST /_monitoring/migrate/alerts and the response will look like this:

{
  "exporters":[
    {
      "name":"thename",
      "type":"http|local",
      "migration_complete":true,
      "reason":"optional - exception"
    },
    {
      "name":"thename2",
      "type":"http|local",
      "migration_complete":false,
      "reason":"optional - exception"
    }
  ]
}

@chrisronline
Copy link
Contributor Author

@ravikesarwani

Should we wait on #85047 and #87377 until the Elasticsearch team implements changes to detect if users have actions connected to watcher (aka: elastic/elasticsearch#50032 (comment))? If we don't wait, it means we will either have to always notify users they may need to re-setup an action connector, or never show it and rely on docs to communicate this.

@ravikesarwani
Copy link
Contributor

Currently we deploy new alerts when user visits the SM UI. Can we add a generic message whenever we deploy new alerts? Something on the lines of "New alerts have been created. Review the alert definition using Setup mode and configure additional action connectors to get notified via your favorite method".
The message comes only when new alerts have been created.

This will help cover many different use cases.

@chrisronline
Copy link
Contributor Author

@ravikesarwani Sure, sounds good

@chrisronline
Copy link
Contributor Author

@ravikesarwani Currently, these legacy cluster alerts are limited to gold+ customers, primarily because they relied on watcher which is a gold+ feature. With this change, I'm assuming we'll bring these back down to basic (along with the rest of monitoring) and just wanted to double check that with you.

@ravikesarwani
Copy link
Contributor

Yes, in subscription page we will remove the following row in 7.12 (Automatic stack issue alerts). We added "Kibana alerting and actions" and has check mark in Basic.

We should account for doc changes related to this change as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team:Monitoring Stack Monitoring team
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants