Project

General

Profile

Bug #34525

Ensure that the insights snippet is being called by honoring the value of host_registration_insights parameter

Added by Sayan Das 10 months ago. Updated 9 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Templates
Target version:
-

Description

Description of problem:

The insights registration steps happen during host build even if the host_registration_insights parameter is set to false in Satellite 7.0

The problem is that we call the snippet "insights" directly in "kickstart default" template without validating the conditional.

Steps to Reproduce:

1. Build a Satellite 7 beta and configure it for PXE provisioning

2. Set the value of "host_registration_insights" boolean to false in global parameters.

3. Create and submit the host for build.

4. Once the host is built either render the Provisioning template against the host or else check the /root/install.post.log file on the host.

Actual results:

We will be able to see that even after the host_registration_insights parameter is set to false, as a part of build process, the insights-client package was installed and the registration was attempted.

Expected results:

The build process should honor the value of host_registration_insights boolean and based on that, it should execute the insights-related actions in the post-provisioning steps.

Additional info:

In kickstart_default template and kickstart_ovirt template, we directly call the snippet for insights in this way:

<%= snippet 'insights' -%>

but it should be called in this way:

<% if host_param_true?('host_registration_insights') ->
<%= snippet 'insights' -%>
<
end -%>

Associated revisions

Revision db399691 (diff)
Added by Sayan Das 9 months ago

Fixes #34525 - Call insights snippet by honoring host_registration_insights param during system build

History

#1 Updated by The Foreman Bot 10 months ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/9121 added

#2 Updated by Sayan Das 9 months ago

  • Assignee set to Sayan Das

#3 Updated by The Foreman Bot 9 months ago

  • Fixed in Releases 3.3.0 added

#4 Updated by Sayan Das 9 months ago

  • Status changed from Ready For Testing to Closed

#5 Updated by The Foreman Bot 9 months ago

  • Pull request https://github.com/theforeman/foreman/pull/9143 added

#6 Updated by The Foreman Bot 9 months ago

  • Pull request https://github.com/theforeman/foreman/pull/9144 added

#7 Updated by The Foreman Bot 9 months ago

  • Fixed in Releases 3.2.0 added

#8 Updated by The Foreman Bot 9 months ago

  • Fixed in Releases 3.1.3 added

Also available in: Atom PDF