Project

General

Profile

Refactor #13913

Split importing of host and facts into separate methods

Added by Marek Hulán almost 5 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Importers
Target version:
Difficulty:
Triaged:
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

Host#import_host_and_facts does to many things. While importing host is class responsibility, facts importing is host instance responsibility. This makes it hard if plugins like foreman-tasks wants to make facts importing a background process while keeping host importing as is. In fact it makes the foreman tasks plugin to re-implement the importing which gets out of sync every now and then.


Related issues

Related to Discovery - Refactor #13914: Split import_hosts_and_factsClosed2016-02-26

Associated revisions

Revision d4ec6441 (diff)
Added by Marek Hulán almost 5 years ago

Fixes #13913 - split importing of host and facts

History

#1 Updated by The Foreman Bot almost 5 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/3242 added

#2 Updated by Lukas Zapletal almost 5 years ago

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

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

#4 Updated by Dominic Cleal almost 5 years ago

  • Legacy Backlogs Release (now unused) set to 136

Also available in: Atom PDF