Refactor #13913
Split importing of host and facts into separate methods
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
Associated revisions
History
#1
Updated by The Foreman Bot about 6 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 about 6 years ago
- Related to Refactor #13914: Split import_hosts_and_facts added
#3
Updated by Marek Hulán about 6 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset d4ec64418b8db6093375f8d4b690d506705163f7.
#4
Updated by Dominic Cleal about 6 years ago
- Legacy Backlogs Release (now unused) set to 136
Fixes #13913 - split importing of host and facts