Project

General

Profile

Refactor #35985

Move katello certificate integration into puppet-puppet

Added by Ewoud Kohl van Wijngaarden 2 months ago. Updated 2 days ago.


Description

Today we have --puppet-foreman-proxy-content-puppet true, and then the user needs to supply certificate paths to our theforeman/puppet module. Instead we should move this integration into our theforeman/puppet module.

I'm proposing a boolean parameter which can be set via Hiera. Then in our installer's Hiera configuration we can set this for the appropriate scenarios.


Related issues

Related to Installer - Feature #33337: For Katello scenario, do not deploy Puppet by defaultClosed
Related to Foreman Maintain - Bug #36230: `foreman-maintain plugin purge-puppet` is using `foreman-proxy-content-puppet` which is no longer availableClosed

Associated revisions

Revision 49fb1afb (diff)
Added by Ewoud Kohl van Wijngaarden 2 months ago

Refs #35985 - Implement optional Katello integration

Katello uses a different certificate structure. This moves over the
integration bits from puppet-foreman_proxy_content to this module. It
also means fewer variables need to be set in the installer itself.

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

Refs #35985 - Rely on theforeman/puppet to do integration

The integration is moved into theforeman/puppet so this is no longer
needed.

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

Fixes #35985 - Use Katello integration in theforeman/puppet

History

#1 Updated by The Foreman Bot 2 months ago

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

#2 Updated by The Foreman Bot 2 months ago

  • Pull request https://github.com/theforeman/puppet-foreman_proxy_content/pull/440 added

#3 Updated by The Foreman Bot 2 months ago

  • Pull request https://github.com/theforeman/foreman-installer/pull/828 added

#4 Updated by The Foreman Bot 2 months ago

  • Pull request https://github.com/theforeman/foreman-documentation/pull/1947 added

#5 Updated by Ewoud Kohl van Wijngaarden 2 months ago

  • Related to Feature #33337: For Katello scenario, do not deploy Puppet by default added

#6 Updated by The Foreman Bot 2 months ago

  • Pull request https://github.com/theforeman/forklift/pull/1649 added

#7 Updated by The Foreman Bot about 1 month ago

  • Fixed in Releases 3.6.0 added

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

  • Status changed from Ready For Testing to Closed

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

  • Triaged changed from No to Yes

#10 Updated by Ewoud Kohl van Wijngaarden 8 days ago

  • Related to Bug #36230: `foreman-maintain plugin purge-puppet` is using `foreman-proxy-content-puppet` which is no longer available added

#11 Updated by The Foreman Bot 2 days ago

  • Pull request https://github.com/theforeman/foreman-documentation/pull/2108 added

Also available in: Atom PDF