Project

General

Profile

Actions

Refactor #30127

closed

Refactor #30581: Extract fact importing from Host class to to HostFactImporter

Extract import_facts entry point from host class and create HostFactImporter

Added by Ondřej Ezr almost 4 years ago. Updated over 3 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Facts
Target version:
-
Fixed in Releases:
Found in Releases:

Description

The fact import is now tied with a Host model. We might want to extract it as the clas is too big and it should not be its responsibility.


Related issues 4 (1 open3 closed)

Related to Foreman - Tracker #30128: Remove STI from Host modelNew

Actions
Related to Discovery - Bug #30577: Tests fail due to HostFactImporter#import_factsClosedLukas ZapletalActions
Related to Foreman - Bug #31364: undefined method `without_orchestration'ClosedTomer BriskerActions
Related to Foreman - Refactor #34063: Remove the deprecated Host#import_facts methodClosedTomer BriskerActions
Actions #1

Updated by Ondřej Ezr almost 4 years ago

Actions #2

Updated by The Foreman Bot almost 4 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Ondřej Ezr
  • Pull request https://github.com/theforeman/foreman/pull/7749 added
Actions #3

Updated by Lukas Zapletal over 3 years ago

  • Related to Bug #30577: Tests fail due to HostFactImporter#import_facts added
Actions #4

Updated by Ondřej Ezr over 3 years ago

  • Subject changed from Extract fact import from host class to Extract import_facts entry point from host class and create HostFactImporter
  • Parent task set to #30581
  • Difficulty set to medium
  • Triaged changed from No to Yes
Actions #5

Updated by The Foreman Bot over 3 years ago

  • Pull request https://github.com/theforeman/foreman/pull/7883 added
Actions #6

Updated by The Foreman Bot over 3 years ago

  • Fixed in Releases 2.2.0 added
Actions #7

Updated by Ondřej Ezr over 3 years ago

  • Status changed from Ready For Testing to Closed
Actions #8

Updated by The Foreman Bot over 3 years ago

  • Pull request https://github.com/theforeman/foreman/pull/7975 added
Actions #9

Updated by Tomer Brisker over 3 years ago

  • Related to Bug #31364: undefined method `without_orchestration' added
Actions #10

Updated by Tomer Brisker over 2 years ago

  • Related to Refactor #34063: Remove the deprecated Host#import_facts method added
Actions

Also available in: Atom PDF