Project

General

Profile

Refactor #15711

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

Added by Shimon Shtein over 2 years ago. Updated 4 months ago.

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

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 pluginNew2015-03-01

Related to Foreman - Bug #20066: global params are not evaluated in ENC outputClosed2017-06-21
Related to Foreman Remote Execution - Refactor #20101: Update the input renderer to new HostInfo providerClosed2017-06-23
Blocks Foreman - Feature #2111: Add Foreman configuration values to ENC outputNew2013-01-07
Blocks Katello - Refactor #19586: Move host info extension to provider.Closed2017-05-18

Associated revisions

Revision 60cf5a37 (diff)
Added by Shimon Shtein over 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 over 2 years ago

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

#2 Updated by Dominic Cleal over 2 years ago

  • Category set to External Nodes

#3 Updated by Shimon Shtein about 2 years ago

  • Target version set to 1.7.0

#4 Updated by The Foreman Bot about 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 about 2 years ago

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

#6 Updated by Ivan Necas about 2 years ago

  • Target version changed from 1.7.0 to 1.6.1

#7 Updated by Ivan Necas about 2 years ago

  • Target version changed from 1.6.1 to 127

#8 Updated by Ivan Necas about 2 years ago

  • Target version changed from 127 to 1.5.1

#9 Updated by Ivan Necas almost 2 years ago

  • Target version changed from 1.5.1 to 1.4.1

#10 Updated by Shimon Shtein over 1 year ago

  • Target version changed from 1.4.1 to 1.13.1

#11 Updated by Shimon Shtein over 1 year ago

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

  • Legacy Backlogs Release (now unused) set to 240

#13 Updated by Shimon Shtein over 1 year ago

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

#14 Updated by Ohad Levy over 1 year ago

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

#15 Updated by Ivan Necas over 1 year ago

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

#16 Updated by Ivan Necas over 1 year ago

  • Target version changed from 1.13.1 to 1.14.1

Also available in: Atom PDF