Hi, On 04/04/2011 06:13 PM, Marcus Downing wrote: > As I said, the "sourceselect => all" parameter is necessary to make > puppet merge the listed sources rather than selecting one of them.
ah, thanks for pointing that out again. I completely missed that (nor knew of its existence). > I take your point that variables are a fragile way of specifying. I'll > try refactoring it based on class parameters, but the list of sources > isn't the same length for the different nodes so it'll still have to be > an array; and it will need to add that array to a list of default > sources shared by all instances (that I'd rather not repeat for every one). I don't see an issue with passing this parameter as an array. Alternatively (if the set of options is limited, which I would assume seeing that you used something not unlike inheritance before), you could just pass a flag parameter (sources = "all", sources = "some" etc.) and declare your array of sources in a case statement inside the parameterized class. Regards, Felix -- You received this message because you are subscribed to the Google Groups "Puppet Users" group. To post to this group, send email to puppet-users@googlegroups.com. To unsubscribe from this group, send email to puppet-users+unsubscr...@googlegroups.com. For more options, visit this group at http://groups.google.com/group/puppet-users?hl=en.
