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

[1.x] Fix nonexistent "os_sec_plugin_conf_path" directory error. #81

Merged

Conversation

rodolfovillordo
Copy link
Contributor

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

Description

Fix missing plugin configuration directory

Issues Resolved

#80

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.

@peterzhuamazon
Copy link
Member

Hi @rodolfovillordo thanks for the contribution.
I am just a bit confused why it is not needed in multi-node setup but only requires in single node?
@prudhvigodithi

@rodolfovillordo
Copy link
Contributor Author

rodolfovillordo commented Jul 20, 2022

Hi @peterzhuamazon I havent tried to deploy a multi-node cluster yet. Those steps was the ones I've done when I identified the error. I've just tried on a multi-node deployment and experienced the same behavior:

TASK [linux/opensearch : Security Plugin configuration | Copy the opensearch security internal users template] ******************************************************************************************************************************************************************************************************************************************************************************
fatal: [os1]: FAILED! => {"changed": false, "checksum": "17c615cd8e9089aedb31891e7aedd1ff8fa32318", "msg": "Destination directory /usr/share/opensearch/plugins/opensearch-security/securityconfig does not exist"}

NO MORE HOSTS LEFT **************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************

PLAY RECAP **********************************************************************************************************************************************************************************************************************************************************************************************************************************************************************************
os1                        : ok=27   changed=22   unreachable=0    failed=1    skipped=10   rescued=0    ignored=0
os2                        : ok=20   changed=17   unreachable=0    failed=0    skipped=9    rescued=0    ignored=0
os3                        : ok=20   changed=17   unreachable=0    failed=0    skipped=9    rescued=0    ignored=0
os4                        : ok=20   changed=17   unreachable=0    failed=0    skipped=9    rescued=0    ignored=0
os5                        : ok=20   changed=17   unreachable=0    failed=0    skipped=9    rescued=0    ignored=0

@peterzhuamazon peterzhuamazon changed the base branch from main to 1.x October 4, 2022 18:10
@peterzhuamazon
Copy link
Member

peterzhuamazon commented Oct 4, 2022

New findings, 1.x and 2.x using different folders for security plugin configs.
#80 (comment)

@peterzhuamazon peterzhuamazon changed the title Fix nonexistent "os_sec_plugin_conf_path" directory error. [1.x] Fix nonexistent "os_sec_plugin_conf_path" directory error. Oct 4, 2022
@peterzhuamazon
Copy link
Member

Need one more PR for the 2.x.

@peterzhuamazon peterzhuamazon linked an issue Oct 4, 2022 that may be closed by this pull request
@peterzhuamazon peterzhuamazon changed the base branch from 1.x to main October 4, 2022 23:18
Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
@peterzhuamazon peterzhuamazon changed the base branch from main to 1.x October 4, 2022 23:20
@peterzhuamazon peterzhuamazon added the backport 1.x Backport to 1.x branch after merging to main label Oct 4, 2022
@peterzhuamazon peterzhuamazon merged commit 20af2f8 into opensearch-project:1.x Oct 4, 2022
@peterzhuamazon peterzhuamazon deleted the fix_issue_80 branch October 5, 2022 00:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 1.x Backport to 1.x branch after merging to main
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG][Security Plugin Configuration]
4 participants