diff options
| author | varac <varacanero@zeromail.org> | 2015-03-10 22:51:31 +0100 |
|---|---|---|
| committer | varac <varacanero@zeromail.org> | 2015-03-11 21:58:10 +0100 |
| commit | 0c916d8e70bdb6165fe25d6f4e3ceadb1c0622a0 (patch) | |
| tree | f6565e0ec97bafec65abcd96942b2161ad91305e /puppet/modules/site_check_mk/manifests/agent/mrpe.pp | |
| parent | 96882181d7cdcdf2c10918c007ae15dfd566410a (diff) | |
Use augeas instead of file_line to configure entries in /etc/check_mk/mrpe.cfg (Bug #6788)
We used file_line before, but when the some check parameters change, a
new line would be added, leaving the old line there, resulting in two
checks with the same name but with different parameters.
Augeas can handle this better, but it is important to use 'rm' to
remove all old lines with different parameters before adding the
new line.
Change-Id: Iad69dfd20f487a16d372a4f4a4bc53299f9e4a66
Diffstat (limited to 'puppet/modules/site_check_mk/manifests/agent/mrpe.pp')
| -rw-r--r-- | puppet/modules/site_check_mk/manifests/agent/mrpe.pp | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/puppet/modules/site_check_mk/manifests/agent/mrpe.pp b/puppet/modules/site_check_mk/manifests/agent/mrpe.pp index 6921574f..5e1f087a 100644 --- a/puppet/modules/site_check_mk/manifests/agent/mrpe.pp +++ b/puppet/modules/site_check_mk/manifests/agent/mrpe.pp @@ -11,8 +11,14 @@ class site_check_mk::agent::mrpe { ensure => present, require => Package['check-mk-agent'] } -> - file_line { 'Apt': - line => 'APT /usr/lib/nagios/plugins/check_apt', - path => '/etc/check_mk/mrpe.cfg', + + augeas { + 'Apt': + incl => '/etc/check_mk/mrpe.cfg', + lens => 'Spacevars.lns', + changes => [ + 'rm /files/etc/check_mk/mrpe.cfg/APT', + 'set APT \'/usr/lib/nagios/plugins/check_apt\'' ]; } + } |