Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.

...

Page Properties
label


Status
Status
colourGreen
titleIN PROGRESSDECIDED
Impact

Status
colourYellow
titleMEDIUM

Driver
Approver
Contributors
Informed
Due date
OutcomeOption 2


Background

As users begin to mix more publicly available modules into SIMP systems, we run the risk of fact conflicts where the first one that gets loaded wins. This is quite likely to cause unexpected consequences in both our code and the code of other people.

...

1

Option 0:Option 1:Option 2:Option 3:
Descriptionsimp_<fact_name><modulename>_<fact_name> (single_underscore)<modulename>__<fact_name> (double underscore)<module_name>-<fact_name>
Pros and cons

(plus) Easy to understand

(minus) The hard-coded simp_ prefix would make transitioning ownership of a module to another organization awkward.

(plus) Easy to understand

(minus) May run into issues if <module_name> corresponds to a fact with an underscore

(plus) Minimizes risk of conflicts with any system fact value

(minus) Kind of strange syntactically (but not incorrect)

(plus) VERY unlikely to conflict with anything

(minus) The hyphen is a non-idiomatic character for fact names

(minus) Facts using this can't be accessed in classic $::fact_name format

(minus) Facter and Puppet (via $facts[]) accept this syntax , but there may be weird edge cases or tools that don't

(minus) Puppet recommends not making facts that are not valid puppet variable names since the behavior will be non-deterministic.

Estimated cost
Status
colourGreen
titleSMALL
Status
colourGreen
titleSMALL
Status
colourGreen
titleSMALL
Status
colourGreen
titleSMALL
Votes

2

Kendall Moore

Brandon Riden (Unlicensed)

0

6

Shark Bruhaha

Judith Johnson

0

Action items

  •  

Outcome