search

inkblot / puppet-bind

18 stars 81 forks source link

Feature: `$extra_options` hash parameter #126

Closed cedef closed 7 years ago

cedef commented 7 years ago

This parameter allow any option to be added to the named.conf options {}; directive.

inkblot commented 7 years ago

I have repeatedly and consistently been against having an options hash parameter. Hiera treats such values as unitary, when in practice the members of such a hash will vary independently. This makes overrides of individual options specified in such a hash very awkward.

cedef commented 7 years ago

I have repeatedly and consistently been against having an options hash parameter. Hiera treats such values as unitary, when in practice the members of such a hash will vary independently. This makes overrides of individual options specified in such a hash very awkward.

With Hiera/lookup deep merge, options hash parameters are fine (IMO)

Should I submit a patch to implement additional options I need as individual scalar parameters ?

inkblot commented 7 years ago

Deep merge is not an option for automatic parameter lookups. Please add the scalar parameters you need.