-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Response Ops][Alerting] Update alerts-as-data ILM policy name to be consistent between framework & rule registry #150334
Conversation
f26216a
to
16020b2
Compare
@elasticmachine merge upstream |
@elasticmachine merge upstream |
Pinging @elastic/response-ops (Team:ResponseOps) |
@elasticmachine merge upstream |
@elasticmachine merge upstream |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested locally and works as expected, LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes LGTM! I tested locally in various scenarios and the policy was managed as expected.
@elasticmachine merge upstream |
💚 Build Succeeded
Metrics [docs]Public APIs missing comments
History
To update your PR or re-run it, just comment with: cc @ymao1 |
Resolves #150331
Summary
In a previous PR we started installing an ILM policy for framework alerts as data when the
xpack.alerting.enableFrameworkAlerts
config flag is set to true. In that PR we used a different name than what is used by the rule registry even though the policy bodies were the same.In this PR, we are consolidating the naming of the two ILM policies so that we are only ever installing 1 policy. The
xpack.alerting.enableFrameworkAlerts
config is used to determine which plugin is responsible for installing the policy. When set to true, the alerting plugin installs the policy. When set the false, the rule registry installs the policy. This is an incremental step toward the alerting framework absorbing all of the resource installation functionality of the rule registryTo Verify
A few things to verify:
xpack.alerting.enableFrameworkAlerts=true
xpack.alerting.enableFrameworkAlerts: true
in your Kibana config.alerts-ilm-policy
is installed.alerts-observability.metrics.alerts-default-index-template
that uses the.alerts-ilm-policy
policyxpack.alerting.enableFrameworkAlerts=false
xpack.alerting.enableFrameworkAlerts: false
in your Kibana config.alerts-ilm-policy
is installed.alerts-observability.metrics.alerts-default-index-template
that uses the.alerts-ilm-policy
policyxpack.alerting.enableFrameworkAlerts: false
in your Kibana config.alerts-ilm-policy
is installed.alerts-observability.metrics.alerts-default-index-template
that uses the.alerts-ilm-policy
policyxpack.alerting.enableFrameworkAlerts: true