Bug #5690

Taxonomy fact name not retrieved from settings

Added by Stephen Benjamin about 4 years ago. Updated 7 days ago.

Status:Closed
Priority:Normal
Assignee:Stephen Benjamin
Category:Facts
Target version:1.5.1
Difficulty: Team Backlog:
Triaged: Fixed in Releases:
Bugzilla link:1107695 Found in Releases:
Pull request:

Description

In app/models/host/base.rb:

taxonomy_from_fact = taxonomy_class.find_by_title(facts["#{taxonomy}_fact"])

should be

taxonomy_from_fact = taxonomy_class.find_by_title(facts[Setting["#{taxonomy}_fact"]])

Related issues

Related to Foreman - Bug #3214: No location / Organization if host added by default from ... Closed 10/08/2013
Duplicated by Foreman - Bug #5699: The "location_fact" setting does not actually look for a ... Duplicate 05/13/2014

Associated revisions

Revision ea450c06
Added by Stephen Benjamin about 4 years ago

fixes #5690 - determine host taxonomy fact based on setting value

Revision 9f681976
Added by Stephen Benjamin about 4 years ago

fixes #5690 - determine host taxonomy fact based on setting value

(cherry picked from commit ea450c06aba11e03e6a5a926950b8c0cff1b8f2c)

History

#1 Updated by Dominic Cleal about 4 years ago

  • Legacy Backlogs Release (now unused) set to 16

#2 Updated by Dominic Cleal about 4 years ago

  • Related to Bug #3214: No location / Organization if host added by default from puppet run added

#4 Updated by Stephen Benjamin about 4 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Stephen Benjamin

#5 Updated by Dominic Cleal about 4 years ago

  • Target version set to 1.8.3

#6 Updated by Dominic Cleal about 4 years ago

  • Duplicated by Bug #5699: The "location_fact" setting does not actually look for a custom fact added

#7 Updated by Stephen Benjamin about 4 years ago

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

#8 Updated by Bryan Kearney about 4 years ago

  • Bugzilla link set to https://bugzilla.redhat.com/show_bug.cgi?id=1107695

Also available in: Atom PDF