Skip to content
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

Add missing ISM rollover index setting for OTel span indices #3580

Conversation

JannikBrand
Copy link
Contributor

@JannikBrand JannikBrand commented Nov 2, 2023

Signed-off-by: Jannik Brand [email protected]

Description

Indices require the rollover_alias ISM setting otherwise rollover actions by an ISM policy (raw-span-policy) are failing. Reference: https://opensearch.org/docs/2.11/im-plugin/ism/error-prevention/resolutions/#the-rollover-policy-misses-rollover_alias-index-setting

Issues Resolved

Resolves #3506

Check List

  • New functionality includes testing.
  • New functionality has a documentation issue. Please link to it in this PR.
    • New functionality has javadoc added
  • Commits are signed with a real name per the DCO

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Indices require the rollover_alias ISM setting otherwise rollover actions by an ISM policy (raw-span-policy) are failing.
Reference: https://opensearch.org/docs/2.11/im-plugin/ism/error-prevention/resolutions/#the-rollover-policy-misses-rollover_alias-index-setting

Issue: opensearch-project#3506

Signed-off-by: Jannik Brand <[email protected]>
@juergen-walter
Copy link
Contributor

Superseded by #3590

@dlvenable
Copy link
Member

@JannikBrand , Feel free to take a look at #3590. I think this current PR would not work because of a limitation in the opensearch-java client. In #3590, I worked around that by using a Map and customizing the OpenSearch request to put the index template.

@JannikBrand JannikBrand closed this Nov 7, 2023
@JannikBrand JannikBrand deleted the missing-rollover-alias-setting branch November 7, 2023 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] ISM index rollover actions fail because of missing setting for otel-v1-apm-span-* indices
3 participants