Project

General

Profile

Refactor #15846

Rename use_puppet_default in lookup values to skip_foreman

Added by Shimon Shtein almost 3 years ago. Updated 11 months ago.

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

Description

As part of the effort to remove puppet specific properties from the core, we decided to "upgrade" use_puppet_default into something that any type of parameter could benefit from. We want to rename it to skip_foreman since it's a generic behavior that could be served by the core. The meaning remains pretty much the same - the flag signals the core that it should not manage (and serve) this parameter for a given host.


Related issues

Related to Foreman - Tracker #9596: Extract Puppet functionality to a pluginNew2015-03-01

Related to Foreman - Refactor #12822: Separate LookupValue to different classes per LookupKey typeNew2015-12-15

Associated revisions

Revision 569650b7 (diff)
Added by Shimon Shtein over 2 years ago

Fixes #15846 - Renamed use_puppet_default to omit in LookupValue

Revision a56e7de3 (diff)
Added by Dominic Cleal over 2 years ago

refs #15846 - hide API deprecation warnings in tests

Revision 63b92169 (diff)
Added by Dominic Cleal over 2 years ago

refs #15846 - correct spelling of parameter

History

#1 Updated by Shimon Shtein almost 3 years ago

  • Related to Tracker #9596: Extract Puppet functionality to a plugin added

#2 Updated by Ivan Necas almost 3 years ago

  • Status changed from New to Assigned
  • Assignee set to Shimon Shtein
  • Target version set to 1.7.0

#3 Updated by Shimon Shtein almost 3 years ago

  • Related to Refactor #12822: Separate LookupValue to different classes per LookupKey type added

#4 Updated by The Foreman Bot almost 3 years ago

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

#5 Updated by Ivan Necas almost 3 years ago

  • Target version changed from 1.7.0 to 1.6.1

#6 Updated by Shimon Shtein over 2 years ago

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

#7 Updated by Dominic Cleal over 2 years ago

  • Category set to Smart Variables
  • Legacy Backlogs Release (now unused) set to 189

#8 Updated by The Foreman Bot over 2 years ago

  • Pull request https://github.com/theforeman/foreman/pull/3833 added

#9 Updated by The Foreman Bot over 2 years ago

  • Pull request https://github.com/theforeman/foreman/pull/3870 added

Also available in: Atom PDF