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

Added global alert_context option #21

Closed grega closed 3 years ago

grega commented 8 years ago

Allows:

  class { 'monit':
    alert_emails  => 'example@email.com',
    alert_context => 'but not on { pid, resource, instance }',
  }

which results in the following in monitrc:

 set alert example@email.com but not on { pid, resource, instance }

Ref: https://mmonit.com/monit/documentation/monit.html#Setting-an-event-filter

grega commented 8 years ago

Travis build failure doesn't appear to be related to changes in this PR:

Failures:

  1) monit On a RedHat OS with no package name specified should contain Package[monit]

     Failure/Error: should contain_package('monit')

     Puppet::PreformattedError:
       Evaluation Error: Error while evaluating a Function Call, 'versioncmp' parameter 'a' expects a String value, got Undef at /home/travis/build/echoes-tech/puppet-monit/spec/fixtures/modules/monit/manifests/params.pp:47:10 on node testing-worker-linux-docker-f341370a-3190-linux-4.prod.travis-ci.org
dgarbus commented 8 years ago

I would also like to see this merged. I am currently running this in my test environment without issue.