camptocamp/puppet-bind

Missing path in Exec[reload bind9] in base.pp

Closed this issue · 4 comments

slapers commented

Error: Failed to apply catalog: Parameter onlyif failed on Exec[reload bind9]: 'named-checkconf -jz /etc/bind/named.conf' is not qualified and no path was specified. Please qualify the command or specify a path.

This seems to solve the issue:

exec {'reload bind9':
command => 'service bind9 reload',
onlyif => 'named-checkconf -jz /etc/bind/named.conf',
path => ['/sbin', '/usr/sbin'],
refreshonly => true,
}

cjeanneret commented

Hello,

In fact, we're used to declare a global path, for example in site.pp:

Exec {
  path => '/what/you/need:/what/you/want:/some/other',
}

This way we prevent any differences between operating system.
We should add this to the readme file, I guess.

Cheers,

C.

slapers commented

Hi,

Jup, that would be a solution too :-)

thanks,
Stefan

braddeicide commented

This variable dependency should be listed in the readme, or since this module has OS detection it could have defaults.

cjeanneret commented

Hello,
I just added the Exec stuff in the readme.

Thus I'm closing this issue.

Cheers,

C.