Actions
Bug #21654
closed
Host discovery fails with undefined method `new` for nil:NIlClass
Difficulty:
trivial
Triaged:
No
Pull request:
Description
The problem is that in the old (pre-PR) code importers registry is kept in FactImporter class that gets reloaded. Once the class is reloaded, it looses its default importer registration that point to puppet. See my comment for more information.
This is a regression in the import_facts core codebase introduced by #15409.
Let's add one REAL (non-mocked) test of discovering a host :-)
Updated by Lukas Zapletal over 6 years ago
- Related to Bug #21653: Unable to render host without memory info added
Updated by Lukas Zapletal over 6 years ago
As per IRC, the issue is in Host::Base#import_facts:
type = facts.delete(:_type)It was previously:
type = facts.delete(:_type) || 'puppet'
Updated by Rahul Bajaj about 6 years ago
- Project changed from Discovery to Foreman
- Category deleted (
Discovery plugin)
Updated by Rahul Bajaj about 6 years ago
- Pull request https://github.com/theforeman/foreman/pull/5010 added
Updated by Rahul Bajaj about 6 years ago
- Related to Bug #22705: Must use the ImporterRegistry rather than FactImporter added
Updated by The Foreman Bot about 6 years ago
- Status changed from New to Ready For Testing
Updated by Lukas Zapletal almost 6 years ago
- Triaged set to No
It looks like this happens only when Rails reloads some classes: #21573
Updated by Lukas Zapletal almost 6 years ago
- Related to Bug #21573: Importing facts in development breaks on code reload added
Updated by Rahul Bajaj about 4 years ago
- Status changed from Ready For Testing to Closed
Applied in changeset 5b8e09dc6a465066bfa2c1bb7a9645cbd7994ce1.
Updated by
Updated by Lukas Zapletal over 2 years ago
- Related to Bug #34026: Importing puppet report via the API does not authorize added
Actions