Project

General

Profile

Actions

Bug #37184

closed

"Registered Content Hosts" report generates incorrect hosts' kernel version

Added by Hao Yu 8 months ago. Updated 4 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Reporting
Target version:
-
Fixed in Releases:
Found in Releases:

Description

Cloned from bugzilla https://bugzilla.redhat.com/show_bug.cgi?id=2265062

Description of problem:
Old or incorrect host kernel versions are printed in the generated report.

Multiple services are reporting kernel version facts to the Satellite, such as Ansible, Puppet and RHSM. While resolving the kernel version for a host, Satellite will pick the first one order by fact type. Since "Ansible" starts with 'A', kernel version reported by Ansible will normally be picked. If Ansible roles job is not run after the last host kernel update, then old or wrong kernel version will be reported.

How reproducible:
Easy

Steps to Reproduce:
1. Register a host to the Satellite.
2. Run Ansible roles against the host
3. Update the kernel of the host

Actual results:
Report template generated the old kernel version of the host.

Expected results:
Report template generated the upgraded kernel version of the host.

Additional info:

We can run the following hammer command to query all reported kernel versions of a host.

hammer fact list --search "host = content_host.example.com and (name = kernelrelease or name = ansible_kernel or name = kernel::release or name = uname::release)" 

As we can see in the example below, Ansible fact has the outdated kernel version.

----------------------------|----------------|-----------------------------
HOST                        | FACT           | VALUE
----------------------------|----------------|-----------------------------
content_host.example.com    | ansible_kernel | 3.10.0-1062.4.1.el7.x86_64  <==============
content_host.example.com    | uname::release | 3.10.0-1160.105.1.el7.x86_64
----------------------------|----------------|-----------------------------

As we can see below, the results are sorted by "fact_names.type" which means Ansible fact type always took precedence over other fact types.

# /usr/share/foreman/app/models/host/base.rb
module Host
  class Base < ApplicationRecord
    <snip>
    has_one :kernel_release, -> { joins(:fact_name).where({ 'fact_names.name' => KERNEL_RELEASE_FACTS }).order('fact_names.type') }, :class_name => '::FactValue', :foreign_key => 'host_id'

Actions

Also available in: Atom PDF