Project

General

Profile

Actions

Bug #16591

closed

Smart matchers with multiple, more specific facts match on less specific facts

Added by TJ Guthrie over 7 years ago. Updated about 5 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Parameters
Target version:
-
Difficulty:
Triaged:
No
Fixed in Releases:
Found in Releases:

Description

Hello,

I think I am running into a bug when overriding a puppet class parameter. I am having trouble getting the matchers to work.

Say you have a set of facts as a priority in the order section that a host matches. The bug occurs when the following conditions are met:
- you have a subset of facts as a priority in the order section that the host also matches
- that subset is a lower priority than the original set in the order section
- there is a value present to the subset of facts in the matcher section
- the subset is listed before the original set in the matcher section

I attached images showing an example.


Files

SmartClassParameter.png View SmartClassParameter.png 107 KB Shows manufacturer,operatingsystemmajrelease,osfamily at a higher priority than manufacturer TJ Guthrie, 09/17/2016 12:58 PM
FactMatch.png View FactMatch.png 205 KB Shows the host eng-fs01 matching the facts TJ Guthrie, 09/17/2016 12:58 PM
ParameterValue.png View ParameterValue.png 158 KB Shows the incorrect value getting assigned to the parameter TJ Guthrie, 09/17/2016 12:58 PM
Actions #1

Updated by Dominic Cleal over 7 years ago

  • Subject changed from smart class parameter matchers not working to Smart matchers with multiple, more specific facts match on less specific facts
  • Category set to Parameters
Actions #2

Updated by Tomer Brisker about 5 years ago

  • Bugzilla link set to 1670002
Actions #3

Updated by The Foreman Bot about 5 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Tomer Brisker
  • Pull request https://github.com/theforeman/foreman/pull/6439 added
Actions #4

Updated by Marek Hulán about 5 years ago

  • Fixed in Releases 1.22.0 added
Actions #5

Updated by Tomer Brisker about 5 years ago

  • Status changed from Ready For Testing to Closed
Actions

Also available in: Atom PDF