search

metrixware-echoes-tech / puppet-monit

Puppet module to manage monit installation and configuration
https://forge.puppetlabs.com/echoes/monit
Apache License 2.0
4 stars 48 forks source link

Correct scoping of monit_version in init.pp #15

Closed jonoterc closed 8 years ago

jonoterc commented 8 years ago

I'm using start_delay and encountering comparison errors when puppet tries to compare $::monit_version, which evaluates as nil. Scoping as $::monit::params::monit_version addresses this.

Thanks for sharing this module, it's looking quite useful for us!

ghoneycutt commented 8 years ago

This is fixed in PR #20

marcgascon commented 8 years ago

+1 but you can also use:

$monit_version

because you are inheriting from params, so you don't need to provide the top scope variable path.

Appart from that, I have the same problem when trying to use the module (I downloaded from puppet forge). So please could you merge this @floSoX ???

Many thanks in advance.

FlorentPoinsaut commented 8 years ago

Hi,

Thanks for your interest and your work. And please, excuse me for the delay. I published a new release today.

Regards,