Issue #7793 has been updated by Adrien Thebo. Status changed from In Topic Branch Pending Review to Rejected
My understanding is that per https://projects.puppetlabs.com/issues/5517#note-39 and https://github.com/puppetlabs/puppet/pull/1706, we explicitly do not want to do this. Right now classes do not behave like resources and we do not want to work towards that end. There are just too many complexities with them, and `noop` has some especially strange behavior in this context. The feature at https://github.com/puppetlabs/puppet/pull/1783 could provide a workaround for this behavior. Since I've rejected the pull request, I'm going to take the preliminary step to mark this as rejected. This doesn't mean that this isn't a valid issue, I'm just closing this with the mindset that we don't intend to work on this right now. ---------------------------------------- Bug #7793: setting the $noop metaparameter as class parameter does not work https://projects.puppetlabs.com/issues/7793#change-95824 * Author: Michael Grüner * Status: Rejected * Priority: Normal * Assignee: * Category: metaparameters * Target version: * Affected Puppet version: 3.0.2 * Keywords: noop * Branch: https://github.com/puppetlabs/puppet/pull/1706 ---------------------------------------- As explained in https://projects.puppetlabs.com/issues/6526 I am trying to set the $noop metaparameter as a class parameter to set a class and all its resources to $noop. This does not seem to work: <pre> mgruener@lt-mgr:~$ cat test.pp class foo($noop=true) { file{ "/tmp/test1": content => "I should not work in noop mode!", } } node default { include foo } mgruener@lt-mgr:~$ envpuppet puppet --version 2.6.8 mgruener@lt-mgr:~$ envpuppet puppet apply test.pp warning: noop is a metaparam; this value will inherit to all contained resources notice: /Stage[main]/Foo/File[/tmp/test1]/ensure: defined content as '{md5}73878c2b0e1386f3127bb77f5a1c96e3' notice: Finished catalog run in 0.01 seconds mgruener@lt-mgr:~$ ls -la /tmp/test1 -rw-r--r-- 1 mgruener mgruener 31 2011-06-06 11:56 /tmp/test1 mgruener@lt-mgr:~$ </pre> I would expect it to work as if the noop parameter was set directly for the resource. -- You have received this notification because you have either subscribed to it, or are involved in it. To change your notification preferences, please click here: http://projects.puppetlabs.com/my/account -- You received this message because you are subscribed to the Google Groups "Puppet Bugs" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected]. Visit this group at http://groups.google.com/group/puppet-bugs. For more options, visit https://groups.google.com/groups/opt_out.
