Project

General

Profile

Refactor #36131

Remove Puppet host_param options from template

Added by Ewoud Kohl van Wijngaarden about 1 month ago. Updated about 1 month ago.

Status:
Closed
Priority:
Normal
Category:
Unattended installations
Target version:
-
Difficulty:
Triaged:
Yes
Bugzilla link:
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

Description

In #35187 the puppet.conf snippet added explicit host_param options for the Puppet server and Puppet CA server, but the host_puppet_server and host_puppet_ca_server macros were supposed to already use those parameters. In #35306 a bug was solved, so they are now properly respected. This means the explicit host_param parts are no longer needed.


Related issues

Related to Foreman - Feature #35187: Permit puppet settings when puppet-plugins are not loadedClosed
Related to Foreman - Bug #35306: host_puppet_* template macros ignores host_paramClosed

Associated revisions

Revision 58309096 (diff)
Added by Ewoud Kohl van Wijngaarden about 1 month ago

Fixes #36131 - Remove puppet params for non-integrated

This reverts commit 04a93af8dacb7cc0611d6e1489d0ce88cd3b6090 because
eebd309f89c71567c2b7e6d4e4fc8f4b00d0edc6 fixed it properly.

History

#1 Updated by Ewoud Kohl van Wijngaarden about 1 month ago

  • Related to Feature #35187: Permit puppet settings when puppet-plugins are not loaded added

#2 Updated by Ewoud Kohl van Wijngaarden about 1 month ago

  • Related to Bug #35306: host_puppet_* template macros ignores host_param added

#3 Updated by The Foreman Bot about 1 month ago

  • Assignee set to Ewoud Kohl van Wijngaarden
  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/9630 added

#4 Updated by The Foreman Bot about 1 month ago

  • Fixed in Releases 3.6.0 added

#5 Updated by Ewoud Kohl van Wijngaarden about 1 month ago

  • Status changed from Ready For Testing to Closed

#6 Updated by Ewoud Kohl van Wijngaarden about 1 month ago

  • Triaged changed from No to Yes

Also available in: Atom PDF