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.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Added sudo check to WIA #2801

Merged
merged 2 commits into from
Jan 30, 2024
Merged

Added sudo check to WIA #2801

merged 2 commits into from
Jan 30, 2024

Conversation

davidcr01
Copy link
Contributor

@davidcr01 davidcr01 commented Jan 30, 2024 •
edited

Related issue
#2776

Description

The aim of this PR is to add a check to the Wazuh installation assistant that ensures that the sudo package is installed in the system. We found that in some systems, the sudo is not installed by default and it is necessary for the Wazuh installation.
However, it has not been included as a depenency. Therefore, if this tool is not installed the Wazuh installation is stopped informing with an error

root@debian11sudo:/home/vagrant# bash wazuh-install.sh -a -i -v
29/01/2024 16:40:00 DEBUG: Checking root permissions.
29/01/2024 16:40:00 DEBUG: Checking sudo package.
29/01/2024 16:40:00 ERROR: The sudo package is not installed and it is necessary for the installation.

Tests

馃煝 CentOS 8: https://ci.wazuh.info/job/Test_unattended/5190/
馃煝 CentOS 7: https://ci.wazuh.info/job/Test_unattended/5189/
馃煝 Ubuntu 18: https://ci.wazuh.info/job/Test_unattended/5191/
馃煝 Ubuntu 16: https://ci.wazuh.info/job/Test_unattended/5192/
馃煝 Ubuntu 20: https://ci.wazuh.info/job/Test_unattended/5193/
馃煝 AL2: https://ci.wazuh.info/job/Test_unattended/5194/
馃煝 RHEL7: https://ci.wazuh.info/job/Test_unattended/5195/
馃煝 RHEL8: https://ci.wazuh.info/job/Test_unattended/5188/

@davidcr01 davidcr01 self-assigned this Jan 30, 2024
@teddytpc1 teddytpc1 merged commit cf56ce8 into 4.8.0 Jan 30, 2024
4 checks passed
@teddytpc1 teddytpc1 deleted the bug/2776-check-sudo-package-4.8.0 branch January 30, 2024 11:34
@teddytpc1 teddytpc1 linked an issue Jan 30, 2024 that may be closed by this pull request
Wazuh installation assistant failed #2776
Closed
@davidcr01 davidcr01 mentioned this pull request Feb 1, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@teddytpc1 teddytpc1 teddytpc1 approved these changes

@vcerenu vcerenu vcerenu approved these changes

Assignees

@davidcr01 davidcr01

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Wazuh installation assistant failed
3 participants
@davidcr01 @teddytpc1 @vcerenu