Project

General

Profile

Bug #16449

Taxonomy is detected too late during discovery

Added by Lukas Zapletal almost 4 years ago. Updated about 1 year ago.

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

Description

The problem is that on the host the set_taxonomy from facts is only done after the facts that set the interfaces is done. And in the set of the interfaces there is a check that validates that the host.location must be included in the subnet.location. This will ofcourse fail because the host.location is then still NULL.

From https://bugzilla.redhat.com/show_bug.cgi?id=1349364


Related issues

Related to Foreman - Refactor #15409: Refactor (puppet) facts out of host classes.Closed
Related to Foreman - Bug #26038: Puppet importer refactoring changed taxonomy flowClosed

Associated revisions

Revision c49eb07f (diff)
Added by Lukas Zapletal almost 4 years ago

Fixes #16449 - reordered taxonomy detection during import

Revision 860e0c7d (diff)
Added by Lukas Zapletal almost 4 years ago

Fixes #16449 - reordered taxonomy detection during import

(cherry picked from commit c49eb07f93f6821d7a7cad518ef537cd9ca41478)

History

#1 Updated by Lukas Zapletal almost 4 years ago

  • Project changed from Discovery to Foreman
  • Category changed from Discovery plugin to Facts

This needs to be changed in core actually.

#2 Updated by The Foreman Bot almost 4 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Lukas Zapletal
  • Pull request https://github.com/theforeman/foreman/pull/3816 added

#3 Updated by Daniel Lobato Garcia almost 4 years ago

  • Target version set to 1.6.2

#4 Updated by Lukas Zapletal almost 4 years ago

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

#5 Updated by Dominic Cleal almost 4 years ago

  • Legacy Backlogs Release (now unused) set to 160

#6 Updated by Dominic Cleal almost 4 years ago

  • Legacy Backlogs Release (now unused) changed from 160 to 190

#7 Updated by Lukas Zapletal over 1 year ago

  • Related to Refactor #15409: Refactor (puppet) facts out of host classes. added

#8 Updated by Lukas Zapletal over 1 year ago

  • Related to Bug #26038: Puppet importer refactoring changed taxonomy flow added

Also available in: Atom PDF