-
Notifications
You must be signed in to change notification settings - Fork 178
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
Fixes in Molecule tests, Wazuh components installation and Versioning Control #206
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…n ansible and molecule packages' versions
…ge version - Differentiating between CentOS/RedHat and Debian
…d filebeat installations and testings
…va in Ubuntu Trusty
… - Test passed successfully
manuasir
suggested changes
Jul 17, 2019
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 permissions here look incorrect. Please fix it and push again, thanks!
manuasir
reviewed
Jul 17, 2019
manuasir
reviewed
Jul 17, 2019
manuasir
reviewed
Jul 17, 2019
manuasir
reviewed
Jul 17, 2019
manuasir
suggested changes
Jul 17, 2019
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.
Hi @rshad , please add the commented changes. thanks
manuasir
reviewed
Jul 17, 2019
manuasir
approved these changes
Jul 17, 2019
This was referenced Jul 23, 2019
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hi team!
This PR includes many fixes and issues resolving for Molecule tests, Ansible roles and other components in wazuh-ansible repository.
1- Molecule Tests:
Molecule tests, found in the branch 3.9.2_7.1.1_tests were not working as expected. Many errors from different types in different roles. Errors like:
1.1 - Fixed Ansible-Lint errors. => Trailing spaces, changing shell directive by command, etc ...
1.2 - Fixed installation issues for many roles, but dedicated more than half the time to Elasticsearch issues which can be resumed as follows:
Java
installed. However, Elasticsearch 7.* indicates that there is no need to install Java explicitly because it already includes a JVM in its installation. But this way was not the case in CentOS 6 and Ubuntu Trusty. Check here the official documentation1.3 - Idempotence Test errors:
We adapted the failing tasks in order to idempotent. so a task which already made a change in the system, when the idempotence test is run, this change must not be produced again.
1.4 - MAX LOCKED MEMORY limits issue.
When running
Molecule
tests on containers for all distribution, sometimes "not always", an error was produced in Ubuntu Trusty when trying to start Elasticsearch:After investigating, we discovered that this issue is related to the privileges of the user running the corresponding task in Ubuntu Trusty, in such case the user needed more privileges than the ones given by default. in order to solve this issue, we needed to add the directive
privileged: true
for Ubuntu Trusty.With this error message, we also faced warnings such as:
by which the execution failed. CHECK A POSSIBLE SOLUTION HERE
.2. Versioning Control for Wazuh components when installing with Ansible
In this issue
https://github.com/wazuh/wazuh-ansible/issues/197
we had a request to add versioning for Wazuh Manager, Agent, and Filebeat. This functionality was added successfully for each Wazuh component. So now, we can indicate a determined version of Wazuh or Elasticsearch when running wazuh-ansible installation..3. Create Ansible playbook in order to automate the uninstallation of already installed Wazuh components
We created a new role in order to uninstall wazuh-manager.
Related Branches
Related Issues
Kind Regards,
Rshad Zhran