opensearch-project / ansible-playbook

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

Remove unecessary logstash user from default deployment #87

Closed rodolfovillordo closed 1 year ago

rodolfovillordo commented 2 years ago

Signed-off-by: Rodolfo Camara Villordo rodolfovillordo@gmail.com

Description

PR #63 left an unnecessary user for the default OpenSearch deployment. This PR is to remove the user from the internal_users.yml file.

Issues Resolved

Fix #86

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.

rodolfovillordo commented 2 years ago

I've identified another issue when deploying a multi-node cluster. Working on the fix ATM.

rodolfovillordo commented 2 years ago

Sorry, I had forgotten to apply the fix for issue #83 and on multi-node deployment the behavior is different when it's not applied. ref: https://github.com/opensearch-project/ansible-playbook/issues/83#issuecomment-1190783403

peterzhuamazon commented 1 year ago

@rodolfovillordo there is still another referencing in files/roles_mapping.yml.

peterzhuamazon commented 1 year ago

Hi @rodolfovillordo any progress on this one? Thanks.

prudhvigodithi commented 1 year ago

Hey @rodolfovillordo closing this PR, please re-open if you think it is still useful. Thank you