-
Notifications
You must be signed in to change notification settings - Fork 24.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
Introduce the index.lifecycle.prefer_ilm setting #95423
Conversation
This introduces an index setting that'll allow to switch the preference for the system that manages a data stream backing index. This setting defaults to `true` (preferring ILM when both ILM and DLM are applicable for an index). Note that the prefer_ilm setting will only express a preference, it'll not enforce the presence of a particular management service e.g. if an index has `prefer_ilm: false` and an ILM policy configured, but no DLM configuration, the index will be managed by ILM (despite the preference for DLM)
Hi @andreidan, I've created a changelog YAML for you. |
Pinging @elastic/es-data-management (Team:Data Management) |
...plugin/ccr/src/main/java/org/elasticsearch/xpack/ccr/action/TransportResumeFollowAction.java
Outdated
Show resolved
Hide resolved
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.
LGTM, thanks for all the tests for this Andrei, they're great.
...plugin/ccr/src/main/java/org/elasticsearch/xpack/ccr/action/TransportResumeFollowAction.java
Outdated
Show resolved
Hide resolved
@elasticmachine update branch |
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.
Thank you for adding this flag @andreidan . LGTM, I have one minor comment, (apologies if they are there and I missed them) I was thinking it would be nice to a couple of tests also from DLM to ILM, if it's a path we support it should be tests. It can also come from a subsequent PR.
server/src/main/java/org/elasticsearch/cluster/metadata/DataStream.java
Outdated
Show resolved
Hide resolved
@gmarouli Thanks for the review
That's a great shout, I'll add them in a follow up PR if that's ok (this PR is quite big already) |
@elasticmachine update branch |
Opened #95524 @elasticmachine run elasticsearch-ci/part-2 |
This introduces an index setting that'll allow to switch the preference for the system that manages a data stream backing index. This setting defaults to
true
(preferring ILM when both ILM and DLM are applicable for an index).Note that the prefer_ilm setting will only express a preference, it'll not enforce the presence of a particular management service e.g. if an index has
prefer_ilm: false
and an ILM policy configured, but no DLM configuration, the index will be managed by ILM (despite the preference for DLM)Note that documentation for the setting will be done in a subsequent PR.
There will also be subsequent work to improve the
GET _data_stream
API to better indicate which system manages each backing index.Relates to #93596