On 2015-20-07 17:59, Alessandro Franceschi wrote:
Thanks Henrik, useful and detailed info as usual. In this case I've to manage a puppet4 version of a function, params_lookup which is widely used in all my old modules, so it's impossible to change the params passed to it without changing all those modules.
Note that there is nothing forcing you to use the 4.x function API (at least not yet) ;-). Also well aware of the special functions people have to do in order to move older code along. Hence, sharing the information about how to achieve this (but have to include the reasons why system functions with "special powers" should be used sparingly).
I fear I'll need to make a system function, even if not recommended. Do you mind to give a look to it, once I complete it, in order to avoid possible bad side effects?
Ping me when you have something for review. - henrik
Thanks as usual al On Monday, July 20, 2015 at 4:10:20 PM UTC+2, henrik lindberg wrote: On 2015-19-07 18:14, Alessandro Franceschi wrote: > This is a silly question that will take to guys like Henrik less than a > nanosecond to reply, I suppose, but I'm struggling hard with this and > haven't found a way out. > > I need to get the value of the compiler variable module_name from within > a Puppet 4 function. > I've tried various approach, the one most resembling the approach in > Puppet 3 function is: > mod = closure_scope['module_name'] > but it doesn't work. > > Any hint? > Your nanosecond is worth at least a sunday afternoon of mine. > The 'closure scope' is the scope the function is defined in. You can also get access to 'calling scope' by creating a 'system function' instead of a regular function. A 'system function' is special as it has more power, but it also comes with more responsibility. Ideally functions should not ever need anything but the arguments given to them. It is good practice to write such pure functions. (i.e. consider requiring that $module_name is given to the function). A 'system function' is written like this: Puppet::Functions.create_function(:inline_epp, Puppet::Functions::InternalFunction) do dispatch :inline_epp do scope_param() param 'String', :template optional_param 'Hash[Pattern[/^\w+$/], Any]', :parameters end def inline_epp(scope, template, parameters = nil) Puppet::Pops::Evaluator::EppEvaluator.inline_epp(scope, template, parameters) end end If you cannot spot it immediately, there are two things you need to do in your function to make it a 'system function': * Use create_function(:name, Puppet::Functions::InternalFunction) instead of just create_functon(:name). * Make the dispatcher inject the calling scope by calling 'scope_param()' in the dispatchers definition of parameters. Then, use the calling scope to get the calling scope's variables. The reason it is bad to directly get variables from a "calling scope" is that it makes it hard to write general functions. What if you in a module want to provide a function that wraps calls to a function that picks up $module_name, or $calling_module from scope? Thus, use 'InternalFunction' and 'scope_param()' sparingly, if at all. - henrik -- Visit my Blog "Puppet on the Edge" http://puppet-on-the-edge.blogspot..se/ <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 <mailto:puppet-dev+unsubscr...@googlegroups.com>. To view this discussion on the web visit https://groups.google.com/d/msgid/puppet-dev/2534c29c-72d7-4bbe-84c8-debe3b59f1c8%40googlegroups.com <https://groups.google.com/d/msgid/puppet-dev/2534c29c-72d7-4bbe-84c8-debe3b59f1c8%40googlegroups.com?utm_medium=email&utm_source=footer>. For more options, visit https://groups.google.com/d/optout.
-- 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/mp1aqt%24e03%241%40ger.gmane.org. For more options, visit https://groups.google.com/d/optout.