diff options
| author | Gabriel Filion <lelutin@gmail.com> | 2012-07-19 03:22:46 -0400 |
|---|---|---|
| committer | Gabriel Filion <lelutin@gmail.com> | 2012-07-19 03:22:46 -0400 |
| commit | 3053b8b340b86d7fd23d2f0c8450a83f3991baef (patch) | |
| tree | e2f5f8f0eebdee8a1df476b8e0e74cdd4e8b80f4 /manifests/plugins/setup.pp | |
| parent | 1d429bf94412228de056b4517ac4b1e6548b9bca (diff) | |
Fix requirements for plugins
When including munin::host without explicitly including munin::client,
puppet fails to run with the following message:
Could not find dependent Service[munin-node] for
File[/etc/munin/plugins] at
/tmp/vagrant-puppet/modules-0/munin/manifests/plugins/setup.pp:14
This is because munin::host declares a munin plugin, and in turn this
one creates a dependency on the 'munin-node' package and service. Since
this dependency is not fulfilled, the run fails.
The munin::plugins::setup class needs to fulfill its own dependencies by
including munin::client.
Signed-off-by: Gabriel Filion <lelutin@gmail.com>
Diffstat (limited to 'manifests/plugins/setup.pp')
| -rw-r--r-- | manifests/plugins/setup.pp | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/manifests/plugins/setup.pp b/manifests/plugins/setup.pp index c2c5ed3..853bc00 100644 --- a/manifests/plugins/setup.pp +++ b/manifests/plugins/setup.pp @@ -1,4 +1,8 @@ class munin::plugins::setup { + + # This is required for the munin-node service and package requirements below. + include munin::client + file { [ '/etc/munin/plugins', '/etc/munin/plugin-conf.d' ]: source => "puppet:///modules/common/empty", |