Project

General

Profile

Actions

Feature #32741

closed

add merge-option for variables of type "yaml" (Ansible/Salt/Puppet)

Added by Jonas Trüstedt almost 3 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Normal
Category:
Parameters
Target version:
-
Difficulty:
Triaged:
No
Fixed in Releases:
Found in Releases:

Description

Currently for the different variable-types in Puppet/Ansible/Salt only Arrays and Hashes support the Option to Merge (Merge, Merge-with-default, Merge-remove-duplicates).
Depending on how you define your variables, you might want to use other types as well.

For example if you build a puppet-class where you declare only one variable of type "yaml" to reduce the amount of variables and to define everything you need per class in just one variable. In this case you often end up that in certain environments/matchers you want to add something without having to copy&paste everything. In this case a merge-option for "yaml" would be great.

Since in the current case it is puppet, it applies to salt and ansible as well (and maybe other? Host-parameter?) and it might be useful as well for other types, such as "json".
For this I put it in the general foreman-project and not three times for the different configuration-management-plugins.

Actions #1

Updated by The Foreman Bot almost 3 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Bernhard Suttner
  • Pull request https://github.com/theforeman/foreman/pull/8614 added
Actions #2

Updated by The Foreman Bot almost 3 years ago

  • Fixed in Releases 3.0.0 added
Actions #3

Updated by Anonymous almost 3 years ago

  • Status changed from Ready For Testing to Closed
Actions #4

Updated by Tomer Brisker over 2 years ago

  • Category set to Parameters
Actions

Also available in: Atom PDF