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

Check OS cluster for data streams and templates for index template creation #106

Merged
merged 1 commit into from
Jun 26, 2023

Conversation

adiforluls
Copy link
Member

DESCRIPTION HERE
Fixes #103

(check all that apply)

  • tests added
  • tests passing
  • README updated (if needed)
  • README Table of Contents updated (if needed)
  • History.md and version in gemspec are untouched
  • backward compatible

…eation

Signed-off-by: adiforluls <aditya.b.bharadwaj@oracle.com>
@adiforluls
Copy link
Member Author

cc: @cosmo0920
I see coverage has dropped but I'm not sure what the test for this would be like. Let me know if I should add a test.

@cosmo0920
Copy link
Collaborator

Seems reasonable. Don't worry for declined coverage. For this case, it's not worth to add test cases.

@adiforluls
Copy link
Member Author

Thanks for the quick review and approval Hiroshi!

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.

opensearch_data_stream: Check if the index template exists every time before pushing data
2 participants