-
Notifications
You must be signed in to change notification settings - Fork 24.9k
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
Allocate new indices on "hot" or "content" tier depending on data stream inclusion #62338
Allocate new indices on "hot" or "content" tier depending on data stream inclusion #62338
Conversation
…eam inclusion This commit changes the default allocation on the "hot" tier to allocating the newly created index to the "hot" tier if it is part of a new or existing data stream, and to the "content" tier if it is not part of a data stream. Overriding any of the `index.routing.allocation.(include|exclude|require).*` settings continues to cause the initial allocation not to be set (no change in behavior). Relates to elastic#60848
Pinging @elastic/es-core-features (:Core/Features/Features) |
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 implementing this Lee.
I've left a couple of uber minor suggestions
...plugin/core/src/test/java/org/elasticsearch/xpack/cluster/routing/allocation/DataTierIT.java
Outdated
Show resolved
Hide resolved
...plugin/core/src/test/java/org/elasticsearch/xpack/cluster/routing/allocation/DataTierIT.java
Show resolved
Hide resolved
...s/src/internalClusterTest/java/org/elasticsearch/xpack/datastreams/DataTierDataStreamIT.java
Outdated
Show resolved
Hide resolved
...s/src/internalClusterTest/java/org/elasticsearch/xpack/datastreams/DataTierDataStreamIT.java
Outdated
Show resolved
Hide resolved
@elasticmachine run elasticsearch-ci/packaging-sample-windows (CI timed out trying to download gradle) |
1 similar comment
@elasticmachine run elasticsearch-ci/packaging-sample-windows (CI timed out trying to download gradle) |
@elasticmachine update branch |
@elasticmachine update branch |
@elasticmachine update branch |
…eam inclusion (elastic#62338) This commit changes the default allocation on the "hot" tier to allocating the newly created index to the "hot" tier if it is part of a new or existing data stream, and to the "content" tier if it is not part of a data stream. Overriding any of the index.routing.allocation.(include|exclude|require).* settings continues to cause the initial allocation not to be set (no change in behavior). Relates to elastic#60848
This commit changes the default allocation on the "hot" tier to allocating the newly created index
to the "hot" tier if it is part of a new or existing data stream, and to the "content" tier if it is
not part of a data stream.
Overriding any of the
index.routing.allocation.(include|exclude|require).*
settings continues tocause the initial allocation not to be set (no change in behavior).
Relates to #60848