-
Notifications
You must be signed in to change notification settings - Fork 96
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
Creating Index patterns via role #68
Conversation
Signed-off-by: Sergey Shubin <[email protected]>
Signed-off-by: Sergey Shubin <[email protected]>
…and set them /bin/false shell Signed-off-by: Sergey Shubin <[email protected]>
1. Added the ability to log in via OpenID 2. Added the ability to install custom configuration files for the cluster 3. Added the ability to reconfigure the cluster (in particular, update certificates) when expanding it 4. Added the ability not to change certificates if the cluster composition has not changed, but only the settings have changed. Signed-off-by: Sergey Shubin <[email protected]>
Signed-off-by: Sergey Shubin <[email protected]>
…cies" section have been added to the readme. Signed-off-by: Sergey Shubin <[email protected]>
…patterns" section have been added to the readme. Signed-off-by: Sergey Shubin <[email protected]>
Calling @saravanan30erd @prudhvigodithi for the review. |
@saravanan30erd Please let me know if you have bandwidth to review this. |
1 similar comment
then only in this case the task of setting passwords for internal users from variables is started. If the file `internal_users.yml` | ||
is not located in the `files` directory, but, for example, in one of its subdirectories, then playbook will not work correctly | ||
|
||
### IaC (Infrastructure-as-Code) |
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.
pkPassword: none | ||
file: root-ca.pem | ||
|
||
### Default values and global settings | ||
defaults: | ||
validityDays: 730 | ||
validityDays: {{ cert_valid_days }} |
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.
Same as this existing PR @ssi444
https://github.com/opensearch-project/ansible-playbook/pull/63/files
Should we revisit this until the other is merged ?
@saravanan30erd @peterzhuamazon
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.
The code is divided into logical blocks at @saravanan30erd request, so there are 3 PR's. @prudhvigodithi
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.
I suggest merging the code in the order of creating PR
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 |
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 |
@prudhvigodithi @saravanan30erd @peterzhuamazon Anyone looking in to this PR? |
Description
Added the ability to automatically create index patterns.
Issues Resolved
#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.