search

elastic / kibana

Your window into the Elastic Stack
https://www.elastic.co/products/kibana
Other
19.68k stars 8.23k forks source link

[Alerting] Add proper check to see if TLS is required: replace `/_xpack/usage` logic with `areAPIKeysEnabled` check. #97487

Open YulNaumenko opened 3 years ago

YulNaumenko commented 3 years ago

This issue is a follow up from the original Alerting changes issue. The current implementation use /_xpack/usage to determine whether to bootstrap the monitoring alerts in x-pack/plugins/monitoring/server/lib/elasticsearch/verify_alerting_security.ts. But we replaced this logic for alerting framework to use areAPIKeysEnabled check exposed by the security API. Stack monitoring alerts should follow this logic as well. Otherwise we may run into the issue where alerting is enabled but none of the stack monitoring alerts are created.

elasticmachine commented 3 years ago

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