Project

General

Profile

Actions

Refactor #15846

closed

Rename use_puppet_default in lookup values to skip_foreman

Added by Shimon Shtein over 8 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Parameters
Target version:
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 3 (2 open1 closed)

Related to Foreman - Tracker #9596: Extract Puppet functionality to a pluginNew03/01/2015

Actions
Related to Foreman - Refactor #12822: Separate LookupValue to different classes per LookupKey typeNew12/15/2015Actions
Related to Foreman - Refactor #28583: Remove `use_puppet_default` api paramsClosedTomer BriskerActions
Actions #1

Updated by Shimon Shtein over 8 years ago

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

Updated by Ivan Necas over 8 years ago

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

Updated by Shimon Shtein over 8 years ago

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

Updated by The Foreman Bot over 8 years ago

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

Updated by Ivan Necas over 8 years ago

  • Target version changed from 1.7.0 to 1.6.1
Actions #6

Updated by Shimon Shtein over 8 years ago

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

Updated by Dominic Cleal over 8 years ago

  • Category set to Parameters
  • Translation missing: en.field_release set to 189
Actions #8

Updated by The Foreman Bot over 8 years ago

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

Updated by The Foreman Bot about 8 years ago

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

Updated by Tomer Brisker almost 5 years ago

Actions

Also available in: Atom PDF