search

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

Additional condition for ignoring of absent selinux module #113

Closed FactorT closed 1 year ago

FactorT commented 1 year ago

Description

When task "Disable the selinux" runs on OS without installed SELinux, for Example Debian, it is failed with error: "An exception occurred during task execution. The error was: ModuleNotFoundError: No module named 'selinux'". Additional condition in "failed_when" allow to ignore error of missing module. The problem was discussed in Selinux module fails if selinux is not installed

Issues Resolved

Issue 111

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.

jordarlu commented 1 year ago

thanks for raising the PR , we will review and get back to you. thanks!

saravanan30erd commented 1 year ago

@FactorT Thanks for the contribution. This PR #122 contains changes / commits from this PR plus some enhancements so we will follow there hence closing this PR.