search

voxpupuli / puppet-mcollective

MCollective Server and Client Puppet Module
https://forge.puppet.com/puppet/mcollective
Apache License 2.0
120 stars 178 forks source link

Use of $facts in mcollective::actionpolicy::rule conflicts with puppet's Trusted Facts feature #221

Open mcallaway opened 9 years ago

mcallaway commented 9 years ago

See http://docs.puppetlabs.com/puppet/3.6/reference/lang_facts_and_builtin_vars.html#trusted-facts

When one sets trusted_node_data = true in puppet.conf, you also enable the "facts" hash: http://docs.puppetlabs.com/puppet/3.6/reference/lang_facts_and_builtin_vars.html#the-facts-hash

This results in an error:

Error: Could not retrieve catalog from remote server: Error 400 on SERVER: Attempt to assign to a reserved variable name: 'facts' at /etc/puppet/environments/production/modules/mcollective/manifests/actionpolicy/rule.pp:2 on node puppetmaster
igalic commented 8 years ago

i'm intruiged as to how it does this… is that just a random name-clash? /cc @richardc.

richardc commented 8 years ago

I have nothing to say on this.

jyaworski commented 8 years ago

Hello:

Is this still needed?