On 2016-04-02 11:46, Martin Alfke wrote:

On 01 Feb 2016, at 19:15, Henrik Lindberg <henrik.lindb...@cloudsmith.com> 
wrote:


That is exactly what you should do. An external (file based epp) when called, 
does not get to see variables in the scope from which it was called/used. This 
design is deliberate. Think of the template as a function you are calling, and 
you have to give it its arguments.

In this case the documentation on the website is wrong:
https://docs.puppetlabs.com/puppet/latest/reference/lang_template_epp.html#example-template

The long example is wrong in that it does not show that arguments must be given in a hash.

The documetation for epp has unfortunately been boiled down to just the bare minimum and the important distinction that variables must be given has been lost in the series of rewrites that were made.

The implementation clearly restricts epp() to only see global scope.

The specification is also clear about scoping: https://github.com/puppetlabs/puppet-specifications/blob/master/language/templates.md#visibility-of-scoped-variables

I filed a documentation ticket.

The example just switches from template() to epp() function.
According to your writing you must pass all parameters to the epp() function.


yes, epp() requires that arguments are given in the call. epp() templates only have access to global variables (i.e. $::osfamily and such).



Contrast this with the inline_epp, which you can think of as a 
lambda/code-block. Here the code block gets to see the variables in scope, 
since it is itself in that scope (part of the same piece of code).

Got it.
To be honest: this is ugly.
I always thought of epp() being a full replacement for template() without the 
scope issue.

The ERB based template() function is for templates in files. The epp() function is also for files. The big difference between them is that all variables used by the template processed by epp() must be given to the function.

The inline_epp() may in the special case, when it does not declare parameters have access to all variables in scope. This was deemed to be harmless as all of the logic is in one place, and there is never a question of reuse. It is also seen as an easy way to transition to inline_epp() from inline_template().

With that, I am not sure I understand your comments. What is it that you think is ugly?

This especially feels bad, when we will have performance improvement for epp in 
the future.


I have not seen tickets with reports of performance problems. How bad is it?

Best
- henrik

--

Visit my Blog "Puppet on the Edge"
http://puppet-on-the-edge.blogspot.se/

--
You received this message because you are subscribed to the Google Groups "Puppet 
Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to puppet-dev+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/puppet-dev/n8vjpn%244u0%241%40ger.gmane.org.
For more options, visit https://groups.google.com/d/optout.

Reply via email to