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

allow integration management via hiera #907

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

alaunay
Copy link

@alaunay alaunay commented Jan 17, 2024

Hello,

This patch allows the configuration of integration of external tools (mattermost, slack, etc) using hiera.

Previously, there was a hook
define wazuh::integration()

which wasn't in use anywhere.

@frittenlab
Copy link

We stumbled across the same issue today. I found your merge request and adapted your changes. Everything works perfectly. Thanks

@jzelina
Copy link

jzelina commented Jan 17, 2024

@alaunay Thanks for the contribution.
From trying your change we notices that so far (only) one integration seams to supported via hiera.
As Wazuh can handle multiple integration configurations, a support for multiple integrations would also be valuable to consider.

@alaunay
Copy link
Author

alaunay commented Jan 17, 2024

@AiaiM As it was originally your code, would you have a look at how supporting multiple integ ?

@alaunay
Copy link
Author

alaunay commented Jan 19, 2024

Hello @jzelina,
@AiaiM modified the code to support multi-integration, can you check if it fits your needs ?

@frittenlab
Copy link

Thanks for your contribution. I found one minor mistake. You renamed the template file to _integrations_epp. The file reference in the params_manager.pp still points to the old file. It should be:

$ossec_integration_template = 'wazuh/fragments/_integration.epp'

@AiaiM
Copy link

AiaiM commented Jan 22, 2024

Hello @frittenlab,
I have fixed the file path in the params_manager.pp.

@jzelina
Copy link

jzelina commented Jan 22, 2024

Hello, we did try the integration and it did work with multiple targets.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants