Details
-
Bug
-
Status: Closed
-
Normal
-
Resolution: Duplicate
-
None
-
None
-
None
Description
Looking at the puppet-community collectd module I came across something that thoroughly confused me: https://github.com/puppet-community/puppet-collectd/blob/29dd796685469b3dc74a973eefca62ac1ed6e27f/manifests/init.pp#L21-L25
As you can see, the version parameter is defined twice, once with a params lookup, the other time set to installed. Contrary to my expectations this actually compiles. Had this been an actual variable that you tried to define twice Puppet would violently complain.
I expect Puppet not to allow this. It's surprising, a bug (imho) and it's not clear which of the two would win in the case the parameter is not explicitly passed in.
Attachments
Issue Links
- duplicates
-
PUP-5590 No error on duplicate parameters on classes and resources
-
- Closed
-