Review and fix for the exec resource declarations in wazuh::indexer, wazuh::dashboard and wazuh::filebeat_oss

[vaskosmihaylov]

Description

Classes

Indexer, Dashboard, Filebeat

Behavior

• Improving the exec resources in the aforementioned classes

Indexer code

Dashboard code

Filebeat related code

The above exec declarations run on every puppet run even though the certificates, folders and configuration are already added:

• Also, the following exec is incorrect:

    path    => '/usr/bin:/bin/',
    command => "echo 'elasticsearch - nofile  65535\nelasticsearch - memlock unlimited' >> /etc/security/limits.conf",
    require => Package[$indexer_package],
  }

It echoes the user elasticsearch in the /etc/security/limits.conf while all the wazuh-indexer directories are owned by wazuh-indexer user and also running on every puppet run it add the line again and again:

grep elasticsearch /etc/security/limits.conf | wc -l
20

Solution

Our team would like to create a PR request, fix and improve the exec declaration and submit a merge request, however before proceeding further we would like to know if this is applicable for the Wazuh team and also if our merge request will be approved.

[vcerenu]

Hello @vaskosmihaylov

We have analyzed and tested the PR that you sent us and it is already merged into the development branch of the current version, so it will be available in the next Wazuh release.

Thank you very much for the development and if you have any questions or ideas, do not hesitate to contact us.