Project

General

Profile

Feature #33419

Feature #33329: Refactor fact parsers

Add fact parser registry

Added by Dominik Matoulek over 1 year ago. Updated over 1 year ago.

Status:
Closed
Priority:
Normal
Category:
Facts
Target version:
-
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

Description

Since the parsers are in core. It's a good idea to move registering of parsers to a separate component.

Associated revisions

Revision fcd63a38 (diff)
Added by Dominik Matoulek over 1 year ago

Fixes #33419 - Introduce Fact Parser Registry

FactParser class has two purposes in current codebase. It acts as parent
for other fact parses (PuppetFactParser, AnsibleFactParser,
RhsmFactParser...) and also as a registry for registering parsers for
given type/key. This PR introduces FactParserRegistry under
Foreman::Plugin namespace. Thanks to that, the FactParser
can takes care only for facts!

History

#1 Updated by Dominik Matoulek over 1 year ago

  • Description updated (diff)

#2 Updated by The Foreman Bot over 1 year ago

  • Assignee set to Dominik Matoulek
  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/8762 added

#3 Updated by The Foreman Bot over 1 year ago

  • Pull request https://github.com/theforeman/foreman/pull/8781 added

#4 Updated by The Foreman Bot over 1 year ago

  • Fixed in Releases 3.1.0 added

#5 Updated by Dominik Matoulek over 1 year ago

  • Status changed from Ready For Testing to Closed

#6 Updated by Amit Upadhye over 1 year ago

  • Category set to Facts

Also available in: Atom PDF