puppetlabs / puppetlabs-postgresql

Puppet module for managing PostgreSQL
Apache License 2.0
228 stars 614 forks source link

Add EL9 support #1591

Closed bastelfreak closed 3 months ago

bastelfreak commented 6 months ago

Summary

Provide a detailed description of all the changes present in this pull request.

Additional Context

Add any additional context about the problem here.

Related Issues (if any)

Mention any related issues or pull requests.

Checklist

SimonHoenscheid commented 6 months ago

EL9 manages the default postgres in dnf modules, the postgres repo does not.

Recommended way is to disable the postgres dnf modules and install from the postgres repo.

the postgres repo requires a version which is defined via postgresql::globals::version, this is also the default version which is passed to the dnf module via the globals_version variable, which would need a value of 'disabled' here, to act correcty, the globals class requires a patch to make this possible.

My suggestion is to add an optional_parameter like dnf_module_version, unless set real_dnf_module_version defaults to $version/global_version.

https://github.com/puppetlabs/puppetlabs-postgresql/blob/main/manifests/globals.pp#L289