Project

General

Profile

Refactor #30817

Remove explicit directories from Puppet provisioning setup snippet

Added by Ewoud Kohl van Wijngaarden 5 months ago. Updated 4 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Ewoud Kohl van Wijngaarden
Category:
Templates
Target version:
-
Difficulty:
Triaged:
No
Bugzilla link:
Pull request:
https://github.com/theforeman/foreman/pull/7979
Fixed in Releases:
2.3.0
Found in Releases:

Description

Puppet has built in defaults for paths, at least for the AIO packages. It's not needed to set these in puppet.conf.

Associated revisions

Revision 37d7986a (diff)
Added by Ewoud Kohl van Wijngaarden 4 months ago

Fixes #30817 - Remove AIO dirs in puppet.conf

In Puppet's AIO package the default puppet.conf doesn't contain any of
these directories so it's not needed to set them.

I confirmed this on CentOS 7 with Puppet 5 and 6 as well as on Debian 10
with Puppet 6. Since Puppet 4 is EOL, there's no need to test that. It's
to be expected that Puppet 7 will also not need these directories.

I've left in puppet.conf on Windows and non-AIO. On Debian non-AIO some
directories are still set and I can't verify other platforms.

The digest_algorithm on FIPS is not needed since Puppet 5.4.0. Anything
older is EOL so it can be safely dropped.

History

#1 Updated by The Foreman Bot 5 months ago

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

#2 Updated by The Foreman Bot 4 months ago

  • Fixed in Releases 2.3.0 added

#3 Updated by Ewoud Kohl van Wijngaarden 4 months ago

  • Status changed from Ready For Testing to Closed

Also available in: Atom PDF