-
Notifications
You must be signed in to change notification settings - Fork 137
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
base: master
Are you sure you want to change the base?
Conversation
We stumbled across the same issue today. I found your merge request and adapted your changes. Everything works perfectly. Thanks |
@alaunay Thanks for the contribution. |
@AiaiM As it was originally your code, would you have a look at how supporting multiple integ ? |
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:
|
Hello @frittenlab, |
Hello, we did try the integration and it did work with multiple targets. |
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.