Refactor #15711

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

Added by Shimon Shtein almost 2 years ago. Updated 6 days ago.

Status:Closed
Priority:Normal
Assignee:Shimon Shtein
Category:External Nodes
Target version:1.16.0
Difficulty: Team Backlog:
Triaged: Fixed in Releases:
Bugzilla link: Found in Releases:
Pull request:https://github.com/theforeman/foreman/pull/3701

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 about 1 year 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 almost 2 years ago

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

#2 Updated by Dominic Cleal almost 2 years ago

  • Category set to External Nodes

#3 Updated by Shimon Shtein almost 2 years ago

  • Target version set to 1.7.0

#4 Updated by The Foreman Bot almost 2 years ago

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

#5 Updated by Marek Hulán almost 2 years ago

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

#6 Updated by Ivan Necas almost 2 years ago

  • Target version changed from 1.7.0 to 1.6.1

#7 Updated by Ivan Necas almost 2 years ago

  • Target version changed from 1.6.1 to 127

#8 Updated by Ivan Necas almost 2 years ago

  • Target version changed from 127 to 1.5.1

#9 Updated by Ivan Necas over 1 year ago

  • Target version changed from 1.5.1 to 1.4.1

#10 Updated by Shimon Shtein about 1 year ago

  • Target version changed from 1.4.1 to 1.13.1

#11 Updated by Shimon Shtein about 1 year ago

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

  • Legacy Backlogs Release (now unused) set to 240

#13 Updated by Shimon Shtein about 1 year ago

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

#14 Updated by Ohad Levy about 1 year ago

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

#15 Updated by Ivan Necas about 1 year ago

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

#16 Updated by Ivan Necas about 1 year ago

  • Target version changed from 1.13.1 to 1.14.1

Also available in: Atom PDF