Skip to content

IndexSettingsLifecycle requires a name to be set #456

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.

Sign up for GitHub

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

Jump to bottom
Closed
christian-ertl opened this issue Nov 30, 2022 · 2 comments · Fixed by #776
Closed

IndexSettingsLifecycle requires a name to be set #456

christian-ertl opened this issue Nov 30, 2022 · 2 comments · Fixed by #776
Labels
Area: Specification Related to the API spec used to generate client code

Comments

@christian-ertl
Copy link

Java API client version

8.5.2

Java version

11

Elasticsearch Version

8.4.1

Problem description

IndexSettingsLifecycle requires a name to be set which is not necessary if the API is used directly. According to the JavaDoc, this is the name of the policy to be used to manage the index. We use component templates to specify this setting.

The setting that I want to overwrite for the specific index template is the rollover alias. We could work around this issue by specifying it through otherSettings on IndexSettings.
@christian-ertl
Copy link
Author

It's actually worse since this cannot be worked around when reading the templates.

@l-trotta l-trotta added the Area: Specification Related to the API spec used to generate client code label Mar 8, 2024
@l-trotta
Copy link
Contributor

l-trotta commented Mar 8, 2024

Hello, thanks for the report! This is an issue in the API specification used to produce the Java code, the policy name should be optional. It will be fixed soon and the Java client code will be updated to resolve this issue.

This was referenced Apr 4, 2024
Merged
Merged
Merged
JoshMock added a commit to elastic/elasticsearch-specification that referenced this issue Apr 9, 2024
@JoshMock
Index settings lifecycle name should be optional, not ILM policy name
fe0c8bf 
See elastic/elasticsearch-java#456
@JoshMock JoshMock mentioned this issue Apr 9, 2024
Merged
JoshMock added a commit to elastic/elasticsearch-specification that referenced this issue Apr 10, 2024
@JoshMock
Index settings lifecycle name should be optional, not ILM policy name (…
c2b68f5 
…#2498)

See elastic/elasticsearch-java#456
github-actions bot pushed a commit to elastic/elasticsearch-specification that referenced this issue Apr 10, 2024
@JoshMock @github-actions
Index settings lifecycle name should be optional, not ILM policy name (…
8157538 
…#2498)

See elastic/elasticsearch-java#456

(cherry picked from commit c2b68f5)
l-trotta pushed a commit to elastic/elasticsearch-specification that referenced this issue Apr 10, 2024
@github-actions @JoshMock
Index settings lifecycle name should be optional, not ILM policy name (…
3565380 
…#2498) (#2500)

See elastic/elasticsearch-java#456

(cherry picked from commit c2b68f5)

Co-authored-by: Josh Mock <[email protected]>
l-trotta added a commit to elastic/elasticsearch-specification that referenced this issue Apr 16, 2024
@l-trotta @JoshMock
Index settings lifecycle name should be optional, not ILM policy name (…
8ec1321 
…#2498) (#2500)

See elastic/elasticsearch-java#456

(cherry picked from commit c2b68f5)

Co-authored-by: Josh Mock <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Area: Specification Related to the API spec used to generate client code
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fixes from pr 2485 elastic/elasticsearch-java
2 participants
@christian-ertl @l-trotta