Project

General

Profile

Feature #10731

Allow matches to merge with default values.

Added by Alejandro Falcon about 4 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Smart Variables
Target version:
Difficulty:
Triaged:
Bugzilla link:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

I like a lot the merge matches features for hash and array parameters. It would be even better though is we could merge the overrides with the default values as well.

For example: I have user accounts, ssh keys set through hashes and I have a default set of user and keys for all servers and then override specific groups and hosts but currently I have to include same default users in every override. I would like to have a check box to override with default and just add new values in overrides.


Related issues

Related to Foreman - Tracker #4470: Usability of parameters and overridesNew

Related to Foreman - Feature #3309: Support deep merging of hash structures in smart class parametersClosed2013-10-17

Associated revisions

Revision 8347a5ef (diff)
Added by Ori Rabin almost 4 years ago

Fixes #10731 - Allow matches to merge with default values

Revision 77e30352 (diff)
Added by Ori Rabin almost 4 years ago

Refs #10731 - adding match_default to smart variables and smart class parameters

Revision ff06fcb4
Added by Tomas Strachota almost 4 years ago

Merge pull request #194 from orrabin/10731

Refs #10731 - adding match_default to smart variables and smart class parameters

History

#1 Updated by Ohad Levy about 4 years ago

  • Related to Tracker #4470: Usability of parameters and overrides added

#2 Updated by Dominic Cleal about 4 years ago

  • Related to Feature #3309: Support deep merging of hash structures in smart class parameters added

#3 Updated by Dominic Cleal about 4 years ago

  • Category set to Smart Variables

#4 Updated by Ori Rabin about 4 years ago

  • Status changed from New to Assigned
  • Assignee set to Ori Rabin

#5 Updated by The Foreman Bot about 4 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/2606 added
  • Pull request deleted ()

#6 Updated by Dominic Cleal almost 4 years ago

  • Legacy Backlogs Release (now unused) set to 63

#7 Updated by Ori Rabin almost 4 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

Also available in: Atom PDF