Project

General

Profile

Actions

Bug #2140

closed

Parametrized_Classes_in_ENC shouldn't be TRUE by default

Added by Romain Vrignaud over 11 years ago. Updated about 10 years ago.

Status:
Rejected
Priority:
Normal
Assignee:
-
Category:
Puppet integration
Target version:
-
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

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.


Related issues 1 (0 open1 closed)

Related to Foreman - Feature #1995: Enable parameterized class support by defaultClosedDominic Cleal12/04/2012Actions
Actions #1

Updated by Dominic Cleal over 11 years ago

  • 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.

Actions #2

Updated by Benjamin Papillon about 10 years ago

  • Description updated (diff)
  • Status changed from Feedback to Rejected

This bug has no meaning anymore, development went ahead that point.

Actions

Also available in: Atom PDF