search

puppetlabs / r10k

Smarter Puppet deployment
Other
802 stars 355 forks source link

Not sure about the hiera.yaml statement in common-patterns #1350

Closed lesinigo closed 11 months ago

lesinigo commented 1 year ago

The current common-patterns.md states:

Use a Control Repo to store your Puppetfile. Hiera data should be in the Control repo OR as a separate source in r10k.yaml. Any hiera.yaml in the Control repo will be ignored on a per environment basis, locating it at /etc/puppetlabs/puppet/hiera.yaml is prefered. Each puppet module should be contained in its own independent forge module or repository.

It may be me, but I'm not understanding the part in bold, it seems to contradict the official Puppet documentation that clearly says that Hiera will use (among others) any per-environment hiera.yaml that it gets.

That file is 6 years old, could it be that it was referring to some older version of Puppet and it is now wrong? Do we need to fix the documentation or is there something else that I'm missing?

justinstoller commented 11 months ago

Yes, the documentation is outdated. Thanks for the heads up, @lesinigo ! I'll fix shortly.

justinstoller commented 11 months ago

Closing this as there's now PR open to track resolving.