Bug #21123
Fact import code does not work
Fixed in Releases:
Found in Releases:
Description
Due to refactoring in core:
ArgumentError: wrong number of arguments (given 2, expected 3) app/models/host/base.rb:174:in `populate_fields_from_facts' /home/lzap/work/foreman_discovery/app/models/host/discovered.rb:89:in `populate_fields_from_facts' app/models/host/base.rb:157:in `parse_facts' app/models/host/base.rb:146:in `import_facts' /home/lzap/work/foreman_discovery/app/models/host/discovered.rb:74:in `import_facts' /home/lzap/work/foreman_discovery/app/models/host/discovered.rb:66:in `import_host'
Related issues
Associated revisions
History
#1
Updated by Lukas Zapletal almost 5 years ago
- Related to Refactor #15409: Refactor (puppet) facts out of host classes. added
#2
Updated by The Foreman Bot almost 5 years ago
- Status changed from New to Ready For Testing
- Assignee set to Shimon Shtein
- Pull request https://github.com/theforeman/foreman_discovery/pull/381 added
#3
Updated by The Foreman Bot almost 5 years ago
- Pull request https://github.com/theforeman/foreman_discovery/pull/382 added
#4
Updated by Anonymous almost 5 years ago
- Pull request deleted (
https://github.com/theforeman/foreman_discovery/pull/382)
#5
Updated by Shimon Shtein almost 5 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset foreman_discovery|e2a51e6e55c71ed8a6fc9dbba882393eeded7666.
Fixes #21123 - Changed populate_fields_from_facts signature
To reflect changes from #3649.