Project

General

Profile

Feature #33419

Feature #33329: Refactor fact parsers

Add fact parser registry

Added by Dominik Matoulek about 1 month ago. Updated 25 days ago.

Status:
Closed
Priority:
Normal
Category:
-
Target version:
-
Difficulty:
Triaged:
No
Bugzilla link:

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 25 days 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 about 1 month ago

  • Description updated (diff)

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

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

#4 Updated by The Foreman Bot 25 days ago

  • Fixed in Releases 3.1.0 added

#5 Updated by Dominik Matoulek 25 days ago

  • Status changed from Ready For Testing to Closed

Also available in: Atom PDF