Bug #2140
closed
Parametrized_Classes_in_ENC shouldn't be TRUE by default
Added by Romain Vrignaud about 12 years ago.
Updated almost 11 years ago.
Category:
Puppet integration
|
Description
As Parametrized_Classes_in_ENC is able
to change puppet behavior, I strongly
recommands that this function is not activated
by default during Foreman's migration.
- Category set to Puppet integration
- Status changed from New to Feedback
- Priority changed from High to Normal
The issue is a change in the ENC output between the array (which was alphabetical) and the hash (which is effectively random) causes Puppet to parse manifests in a different order. In this case, a hosts::server
class referenced a variable in hosts
, but didn't explicitly include hosts
. When the ENC output order changed, hosts::server
was parsed first and couldn't resolve the variable. The fix is a manifest change to explicitly include it.
Roman's argument is that this is a bad default if it can cause behavioural changes during upgrades.
I think this should definitely be enabled for fresh installs, but can possibly see the argument for an upgrade. I'd much rather it was left on and documented in release notes with other breaking changes.
- Description updated (diff)
- Status changed from Feedback to Rejected
This bug has no meaning anymore, development went ahead that point.
Also available in: Atom
PDF