commit 5868593634371290ad013e4a3005f25cb8d7e1fe broke the module for me about puppet-unbound HOT 6 CLOSED

removing the Unix::Path from stub.pp, it fails then with:

Error: Could not retrieve catalog from remote server: Error 500 on SERVER: Server Error: Evaluation Error: Error while evaluating a Resource Statement, Evaluation Error: Error while evaluating a Function Call, Failed to parse template unbound/stub.erb:
Filepath: /etc/puppetlabs/code/environments/production/modules/unbound/templates/stub.erb
Line: 4
Detail: uninitialized constant Puppet::Parser::TemplateWrapper::PuppetX
Did you mean? Puppet_X
PuppetDB
Puppet::Parser::TemplateWrapper::Puppetdb
(file: /etc/puppetlabs/code/environments/production/modules/unbound/manifests/stub.pp, line: 41, column: 16) (file: /etc/puppetlabs/code/environments/production/modules/profile/manifests/dnsserver.pp, line: 35) on node mynode
Warning: Not using cache on failed catalog
Error: Could not retrieve catalog; skipping run

from puppet-unbound.

Hmm, it looks like the path reference should be Stdlib::Unixpath I suspect. I'll test the latest in my environment sometime soon.

from puppet-unbound.

#222 is incomplete. I'll come back.

from puppet-unbound.

Hi @buzzdeee, I believe #222 will solve the issue, can you confirm? Thanks for putting up the issue.

from puppet-unbound.

@xaque208
I tested #222 and it generally works for me.
Only note: before that I had the stub IP address as a string, now it requires an Array due to the loop in the template.
The stub.pp still allows Array and String. So either the stub.pp needs to be more strict, or the template should be more relaxed.

from puppet-unbound.

this was fixed in #222

from puppet-unbound.