Refactor #15711

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

Added by Shimon Shtein 11 months ago. Updated 5 days ago.

Status:Closed
Priority:Normal
Assigned To:Shimon Shtein
Category:External Nodes
Target version:Team Ivan Iteration 14
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 Ready For Testing 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. Ready For Testing 05/18/2017

Associated revisions

Revision 60cf5a37
Added by Shimon Shtein 5 days 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 11 months ago

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

#2 Updated by Dominic Cleal 11 months ago

  • Category set to External Nodes

#3 Updated by Shimon Shtein 11 months ago

  • Target version set to Team Ivan Iteration 1

#4 Updated by The Foreman Bot 11 months 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 11 months ago

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

#6 Updated by Ivan Necas 10 months ago

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

#7 Updated by Ivan Necas 10 months ago

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

#8 Updated by Ivan Necas 9 months ago

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

#9 Updated by Ivan Necas 8 months ago

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

#10 Updated by Shimon Shtein about 1 month ago

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

#11 Updated by Shimon Shtein about 1 month ago

#12 Updated by Marek Hulán 5 days ago

  • Release set to 1.16.0

#13 Updated by Shimon Shtein 5 days ago

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

#14 Updated by Ohad Levy 3 days ago

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

#15 Updated by Ivan Necas about 6 hours ago

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

Also available in: Atom PDF