-
-
Notifications
You must be signed in to change notification settings - Fork 94
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
adapt for new mail notification script as of icinga >=v2.7.0 #361
Comments
The mentioned changes are problematic when updating Icinga 2. I don't see how we could catch this with the Puppet module, can you give some more details about the changes you request? |
We only manage icinga2 objects not the content of an object. You have to mange the content on your own. |
Well, the provided example config is broken with >2.7.0 AFAICS: https://github.com/Icinga/puppet-icinga2/blob/f9165ba556a3c2d9713fb17c0932b0cc81aa3927/examples/example_config.pp#L223 |
But it is just an example.... |
With icinga v2.7.0 notifications per mail are broken if the changes as documented at Icinga/icinga2@bd7d7ef + https://www.icinga.com/2017/08/23/advisory-for-icinga-2-v2-7-update-and-mail-notification-scripts/ aren't applied.
The text was updated successfully, but these errors were encountered: