Project

General

Profile

Bug #5690

Taxonomy fact name not retrieved from settings

Added by Stephen Benjamin about 5 years ago. Updated about 1 year ago.

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

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 puppet runClosed2013-10-08
Has duplicate Foreman - Bug #5699: The "location_fact" setting does not actually look for a custom factDuplicate2014-05-13

Associated revisions

Revision ea450c06 (diff)
Added by Stephen Benjamin about 5 years ago

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

Revision 9f681976 (diff)
Added by Stephen Benjamin about 5 years ago

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

(cherry picked from commit ea450c06aba11e03e6a5a926950b8c0cff1b8f2c)

History

#1 Updated by Dominic Cleal about 5 years ago

  • Legacy Backlogs Release (now unused) set to 16

#2 Updated by Dominic Cleal about 5 years ago

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

#4 Updated by Stephen Benjamin about 5 years ago

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

#5 Updated by Dominic Cleal about 5 years ago

  • Target version set to 1.8.3

#6 Updated by Dominic Cleal about 5 years ago

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

#7 Updated by Stephen Benjamin about 5 years ago

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

#8 Updated by Bryan Kearney about 5 years ago

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

Also available in: Atom PDF