sous-chefs / rsyslog

Development repository for the rsyslog cookbook
https://supermarket.chef.io/cookbooks/rsyslog
Apache License 2.0
65 stars 199 forks source link

New TLS driver functionality breaks recipe on RHEL7 hosts #178

Closed petracvv closed 2 years ago

petracvv commented 3 years ago

:ghost: Brief Description

The new functionality that allows you to select the rsyslog TLS driver via the default['rsyslog']['tls_driver'] = 'ossl' attribute broke runs on RHEL7. This is because the rsyslog-openssl package is not available on that platform and the default is set to use openssl; RHEL/CentOS uses only the gnutls rsyslog implementation.

:pancakes: Cookbook version

v7.4.0

:woman_cook: Chef-Infra Version

Still on 14.13.11 :(

:tophat: Platform details

RHEL7. Also present on RHEL8

Steps To Reproduce

Steps to reproduce the behavior:

Run the recipe on RHEL7/Centos7 with tls enabled and all other attributes default.

:police_car: Expected behavior

The recipe run completes

:heavy_plus_sign: Additional context

It may be useful to have a test case for TLS on/off in the kitchen suite for this cookbook. Would have caught this bug.

ramereth commented 3 years ago

@petracvv If you can make a PR which addresses this, we can certainly review and merge it!