opensearch-project / ansible-playbook

🤖 A community repository for Ansible Playbook of OpenSearch Project.
https://opensearch.org/
Apache License 2.0
86 stars 96 forks source link

Creating Index patterns via role #68

Closed ssi444 closed 4 months ago

ssi444 commented 2 years ago

Description

Added the ability to automatically create index patterns.

Issues Resolved

https://github.com/opensearch-project/ansible-playbook/issues/66

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.

ssi444 commented 2 years ago

Calling @saravanan30erd @prudhvigodithi for the review. Thanks.

ssi444 commented 2 years ago

@peterzhuamazon @saravanan30erd @prudhvigodithi ping

peterzhuamazon commented 2 years ago

@saravanan30erd Please let me know if you have bandwidth to review this. @ssi444 Sorry we are focusing on two minor releases in the past weeks that is why this is pretty slow. Thanks.

ssi444 commented 2 years ago

@peterzhuamazon @saravanan30erd @prudhvigodithi ping

ssi444 commented 2 years ago

@peterzhuamazon @saravanan30erd @prudhvigodithi ping

prudhvigodithi commented 2 years ago

Hey @ssi444 can you please elaborate this functionality, we expect users to add multiple indexes under a pattern, in this scenario how does this functionality helps? Also with this can we load an index to created pattern?

ssi444 commented 2 years ago

Hey @ssi444 can you please elaborate this functionality, we expect users to add multiple indexes under a pattern, in this scenario how does this functionality helps? Also with this can we load an index to created pattern?

@prudhvigodithi After sending logs is configured, indexes for incoming data are created in OpenSearch. But it is impossible to search these indexes until an IndexPattern is created. This change allows you to create the necessary IndexPattern in different Tenants based on the configuration file, and not manually. This allows this role to take another step to manage the OpenSearch cluster using the IaC methodology

prudhvigodithi commented 1 year ago

Hey @ssi444 apologies for the delay, do you think we still need this change to move forward?

ssi444 commented 1 year ago

Hey @ssi444 apologies for the delay, do you think we still need this change to move forward?

yes, I think this is the necessary functionality

bbarani commented 1 year ago

@prudhvigodithi @saravanan30erd @peterzhuamazon Anyone looking in to this PR?

dblock commented 4 months ago

Looks like a good feature, however the PR got stalled and wasn't merged, sorry about that. Closing it for now, if you feel it does need to be merged, please rebase, and I can try to assign it to a maintainer in this repo.

[Catch All Triage - Attendees 1, 2, 3, 4, 5]