You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
@olifreproposed to use $facts['os']['family'] == 'RedHat' rather than $facts['os']['name'] == 'CentOS' in the manifest/params.pp code.
A potential loophole is that Fedora hosts would probably also belong to the RedHat family and the code will not work on ancient Fedora Linux 7 and Fedora Linux 8 (according to this page, they were released before the publication of Python 3). But probably this is rather a theoretical than a practical problem.
Apart from this, the proposed solution is definitely nicer than maintaining a list of RHEL clone names.
But probably this is rather a theoretical than a practical problem.
I agree, effectively, this means we are not supporting ancient Fedora releases ;-).
Given the currently known rebuilds (AlmaLinux, Rocky Linux) and also other related RedHat family distros (Oracle Linux, Amazon Linux, Scientific Linux) and the rate at which these change, going for the RedHat family seems to be the only maintainable option. We could (later on) add the list of tested distributions to our metadata.json and add unit tests to communicate more clearly which distros we have tested things on.
Further down the road, we may want to migrate to the hiera-based approach which modules like puppet-systemd are using to provide OS-specific defaults (this also allows to get rid of params.pp altogether and use a yaml-based syntax instead).
Given the developments in the CentOS Project we should support a wider range of RHEL clones.
The text was updated successfully, but these errors were encountered: