Refactor #15711

Remove inheritance from classification system and move type specific functionality to providers framework.

Added by Shimon Shtein over 1 year ago. Updated 4 months ago.

Status:Closed
Priority:Normal
Assigned To:Shimon Shtein
Category:External Nodes
Target version:Team Ivan Iteration 16
Difficulty: Bugzilla link:
Found in release: Pull request:https://github.com/theforeman/foreman/pull/3701
Story points-
Velocity based estimate-
Release1.16.0Release relationshipAuto

Description

Classification should be a base service provided by the core, but the actual functionality should be provided by various plugins.
One step that should be refactored out of classifier is the output formatter. It should be defined by the ENC type requested.
Other thing that should be refactored is a provider for valid LookupKeys for a given host.


Related issues

Related to Foreman - Tracker #9596: Extract Puppet functionality to a plugin New 03/01/2015
Related to Foreman - Bug #20066: global params are not evaluated in ENC output Closed 06/21/2017
Related to Foreman Remote Execution - Refactor #20101: Update the input renderer to new HostInfo provider Closed 06/23/2017
Blocks Foreman - Feature #2111: Add Foreman configuration values to ENC output New 01/07/2013
Blocks Katello - Refactor #19586: Move host info extension to provider. Closed 05/18/2017

Associated revisions

Revision 60cf5a37
Added by Shimon Shtein 4 months ago

Fixes #15711 - Redesigned host info to providers.

Now you can add your own info by registering a
host method that will return info hash.

History

#1 Updated by Shimon Shtein over 1 year ago

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

#2 Updated by Dominic Cleal over 1 year ago

  • Category set to External Nodes

#3 Updated by Shimon Shtein about 1 year ago

  • Target version set to Team Ivan Iteration 1

#4 Updated by The Foreman Bot about 1 year ago

  • Status changed from New to Ready For Testing
  • Assigned To set to Shimon Shtein
  • Pull request https://github.com/theforeman/foreman/pull/3701 added

#5 Updated by Marek Hulán about 1 year ago

  • Blocks Feature #2111: Add Foreman configuration values to ENC output added

#6 Updated by Ivan Necas about 1 year ago

  • Target version changed from Team Ivan Iteration 1 to Team Ivan Iteration 2

#7 Updated by Ivan Necas about 1 year ago

  • Target version changed from Team Ivan Iteration 2 to Team Ivan Iteration 3

#8 Updated by Ivan Necas about 1 year ago

  • Target version changed from Team Ivan Iteration 3 to Team Ivan Iteration 4

#9 Updated by Ivan Necas 12 months ago

  • Target version changed from Team Ivan Iteration 4 to Team Ivan Iteration 5

#10 Updated by Shimon Shtein 5 months ago

  • Target version changed from Team Ivan Iteration 5 to Team Ivan Iteration 14

#11 Updated by Shimon Shtein 5 months ago

#12 Updated by Marek Hulán 4 months ago

  • Release set to 1.16.0

#13 Updated by Shimon Shtein 4 months ago

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

#14 Updated by Ohad Levy 4 months ago

  • Related to Bug #20066: global params are not evaluated in ENC output added

#15 Updated by Ivan Necas 4 months ago

  • Related to Refactor #20101: Update the input renderer to new HostInfo provider added

#16 Updated by Ivan Necas 4 months ago

  • Target version changed from Team Ivan Iteration 14 to Team Ivan Iteration 16

Also available in: Atom PDF